On Monday, August 22, 2011 05:00:15 PM Helmut Raiger wrote: > On 08/22/2011 04:00 PM, Marek Vasut wrote: > > + u32 pdr0 = __REG(CCM_PDR0); > > Can't we do readl() here? > > > >> + > >> + /* divided by HSP_PODF in PDR0 */ > >> + freq /= ((pdr0>> 11)& 0x7) + 1; > > > > Um, don't we have defined constants for those magic numbers already? > > Masks are not defined for i.MX31 (imx-regs.h), so I simply followed the > code, which was in the file. The generic.c file for MX35 however is a > different story. > > To keep things simple, I suggest we leave it at that for now.
And noone's gonna fix it later. No please, define the proper constants please. Thanks in advance > > Helmut > > > > > > -- > Scanned by MailScanner. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
