Skip to content

[v5.0 release] ALSA: hda: fix headphone detection failed - #1060

Closed
RanderWang wants to merge 1 commit into
thesofproject:release/sof-v5.0from
RanderWang:latest-jack-fix
Closed

RanderWang wants to merge 1 commit into
thesofproject:release/sof-v5.0from
RanderWang:latest-jack-fix

Conversation

@RanderWang

@RanderWang RanderWang commented Jun 26, 2019 •

Copy link
Copy Markdown

In ASoC, cmd io is diabled when there is no user on
the bus and is enabled when a user is active. But unsolicated
event is diabled in snd_hdac_bus_stop_cmd_io and not
enabled in snd_hdac_bus_init_cmd_io. This makes headphone
detection failed.

This patch Enable unsolicated event when cmd io is enabled.

Tested on whiskylake with legacy hda driver and SOF driver

It fixes #1055 #1054

Signed-off-by: Rander Wang rander.wang@linux.intel.com

@RanderWang
RanderWang requested a review from bardliao June 26, 2019 02:46
In ASoC, cmd io is diabled when there is no user on
the bus and is enabled when a user is active. But unsolicated
event is diabled in snd_hdac_bus_stop_cmd_io and not
enabled in snd_hdac_bus_init_cmd_io. This makes headphone
detection failed.

This patch Enable unsolicated event when cmd io is enabled.

Tested on whiskylake with legacy hda driver and SOF driver

Signed-off-by: Rander Wang <rander.wang@linux.intel.com>
@RanderWang RanderWang changed the title ALSA: hda: fix headphone detection failed [v5.0 release] ALSA: hda: fix headphone detection failed Jun 26, 2019
@RanderWang

Copy link
Copy Markdown
Author

This patch is requested by canonical. And I will send it to ALSA mail list later.

@wenqingfu
wenqingfu requested review from kv2019i, lyakh and ranj063 June 27, 2019 02:17
@wenqingfu

Copy link
Copy Markdown

is the patch legitimate on sof-dev? If so, please remove the [] prefix. + more reviewers

@RanderWang

Copy link
Copy Markdown
Author

is the patch legitimate on sof-dev? If so, please remove the [] prefix. + more reviewers

Not now, it is only for v5.0 release. I sent this patch to ALSA mail list and Takashi approved. I will create a PR for sof-dev after it is merged.

@bardliao

Copy link
Copy Markdown
Collaborator

Not now, it is only for v5.0 release. I sent this patch to ALSA mail list and Takashi approved. I will create a PR for sof-dev after it is merged.

Blindly merged to release/sof-v5.0

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