Skip to content

ASoC: Intel: fix YogaBook audio on Linux 7#1

Open
lawrencefrias wants to merge 1 commit into
jekhor:masterfrom
lawrencefrias:pr-linux7-audio
Open

ASoC: Intel: fix YogaBook audio on Linux 7#1
lawrencefrias wants to merge 1 commit into
jekhor:masterfrom
lawrencefrias:pr-linux7-audio

Conversation

@lawrencefrias

Copy link
Copy Markdown

Forward-port YogaBook CHT/RT5677 audio machine driver fixes to Linux 7.

Fixes NULL card access in machine-level DAPM event handlers by replacing
snd_soc_dapm_to_component(w->dapm)->card with snd_soc_dapm_to_card(w->dapm).

Also ensures missing speaker/headphone GPIO lookups do not break audio
initialization.

Tested on Lenovo YogaBook YB1-X91F with Linux 7.0.10:

  • internal cht-yogabook card registers
  • RT5677 codec initializes correctly
  • audio playback works
  • no RIP/#PF crashes during playback

Signed-off-by: Lawrence Frias <lawrence.frias@gmail.com>
@DerKnerd

DerKnerd commented Jun 5, 2026

Copy link
Copy Markdown

Isn't the Kernel still on 6.17?

@jekhor

jekhor commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Actually I prepared a patch series for mainline and it passed first review stage but i still have not submitted second version after review. I found few issues which need to be fixed, for example, sound stops working after module unloading and loading.

First version is here: https://lore.kernel.org/linux-sound/20260217231324.1319392-1-jekhor@gmail.com/#r

@lawrencefrias

lawrencefrias commented Jun 5, 2026

Copy link
Copy Markdown
Author

I created a custom kernel for Linux 7. I'm uploading a new release on my forked repo. https://github.com/lawrencefrias/yogabook-linux-kernel/releases/tag/v7.0.10-yogabook7-audio

@DerKnerd

DerKnerd commented Jun 6, 2026

Copy link
Copy Markdown

@lawrencefrias Does your repo contain the sources for the 7 kernel? Cause it is only one commit ahead of the current master here.

@lawrencefrias

Copy link
Copy Markdown
Author

This PR contains only the fix on top of the current yogabook-linux tree.

My full Linux 7 work is based on the upstream linux-stable tree with additional YogaBook patches applied locally.

I will provide the full branch / source tree if needed. I didn’t include it in the PR to keep the changes minimal and focused on the audio fix.

@DerKnerd

DerKnerd commented Jun 6, 2026

Copy link
Copy Markdown

I think it would be cool if you could create a proper PR with the kernel version 7. I think you also violated the GPL with providing binaries without sources.

@lawrencefrias

lawrencefrias commented Jun 6, 2026

Copy link
Copy Markdown
Author

I created a separate branch with the Linux 7 YogaBook port work based directly on this repository (without including the full linux-stable history):

lawrencefrias:linux-7-yogabook-port

The PR itself only contains the minimal audio fix to keep it reviewable.

It looks like most of the changes I had locally are already aligned with your current tree, so there are no large diffs on top at the moment. I'll work on the full pr next.

@lawrencefrias

Copy link
Copy Markdown
Author

Ok @DerKnerd I think I did everything correctly. Please view my other pr.

@DerKnerd

DerKnerd commented Jun 6, 2026

Copy link
Copy Markdown

@lawrencefrias Great, I am not knowledgable in Linux kernel dev, so @jekhor would need to take a look into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants