Re: [PATCH] ASoC: Add max98371 codec driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mark,

Could you tell me how do you control this device/codec with Linux ? What is
the command ?

Thank you !

Le lun. 21 mars 2016 à 16:03, Mark Brown <broonie@xxxxxxxxxx> a écrit :

> On Sun, Mar 20, 2016 at 06:59:02PM -0700, anish kumar wrote:
>
> This looks mostly good, a few small things:
>
> > +static const char *const voltage_level_text[] = {
> > +     "9.5dB", "10.5dB", "11.5dB", "11.5dB", "12.5dB",
> > +     "13.5dB", "14.5dB", "15.5dB", "16.5dB", "17.5dB",
> > +     "18.5dB", "19.5dB", "20.5dB"
> > +};
>
> Volume controls should be done with TLV data not enums.
>
> > +static const struct snd_kcontrol_new max98371_snd_controls[] = {
> > +     SOC_SINGLE_TLV("Speaker Gain", MAX98371_GAIN,
> > +                     MAX98371_GAIN_SHIFT, (1<<MAX98371_GAIN_WIDTH)-1, 0,
> > +                     max98371_gain_tlv),
>
> Volume controls should end with Volume.
>
> > +static int max98371_probe(struct snd_soc_codec *codec)
> > +{
> > +     struct max98371_priv *max98371 = snd_soc_codec_get_drvdata(codec);
> > +
> > +     max98371->codec = codec;
> > +     return 0;
> > +}
>
> Do we ever actually use this?
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux