linux-kernel - Re: [PATCH 5.17 162/225] ASoC: Intel: sof_es8336: Add a quirk for Huawei Matebook D15

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Sun, 8 May 2022 18:12:32 +0200
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Justin Forbes <jforbes@...oraproject.org>
Cc:     linux-kernel@...r.kernel.org, stable@...r.kernel.org,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>,
        Mark Brown <broonie@...nel.org>,
        Sasha Levin <sashal@...nel.org>
Subject: Re: [PATCH 5.17 162/225] ASoC: Intel: sof_es8336: Add a quirk for
 Huawei Matebook D15

On Wed, May 04, 2022 at 03:41:24PM -0500, Justin Forbes wrote:
> On Wed, May 04, 2022 at 06:46:40PM +0200, Greg Kroah-Hartman wrote:
> > From: Mauro Carvalho Chehab <mchehab@...nel.org>
> > 
> > [ Upstream commit c7cb4717f641db68e8117635bfcf62a9c27dc8d3 ]
> > 
> > Based on experimental tests, Huawei Matebook D15 actually uses
> > both gpio0 and gpio1: the first one controls the speaker, while
> > the other one controls the headphone.
> > 
> > Also, the headset is mapped as MIC1, instead of MIC2.
> > 
> > So, add a quirk for it.
> > 
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@...nel.org>
> > Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>
> > Link: https://lore.kernel.org/r/d678aef9fc9a07aced611aa7cb8c9b800c649e5a.1649357263.git.mchehab@kernel.org
> > Signed-off-by: Mark Brown <broonie@...nel.org>
> > Signed-off-by: Sasha Levin <sashal@...nel.org>
> > ---
> 
> This patch is missing some dependencies and fails to build:
> 
> sound/soc/intel/boards/sof_es8336.c:261:41: error: 'SOF_ES8336_HEADPHONE_GPIO' undeclared here (not in a function)
>   261 |                 .driver_data = (void *)(SOF_ES8336_HEADPHONE_GPIO |
> 
> SOF_ES8336_HEADPHONE_GPIO was defined in upstream commit:
> 6e1ff1459e008 ASoC: Intel: sof_es8336: support a separate gpio to control headphone
> which appeared with 5.18-rc4
> 
> sound/soc/intel/boards/sof_es8336.c:262:41: error: 'SOC_ES8336_HEADSET_MIC1' undeclared here (not in a function)
>   262 |                                         SOC_ES8336_HEADSET_MIC1)
> 
> SOC_ES8336_HEADSET_MIC1 was defined in upstream commit: 
> 7c7bb2a059b22 ASoC: Intel: sof_es8336: add a quirk for headset at mic1 port
> which also appeared with 5.18-rc4
> 
> We either need to bring in these 2 commits or drop this one from the
> stable queue.

I've dropped this one now, thanks.

greg k-h