[alsa-devel] [PATCH] Add support for tlv320aic3007 to tlv320aic3x driver

Randolph Chung rchung42 at gmail.com
Thu Aug 19 12:00:29 CEST 2010


> Rather than passing something in the platform data you should use the
> I2C device ID table for the device and have the board register the
> device as a tlv320aic3007.

Yes, that will work too - if that's the preference I will rework the
patch that way.

> I would review the patch in more detail but you attached it as a base64
> encoded attachment which means my MUA hasn't quoted it for me and won't
> even display it unless I explicitly open it...

My apologies - since you reported whitespace damage in my previous
post I included the patch as an attachment and apparently my mail
client decided to base64 encode it. I will post followups from another
MUA.

> The main thing I noticed
> was that you're using an enum instead of a TLV control for the amp
> volume.

The amp volume (gain) only has 4 steps (0, 6dB, 12dB, 18dB). I can
rework as a TLV if that's the preference. TI has a (non-published?)
driver for this codec that used an enum so this part was copied over.

> As I reminded you last time you always need to CC maintainers on
> patches.  This is very helpful for ensuring that we actually see things
> on busy lists and is helpful for keeping track of what needs reviewing.

You copied my last patch to lrg (and he acked the patch), so I copied
my patch to lrg. Previous patches to this driver were also signed off
by him. Should I copy to somebody else?

randolph


More information about the Alsa-devel mailing list