On Tue, Mar 11, 2014 at 11:19:51AM +0000, 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. > > [ CS42888 supports both I2C and SPI control ports. As initial patch, > this patch only adds the support for I2C. ] > > Signed-off-by: Nicolin Chen <guangyu.c...@freescale.com> > --- > > Changelog > v2: > * Separated driver into I2C and main part drivers. > * Added error out if failed to get clock. > * Enabled Auto-mute function as default. > * Disabled runtime-pm in remove(). > * Added return value checking for regcache_sync(). > * Fixed some trival parts like 'const' as Lars mentioned. > > .../devicetree/bindings/sound/cs42888.txt | 27 ++ > sound/soc/codecs/Kconfig | 10 + > sound/soc/codecs/Makefile | 4 + > sound/soc/codecs/cs42888-i2c.c | 68 +++ > sound/soc/codecs/cs42888.c | 536 > +++++++++++++++++++++ > sound/soc/codecs/cs42888.h | 213 ++++++++ > 6 files changed, 858 insertions(+) > create mode 100644 Documentation/devicetree/bindings/sound/cs42888.txt > create mode 100644 sound/soc/codecs/cs42888-i2c.c > create mode 100644 sound/soc/codecs/cs42888.c > create mode 100644 sound/soc/codecs/cs42888.h > > diff --git a/Documentation/devicetree/bindings/sound/cs42888.txt > b/Documentation/devicetree/bindings/sound/cs42888.txt > new file mode 100644 > index 0000000..7736527 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/cs42888.txt > @@ -0,0 +1,27 @@ > +CS42888 audio CODEC > + > +Required properties: > + > + - compatible : "cirrus,cs42888" > + > + - reg : the I2C address of the device for I2C > + > + - clocks : phandle to the clock source for MCLK > + > + - clock-names : must contain "mclk".
If you use clock-names, please define the clocks property in terms of clock-names to avoid redundancy. e.g. - clocks: a list of phandles + clock-specifiers, one for each entry in clock-names. It makes the relationship between the two clearer, and makes it far easier to alter the binding in future if required. Cheers, Mark. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
