Received-SPF: pass (google.com: domain of linux-kernel+bounces-71173-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Mon, 19 Feb 2024 12:06:57 +0100
Message-ID: <87o7ccogq6.wl-tiwai@suse.de>
From: Takashi Iwai <tiwai@suse.de>
To: Venkata Prasad Potturu <venkataprasad.potturu@amd.com>
Cc: Mark Brown <broonie@kernel.org>,
	stable@vger.kernel.org,
	Jaroslav Kysela <perex@perex.cz>,
	Takashi Iwai <tiwai@suse.com>,
	linux-sound@vger.kernel.org,
	alsa-devel@alsa-project.org,
	LKML <linux-kernel@vger.kernel.org>,
	"Mukunda, Vijendar" <Vijendar.Mukunda@amd.com>,
	"Dommati, Sunil-kumar" <Sunil-kumar.Dommati@amd.com>,
	"Hiregoudar, Basavaraj" <Basavaraj.Hiregoudar@amd.com>,
	Ethan Geller <ethang@valvesoftware.com>,
	Cristian Ciocaltea <cristian.ciocaltea@collabora.com>
Subject: Re: [REGRESSION] Acp5x probing regression introduced between kernel 6.7.2 -> 6.7.4
In-Reply-To: <3e364929-807e-477c-8530-f23db4ae7caf@amd.com>
References: <CAD_nV8BG0t7US=+C28kQOR==712MPfZ9m-fuKksgoZCgrEByCw@mail.gmail.com>
	<7a0cd63f-8a83-4dc5-8763-63dcdae8d68a@leemhuis.info>
	<878r3qxcyr.wl-tiwai@suse.de>
	<871q9hwz2w.wl-tiwai@suse.de>
	<ZdDXJyXiZ++KiNqW@finisterre.sirena.org.uk>
	<87msrzos6m.wl-tiwai@suse.de>
	<3e364929-807e-477c-8530-f23db4ae7caf@amd.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0
Precedence: bulk
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=US-ASCII

On Mon, 19 Feb 2024 11:50:46 +0100,
Venkata Prasad Potturu wrote:
> 
> 
> On 2/18/24 00:04, Takashi Iwai wrote:
> > On Sat, 17 Feb 2024 16:56:23 +0100,
> > Mark Brown wrote:
> >> On Mon, Feb 12, 2024 at 03:12:55PM +0100, Takashi Iwai wrote:
> >>> Takashi Iwai wrote:
> >>>> Interestingly, the system seems working with 6.8-rc3, so some piece
> >>>> might be missing.  Or simply reverting this patch should fix.
> >>> In the bugzilla entry, the reporter confirmed that the revert of the
> >>> commit 4b6986b170f2f2 fixed the problem.
> >> Any news on a patch for this?  Venkata?
> Hi Mark & Takashi,
> Sorry for the late reply.
> 
> 4b6986b170f2f2 ASoC: amd: Add new dmi entries for acp5x platform
> 
> There is no relation with the below error and this reverted patch, this patch is to
> define is to load Legacy driver for Jupiter Steam deck variant.
> 
> We will check on this.
> 
> Error logs.
> [    8.755614] cs35l41 spi-VLV1776:00: supply VA not found, using dummy regulator
> [    8.760506] cs35l41 spi-VLV1776:00: supply VP not found, using dummy regulator
> [    8.777148] cs35l41 spi-VLV1776:00: Cirrus Logic CS35L41 (35a40), Revision: B2
> [    8.777471] cs35l41 spi-VLV1776:01: supply VA not found, using dummy regulator
> [    8.777532] cs35l41 spi-VLV1776:01: supply VP not found, using dummy regulator
> [    8.777709] cs35l41 spi-VLV1776:01: Reset line busy, assuming shared reset
> [    8.788465] cs35l41 spi-VLV1776:01: Cirrus Logic CS35L41 (35a40), Revision: B2
> [    8.877280] snd_hda_intel 0000:04:00.1: enabling device (0000 -> 0002)
> [    8.877595] snd_hda_intel 0000:04:00.1: Handle vga_switcheroo audio client
> [    8.889913] snd_acp_pci 0000:04:00.5: enabling device (0000 -> 0002)
> [    8.890063] snd_acp_pci 0000:04:00.5: Unsupported device revision:0x50
> [    8.890129] snd_acp_pci: probe of 0000:04:00.5 failed with error -22

My understanding is that the fact that snd_acp_pci matches caused the
problem (without backports of other relevant changes).
In the working case, it's probed with snd_pci_acp5x driver.


thanks,

Takashi