[alsa-devel] [PATCH 17/20] ASoC: S3C: I2Sv2: Define new field for controller features
broonie at opensource.wolfsonmicro.com
Wed Mar 10 12:09:59 CET 2010
On Wed, Mar 10, 2010 at 04:49:06PM +0900, Jassi Brar wrote:
> While adding support for new SoCs' I2S, check for the blocks
> of older SoCs that have similar feature and set the flag for
> that feature.
As a result of this...
> +/* Set this flag for I2S controllers that have the bit IISMOD
> + * bridge/break RCLK signal and external Xi2sCDCLK pin.
> + */
> +#define S3C_FEATURE_CDCLKCON (1 << 0)
...I'd suggest moving this into the patch that adds support for
CDCLKCON (or merging that patch in with this one).
Probably worth namespacing these defines a bit more (eg, S3C_IIS_) since
I'd expect they'll end up being used by the arch/arm code too in the
platform data definition. They ought to be in include/sound or arch/arm
for visibility to arch/arm code too.
More information about the Alsa-devel