[alsa-devel] issue with jack detect using rt5645

[Curtis Malainey]

On Thu, Nov 7, 2019 at 3:28 PM Pierre-Louis Bossart
<pierre-louis.bossart at linux.intel.com> wrote:
>
>
>
> On 11/7/19 4:31 PM, Ross Zwisler wrote:
> > We are trying to get a piece of Google hardware which uses the rt5645
> > driver working correctly with newer kernels (currently one based on
> > the latest v4.14 stable), and noticed that the following commit:
> >
> > commit 6b5da66322c5 ("ASoC: rt5645: read jd1_1 status for jd detection")
> >
> > broke jack support for our device.  Basically with that patch we never
> > correctly see a jack insertion event, and audio is never routed to the
> > headphones.
> >
> > With just that one patch reverted, things work correctly on our
> > system.  The revert applies cleanly to linus/master.
> >
> > Here is the series where that patch was originally upstreamed:
> >
> > https://www.spinics.net/lists/alsa-devel/msg64331.html
> >
> > If we reverted that patch upstream, would it break other devices (the
> > "GDP Win" device?) that use the rt5645 driver?  If so, how would you
> > recommend proceeding?
> >
> > Does anyone have one of the "GDP Win" devices so we can test that and
> > our device and make sure whatever solution we end up coming up with
> > works for everyone?
>
> there are all kinds of DMI quirks in sound/soc/codecs/rt5645.c, 4
> directly applicable to Chromebooks ('buddy', 'strago', 'Google Chrome',
> 'Google Setzer')
>
> Can you check if your device is listed in those quirks? it could be that
> either the settings are wrong or that we never supported it.

We have been playing with the DMI values with little luck. (in this
case it's Buddy.) When we flip the polarity (level_triggered_irq
field) we only receive disconnect events.