Skip to content

Lenovo 83hm - #5248

Merged
bardliao merged 2 commits into
thesofproject:topic/sof-devfrom
CirrusLogic:lenovo-83hm
Nov 21, 2024
Merged

bardliao merged 2 commits into
thesofproject:topic/sof-devfrom
CirrusLogic:lenovo-83hm

Conversation

@simontrimmer

Copy link
Copy Markdown
Collaborator

No description provided.

@simontrimmer

Copy link
Copy Markdown
Collaborator Author

@bardliao Here we go - hopefully this is the final piece of the puzzle (in combination with the topology change we have been smithing in thesofproject/sof#9666 )

@lgirdwood

Copy link
Copy Markdown
Member

@bardliao Here we go - hopefully this is the final piece of the puzzle (in combination with the topology change we have been smithing in thesofproject/sof#9666 )

Nitpick: the commit messages could have a bit more context. Might need this for upstream to ALSA.

@bardliao

Copy link
Copy Markdown
Collaborator

The second commit is easy to understand. However, we still need a commit message. Maybe just "Add debug message when SOC_SDW_CODEC_MIC is enabled"

In addition to changing the DMI match to examine the product name rather
than the SKU, this adds the quirk to inform the machine driver to not
bind in the cs42l43 microphone DAI link.

Signed-off-by: Simon Trimmer <simont@opensource.cirrus.com>
… quirk

Add debug message when SOC_SDW_CODEC_MIC is enabled (which informs the
machine driver to not bind in the cs42l43 microphone DAI link).

Signed-off-by: Simon Trimmer <simont@opensource.cirrus.com>
@simontrimmer

Copy link
Copy Markdown
Collaborator Author

Good thinking - I've rebased and updated the latter one to have a description, as it would be easy to miss that it turns something off

@bardliao
bardliao merged commit 5f650cf into thesofproject:topic/sof-dev Nov 21, 2024
@simontrimmer
simontrimmer deleted the lenovo-83hm branch November 21, 2024 15:30
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.

4 participants