Skip to content

kmod_scripts: add support for snd_soc_cml_rt1011_rt5682#2198

Merged
jajanusz merged 1 commit intothesofproject:masterfrom
aiChaoSONG:master
Dec 13, 2019
Merged

kmod_scripts: add support for snd_soc_cml_rt1011_rt5682#2198
jajanusz merged 1 commit intothesofproject:masterfrom
aiChaoSONG:master

Conversation

@aiChaoSONG
Copy link
Copy Markdown
Collaborator

After Kernel PR #1565 being merged, correct driver is loaded on CML with rt1011 and rt5682, we got one more module need to be removed in module unload/reload test

this patch add snd_soc_cml_rt1011_rt5682 module removal support for CML platform with rt1011 and rt5682 codecs.

Copy link
Copy Markdown
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

almost good but minor reorder needed

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, this is in the wrong location. we group machine drivers together, then codec drivers. This should inserted line 46

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@plbossart Thank you, Pierre. Now it is at line 46.

Add snd_soc_cml_rt1011_rt5682 module removal support
for CML with rt1011 and rt5682 codecs.

Signed-off-by: Amery Song <chao.song@intel.com>
Copy link
Copy Markdown
Contributor

@xiulipan xiulipan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@jajanusz jajanusz merged commit e9bec4a into thesofproject:master Dec 13, 2019
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