On Fri, Sep 25, 2009 at 05:08, <[email protected]> wrote: > --- trunk/sound/soc/codecs/adau1371.c 2009-09-25 09:03:21 UTC (rev 7461) > +++ trunk/sound/soc/codecs/adau1371.c 2009-09-25 09:08:31 UTC (rev 7462) > @@ -421,6 +421,9 @@ > adau1371_write(codec, ADAU1371_RADCMIX, AIN1_SIGNAL_ENA); > adau1371_write(codec, ADAU1371_INALVOL, > cache[ADAU1371_INALVOL]); > adau1371_write(codec, ADAU1371_INARVOL, > cache[ADAU1371_INARVOL]); > + /* Disable other ports*/
please review your comments. there needs to be space after the start and before the end. in other words, dont do "ports*/", do "ports */". > + adau1371_write(codec, ADAU1371_PWRCTLA, reg & ~(PWRCTLA_INBPD |\ > + PWRCTLA_INCPD | PWRCTLA_INDPD)); seems like you like to use a lot of line continuations when they're not necessary. please dont. -mike _______________________________________________ Linux-kernel-commits mailing list [email protected] https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits
