Re: [PATCH 1/3] ASoC: dt-bindings: wcd93xx: add bindings for audio switch controlling hp

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

 





On 20/03/2025 09:31, Krzysztof Kozlowski wrote:
On Wed, Mar 19, 2025 at 06:00:51PM +0000, Srinivas Kandagatla wrote:


On 19/03/2025 16:03, Mark Brown wrote:
On Wed, Mar 19, 2025 at 03:59:23PM +0000, Srinivas Kandagatla wrote:
On 19/03/2025 10:01, Dmitry Baryshkov wrote:

Is this regulator supplying the codec or some external component? In the
latter case it's likely that it should not be a part of WCD bindings.

This is regulator powering a mux that is driven by gpio which is part of
codec binding. So I would assume this will fall into the codec.

Where would we fit this if not part of codec?

Unless we mark this regulator as always on.

I would expect that the mux would appear in the DT and consume both the
GPIO and the regulator.
Yes, its doable, so we would endup with a mux driver consuming regulator and
gpio and move the gpio handling in codec to move to use mux control.

Let met try that and see how it looks like.

Looking at schematics this is clearly not a supply of a codec, but as
Dmitry said, separate switch. Actually two switches.

AFAIU, Adding single mux for both seems to be the best option.
Eventhough physically they are two muxes but they can not be driven independently, and logically/functionally they are doing only one thing of handing us/euro headset.


But if we represent these two muxes as two different nodes then the driver is messed up, as all the resources (gpios. pinctrls, regultors) will be shared and we can not control them independently.



thanks,
Srini

Best regards,
Krzysztof





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux