Re: [PATCH] ASoC: dapm: Add speaker driver widget.

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

 



On 6 Dec 2010, at 23:30, Liam Girdwood wrote:
> On Mon, 2010-12-06 at 23:09 +0000, Mark Brown wrote:
>> 

>> Sure, but it's fulfiling the same role in the system - it's just that
>> these days a lot more CODECs are pulling speaker drivers directly into
>> the CODEC die.  Mostly these have worked well handled as PGAs so it's
>> not been an issue.

> In this case as we need to enable the PGA before the driver and disable
> the driver before the PGA for pop reduction. Hence the current ordering
> needs an addition/refactoring to deal with the newer generation of
> CODECs here.

Well, what I'm saying is that...

>> I'd certainly expect to see it handled the same way from a DAPM
>> sequencing point of view as it's fulfilling the same role in the system
>> (so in the same slot rather than separately as the patch was doing).  Do
>> we just need to refactor the existing external widgets to be able to
>> exist in either register or GPIO based versions?

...we don't need to change the ordering at all, we've already got a
sequence point for this sort of widget we can use. We should just make
the existing speaker and headphone widgets be in terms of one, I think.

The only reason it's not come up before is that everything so far has been
able to power up cleanly without splitting from the PGAs.
_______________________________________________
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