All of lore.kernel.org
 help / color / mirror / Atom feed
From: anish kumar <yesanishhere@gmail.com>
To: Mark Brown <broonie@kernel.org>
Cc: Linux-ALSA <alsa-devel@alsa-project.org>,
	Liam Girdwood <lgirdwood@gmail.com>
Subject: Re: [PATCH] ASoC: Add max98371 codec driver
Date: Tue, 26 Apr 2016 11:16:38 -0700	[thread overview]
Message-ID: <CABCoZhCv0u72vDHkEbq--SuHcZGvQYggH9EbSSYLC3ByFdt-cA@mail.gmail.com> (raw)
In-Reply-To: <20160422153124.GH3217@sirena.org.uk>

On Fri, Apr 22, 2016 at 8:31 AM, Mark Brown <broonie@kernel.org> wrote:
> On Wed, Apr 20, 2016 at 11:14:40AM -0700, anish kumar wrote:
>> On Tue, Apr 19, 2016 at 2:46 AM, Mark Brown <broonie@kernel.org> wrote:
>
>> > Hrm, that looks like the driver may be able to set it automatically
>> > using the regulator API and just have a switch for the guaranteed.
>
>> Should i pass it via device tree then? I didn't get your statement about
>
> I'd expect to look up the value by looking at the supply voltage if we
> were going to go that way.

Discussion of this with hardware team revealed that dynamic headroom
tracking is the feature which will be controlled with this gain.

I am also adding two new controls for this: speaker gain min and DHT
rotation point.

I will be adding all of this as TLV as this should be configurable.
>
>> "switch".
>
> A switch is an on/off control.

  reply	other threads:[~2016-04-26 18:16 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-13 20:20 [PATCH] ASoC: Add max98371 codec driver anish kumar
2016-04-18 16:47 ` Mark Brown
2016-04-18 20:20   ` anish kumar
2016-04-19  9:46     ` Mark Brown
2016-04-20 18:14       ` anish kumar
2016-04-22 15:31         ` Mark Brown
2016-04-26 18:16           ` anish kumar [this message]
2016-04-27 16:42             ` Mark Brown
  -- strict thread matches above, loose matches on Subject: below --
2016-04-27 22:39 anish kumar
2016-03-22  6:53 anish kumar
2016-03-28 19:13 ` Mark Brown
2016-04-07 19:34   ` anish kumar
2016-04-08  3:09 ` Lars-Peter Clausen
2016-04-08 20:04   ` anish kumar
2016-04-10 18:04   ` Mark Brown
2016-03-21  1:59 anish kumar
2016-03-21 14:56 ` Mark Brown
2016-03-21 15:29   ` Micka
2016-03-21 15:56     ` Mark Brown
2016-03-21 16:41       ` Micka
2016-03-21 17:05         ` Mark Brown
2016-03-23 12:45           ` Micka
2016-03-23 16:23             ` Amish Kumar
2016-03-24 10:49   ` Micka
2016-03-24 11:05     ` Mark Brown
2016-03-24 11:19       ` Micka
2016-03-24 11:21         ` Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CABCoZhCv0u72vDHkEbq--SuHcZGvQYggH9EbSSYLC3ByFdt-cA@mail.gmail.com \
    --to=yesanishhere@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=lgirdwood@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.