Re: [PATCH v3 4/6] ASoC: dt-bindings: wcd93xx: add bindings for audio mux controlling hp

[Srinivas Kandagatla]

On 24/03/2025 11:18, Krzysztof Kozlowski wrote:
> On 24/03/2025 12:06, srinivas.kandagatla@xxxxxxxxxx wrote:
> > From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
> >
> > On some platforms to minimise pop and click during switching between
> > CTIA and OMTP headset an additional HiFi mux is used. Most common
> > case is that this switch is switched on by default, but on some
> > platforms this needs a regulator enable.
> >
> > Move to using mux-controls so that both the gpio and regulators can be
> > driven correctly, rather than adding regulator handing in the codec.
> >
> > This patch adds required bindings to add such mux controls.
> >
> > Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
> > Tested-by: Christopher Obbard <christopher.obbard@xxxxxxxxxx>
>
> I claim you cannot test a binding in the way we understand Tested-by
> tags. Testing a binding is part of the build process and we do not have
> tested-by for builds...

I see your point, Normally if the tested by is given to series Its 
applied to all the patches in that series, I guess even b4 does that.

Will make a note of this next time and not add tested by to bindings.



Thanks,
Srini

> Anyway,
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> Best regards,
> Krzysztof