Re: [REGRESSION] Acp5x probing regression introduced between kernel 6.7.2 -> 6.7.4

[Takashi Iwai]

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