On 8 September 2011 20:17, Jassi Brar <jaswinder.singh@xxxxxxxxxx> wrote: > On 9 September 2011 00:35, Mans Rullgard <mans.rullgard@xxxxxxxxxx> wrote: >> On 8 September 2011 19:17, Jassi Brar <jassisinghbrar@xxxxxxxxx> wrote: >>> On Thu, Sep 8, 2011 at 11:04 PM, Mans Rullgard <mans.rullgard@xxxxxxxxxx> wrote: >>>> This converts the per-board modules to platform drivers for a >>>> device created by in main platform setup. These drivers call >>>> snd_soc_register_card() directly instead of going via a "soc-audio" >>>> device and the corresponding driver in soc-core. >>>> >>>> Signed-off-by: Mans Rullgard <mans.rullgard@xxxxxxxxxx> >>>> --- >>>> Platform device names fixed. >>>> N8x0 changed to get clocks before registering card. >>>> --- >>>> arch/arm/mach-omap2/board-3430sdp.c | 6 ++ >>>> arch/arm/mach-omap2/board-4430sdp.c | 6 ++ >>>> arch/arm/mach-omap2/board-am3517evm.c | 7 +++ >>>> arch/arm/mach-omap2/board-devkit8000.c | 6 ++ >>>> arch/arm/mach-omap2/board-igep0020.c | 6 ++ >>>> arch/arm/mach-omap2/board-n8x0.c | 6 ++ >>>> arch/arm/mach-omap2/board-omap3beagle.c | 6 ++ >>>> arch/arm/mach-omap2/board-omap3evm.c | 7 +++ >>>> arch/arm/mach-omap2/board-omap3pandora.c | 6 ++ >>>> arch/arm/mach-omap2/board-overo.c | 17 ++++++ >>>> arch/arm/mach-omap2/board-rx51.c | 6 ++ >>>> arch/arm/mach-omap2/board-zoom-peripherals.c | 6 ++ >>> >>> Can't we do by having omap_init_audio() in arch/arm/mach-omap2/devices.c >>> generate a platform device of name depending upon machine_is_* ? >> >> I had the same thought, but I couldn't find a suitable string anywhere. >> Are you suggesting an if(machine_is_foo()) cascade in omap_init_audio()? > Yes. And you could assign same names as you do now in board init files. This would still be omap-specific though, but perhaps that can be acceptable for now. Longer term, I guess this is a candidate for device tree. -- Mans Rullgard Multimedia mans.rullgard@xxxxxxxxxx -- 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