On Sat, Jan 30, 2010 at 06:23:14PM +0100, Maurus Cuelenaere wrote:
Might want to check your mail setup - the date looks wrong.
> This adds sound support for the SmartQ board.
>
> The hardware consists of a S3C6410 coupled with a WM8987 over I²S. As there's
> no driver for the WM8987 currently, this uses the WM8750 driver (which should
> be register compatible).
There's no need to add a separate driver for a register compatible part,
just add the I2C/SPI IDs to the existing driver. Otherwise we just
duplicate it redundantly. Please update this and the comment in the
driver to reflect that.
Otherwise this looks OK, but there's a couple of things that it'd be
nice to update:
> + dev_info(&smartq_snd_device->dev, "SmartQ WM8987 audio driver inited\n");
Best to remove this, it gets a bit noisy.
> +MODULE_AUTHOR("Maurus Cuelenaere");
Better to add your mail address too.
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
