Re: [PATCH v2 4/4] OMAP4: DMIC: Add DMIC codec platform devices

[Mark Brown <broonie@opensource.wolfsonmicro.com>]

On Thu, Jan 06, 2011 at 08:00:39AM -0600, David Lambert wrote:
> This creates the DMIC codec platform devices.
> 
> The platform devices create an instance of the driver during boot up.
> 
> Signed-off-by: David Lambert <dlambert@ti.com>

Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

though:

> +static struct platform_device codec_dmic0 = {
> +	.name   = "dmic-codec",
> +	.id     = -1,
> +};

> +static inline void omap_dmic_init(void)
> +{
> +	platform_device_register(&codec_dmic0);
> +}

This feels like bad namespacing as the name is OMAP-generic but it's
actually board specific.

It might also be more sensible to just have an array of platform devices
with one entry and register that rather than bouncing through this
function as it doesn't add anything and it'll get more and more verbose
as more platform devices are added.

It may be that these things are idiomatic, though - I've not looked at
the surrounding code.