On Fri, Jun 06, 2008 at 07:12:40PM -0700, Tony Lindgren wrote: > @@ -208,10 +365,12 @@ static int __init omap2_init_devices(void) > /* please keep these calls, and their implementations above, > * in alphabetical order so they're easier to sort through. > */ > - omap_init_i2c(); > + omap_init_camera(); > omap_init_mbox(); > omap_init_mcspi(); > + omap_hdq_init(); > omap_init_sti(); > + omap_init_sha1_md5(); > > return 0; > } Hmm, looking through this file, looking at: 1. the conditionals on the machine type 2. the CONFIG_ dependencies for CPU driver versions and then considering... what if a platform only supports one mcspi, should we be registering all mcspi's... Wouldn't it be better to let the platforms themselves decide which devices should be registered, just like AT91 does and now PXA? > +#ifndef __ASM_ARCH_CONTROL_H > +#define __ASM_ARCH_CONTROL_H > + > #include <asm/arch/io.h> > > +#ifndef __ASSEMBLY__ > #define OMAP242X_CTRL_REGADDR(reg) \ > - (void __iomem *)IO_ADDRESS(OMAP242X_CTRL_BASE + (reg)) > + (__force void __iomem *)IO_ADDRESS(OMAP242X_CTRL_BASE + (reg)) > #define OMAP243X_CTRL_REGADDR(reg) \ > - (void __iomem *)IO_ADDRESS(OMAP243X_CTRL_BASE + (reg)) > + (__force void __iomem *)IO_ADDRESS(OMAP243X_CTRL_BASE + (reg)) > #define OMAP343X_CTRL_REGADDR(reg) \ > - (void __iomem *)IO_ADDRESS(OMAP343X_CTRL_BASE + (reg)) > + (__force void __iomem *)IO_ADDRESS(OMAP343X_CTRL_BASE + (reg)) There's those big warning casts again. If you moved that inside IO_ADDRESS() then you wouldn't need all these #ifndef __ASSEMBLY__ stuff whereever you use that macro. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
