[PATCH 1/4] ASoC: Add TI tlv320aic32x4 codec support.

javier Martin javier.martin at vista-silicon.com
Tue Mar 1 02:59:23 EST 2011


Hi Mark,
thank you for your review. I will try to solve every issue you pointed out.

Regarding non-default configurations:

>> +static const struct aic32x4_configs aic32x4_reg_init[] = {
>> +     {AIC32X4_PWRCFG, AIC32X4_AVDDWEAKDISABLE},
>
> What are these "aic32x4_configs" all about?  They look like you're
> setting non-default configurations...

This tlv320aic32x4 codec is kind of a huge monster; it has more than
200 registers which provide more flexibility than it is needed.
For example, BCLK and MCLK are forced here to derive from internal PLL
which is needed for frequency calculation table to be valid. Bypassing
this PLL doesn't provide new or better functionality.
The same happens regarding DAC internal connections. I don't think
allowing the user to swap left and right channels makes sense here.

-- 
Javier Martin
Vista Silicon S.L.
CDTUC - FASE C - Oficina S-345
Avda de los Castros s/n
39005- Santander. Cantabria. Spain
+34 942 25 32 60
www.vista-silicon.com



More information about the linux-arm-kernel mailing list