Re: [PATCH v3] ASoC: cs42888: Add codec driver support

From: Paul Handrigan
Date: Tue Mar 18 2014 - 15:40:33 EST



On Tue, 18 Mar 2014, Brian Austin wrote:

On Tue, 18 Mar 2014, Mark Brown wrote:

On Tue, Mar 11, 2014 at 07:41:31PM +0800, Nicolin Chen wrote:
This patch adds support for the Cirrus Logic CS42888 Audio CODEC that
has four 24-bit A/D and eight 24-bit D/A converters.

Brian, Paul - any review comments on this?

Driver looks good to me. I think we should name it the cs42xx8 since this actually covers CS42448 and CS42888. The CS42888 is derived from the CS42448.

V3 looks good. As Brian said, the name should be changed to cs42xx8.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/