Re: [PATCH 1/2] ASoC: add generic simple-amplifier support

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

 



On Mon, Jun 25, 2018 at 8:39 PM, Mark Brown <broonie@xxxxxxxxxx> wrote:
> On Mon, Jun 25, 2018 at 08:28:32PM +0800, Chen-Yu Tsai wrote:
>
>> I think slightly different semantics apply for "shutdown" vs "mute"
>> pins / controls.
>> "shutdown" implies a low power, if not powered off altogether, mode which might
>> require a slight delay to charge up when the amp is enabled. "mute" is
>> just that,
>> disconnecting the output, which would be useful in masking pops or other noises,
>> but does nothing to help power consumption.
>
>> Not sure if the subtle differences can be handled in DAPM, but it would be nice
>> to have. I've seen devices that have both control pins.
>
> It's certainly potentially useful to have pins for both and treat them
> differently - ideally something that's a mute would get autodisable type
> treatment so we mask out any pops in the input path but also make it
> available to userspace if it needs it.  That's something that could be
> done when the need arises though, until something that actually has both
> controls appears handling a mute pin like a power control is probably
> going to result in the most reliable integration.

Agreed. I'll dig around to see which one of my devices has this.

> Having the binding specify both differently would be good for future
> proofing.

Ack.

Also, would it be possible to specify the name or prefix of the widgets
from the device tree, such that it would be possible to have multiple
instances? I'm thinking one amp for speakers, and one for headphones,
each individually controlled.

ChenYu
_______________________________________________
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