[PATCH 11/11] ASoC: codecs: rt298: Fix jack detection
Mark Brown
broonie at kernel.org
Fri Jun 10 14:25:20 CEST 2022
On Fri, Jun 10, 2022 at 11:46:19AM +0200, Amadeusz Sławiński wrote:
> On 6/9/2022 4:55 PM, Mark Brown wrote:
> > It looks rt298_jack_detect() already forces the pins on? It's not clear
> > to me what the relationship between this code and the existing code is.
> This aligns the code to be similar to other two rt2xx drivers and fixes a
> problem on our side.
> Original code doesn't reach rt298_jack_detect() when jack == NULL, so it
> never disables those pins in this case.
> I could probably fix this by moving rt298_jack_detect() call, but as drivers
> for rt2xx codecs are quite similar to each other I opted to fix the issue by
> minimizing the differences between them.
It would be good to at least clarify what's going on in the changelog -
it's not really clear what exactly the problem is or how this fixes it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20220610/e0cccd2d/attachment-0001.sig>
More information about the Alsa-devel
mailing list