[PATCH 1/4] omap: Clean the serial port defines
Tony Lindgren
tony at atomide.com
Sat Jan 16 14:25:38 EST 2010
* Shilimkar, Santosh <santosh.shilimkar at ti.com> [100115 23:46]:
> Thanks for the nice cleanup.
Thanks for testing :)
> <snip>
> > --- a/arch/arm/plat-omap/include/plat/common.h
> > +++ b/arch/arm/plat-omap/include/plat/common.h
> > @@ -47,6 +47,10 @@ struct omap_globals {
> > void __iomem *prm; /* Power and Reset Management */
> > void __iomem *cm; /* Clock Management */
> > void __iomem *cm2;
> > + unsigned long uart1_phys;
> > + unsigned long uart2_phys;
> > + unsigned long uart3_phys;
> > + unsigned long uart4_phys;
> Considering they are register base address, can
> these be declared as "void __iomem" instead of "unsigned long"
These are physical addresses instead of virtual addresses.
They get ioremapped in mach-omap[12]/serial.c. So they should
not be void __iomem * in omap_globals.
BTW, we should eventually change everything in omap_globals
to be physical addresses.
Regards,
Tony
More information about the linux-arm-kernel
mailing list