[alsa-devel] [PATCH 7/7] ASoC: soc-cache: Introduce raw bulk write support

Dimitris Papastamos dp at opensource.wolfsonmicro.com
Wed May 11 11:20:29 CEST 2011


On Tue, May 10, 2011 at 10:13:18PM +0100, Mark Brown wrote:
> On Tue, Mar 22, 2011 at 10:37:03AM +0000, Dimitris Papastamos wrote:
> 
> > +	switch (codec->control_type) {
> > +	case SND_SOC_I2C:
> > +		ret = i2c_master_send(codec->control_data, data, len);
> > +		break;
> > +	case SND_SOC_SPI:
> > +		ret = do_spi_write(codec->control_data, data, len);
> > +		break;
> 
> Hrm, I now realise that this doesn't actually do what I'd expect - it
> only writes out the provided data, there's no inclusion of the register
> address.  I'd been mislead by the fact that it is taking in the register
> address as an argument and an overly hasty review it seems.  That's a
> little too raw for me, we should be prefixing the data with the base
> register address.

Yes, I was going to send a patch for this because I need it for syncing the
block based rbtree cache.


More information about the Alsa-devel mailing list