[alsa-devel] ASoC, TWL4030 Register defaults

[naveen krishna ch]

On Thu, Nov 20, 2008 at 4:53 PM, Mark Brown <broonie at sirena.org.uk> wrote:

> On Thu, Nov 20, 2008 at 04:49:33PM +0530, naveen krishna ch wrote:
>
> > [PATCH] Register default values for gain, path selection for various
> input
> > and output devices
> > From:
> > chnaveenkrishna <chnaveen at chnaveen.mistral.in>
> >   To:
> > Date:
> > Today 04:49:50 pm
>
> Something (your MUA)?  Is mangling this really badly...

Sorry i will try avoiding this MUA issue, I was using the gmail till now.

>
>
> As I said in reply to your first mail I'd really like to see some sort
> of explanation as to why you are making these changes - are the defaults
> in the existing driver wrong?  There seem to be an awful lot of changes
> here and there's no discussion in the patch as to what they mean.

The existing driver does the default settings for only one input device
(Line in) and one output device (Headset out).
The CODEC supports 5 output devices and 4 input devices.

The default values given by the drivers author are according to his custom
requirements, expanding them to the CODEC's ability these values can be
modified.

What all this patch is doing is setting the default playback and capture
volumes of various input and output devices to reasonable values.

-       0x24, /* REG_ANAMICL            (0x5)   */
-       0x04, /* REG_ANAMICR            (0x6)   */
+       0xb0, /* REG_ANAMICL            (0x5)   */
+       0x10, /* REG_ANAMICR            (0x6)   */
Author is setting default capture device as Line In.  It was modified to
none.

-       0x4b, /* REG_ARXL2_APGA_CTL     (0x1B)  */
-       0x4b, /* REG_ARXR2_APGA_CTL     (0x1C)  */
+       0x2b, /* REG_ARXL2_APGA_CTL     (0x1B)  */
+       0x2b, /* REG_ARXR2_APGA_CTL     (0x1C)  */
The gain controls register settings now modified for minimal play back
volume.

-       0x00, /* REG_EAR_CTL            (0x21)  */
-       0x24, /* REG_HS_SEL             (0x22)  */
-       0x0a, /* REG_HS_GAIN_SET        (0x23)  */
+       0x20, /* REG_EAR_CTL            (0x21)  */
+       0x00, /* REG_HS_SEL             (0x22)  */
+       0x05, /* REG_HS_GAIN_SET        (0x23)  */

He selected the Headset as default output device it was modified to none
with some gain settings.

-       0x00, /* REG_PREDL_CTL          (0x25)  */
-       0x00, /* REG_PREDR_CTL          (0x26)  */
-       0x00, /* REG_PRECKL_CTL         (0x27)  */
-       0x00, /* REG_PRECKR_CTL         (0x28)  */

+       0x20, /* REG_PREDL_CTL          (0x25)  */
+       0x20, /* REG_PREDR_CTL          (0x26)  */
+       0x20, /* REG_PRECKL_CTL         (0x27)  */
+       0x20, /* REG_PRECKR_CTL         (0x28)  */
GVain settings for 3 outputting devices

-       0x00, /* REG_HFL_CTL            (0x29)  */
-       0x00, /* REG_HFR_CTL            (0x2A)  */
+       0x1f, /* REG_HFL_CTL            (0x29)  */
+       0x1f, /* REG_HFR_CTL            (0x2A)  */
gain settings and path selection for handsfree output

-       0x00, /* REG_ANAMIC_GAIN        (0x48)  */
+       0x24, /* REG_ANAMIC_GAIN        (0x48)  */
Main and Sub Mic gain settings.


My upcoming patches will make use of this default settings.


-- 
Thanks,

(: Naveen Krishna Ch :)