Skip to content

ASoC: rt711: init jack after rt711 is attached. - #1254

Closed
bardliao wants to merge 6 commits into
thesofproject:integration/realtek-sdw-codecsfrom
bardliao:rt711-init-jack
Closed

bardliao wants to merge 6 commits into
thesofproject:integration/realtek-sdw-codecsfrom
bardliao:rt711-init-jack

Conversation

@bardliao

Copy link
Copy Markdown
Collaborator

Somehow rt711_set_jack_detect() will not be called again even we
return -EAGAIN.
It is better to do jack init in rt711_io_init() if
rt711_set_jack_detect() is called before rt711 is attached.

Signed-off-by: Bard Liao yung-chuan.liao@linux.intel.com

shumingfan and others added 6 commits September 20, 2019 13:29
Initial support for ALC/RT700 in SoundWire mode

This codec is present on Intel CNL/CML and ICL RVP boards and the code
was tested by both Realtek and Intel.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Shuming Fan <shumingf@realtek.com>
The driver for this amplifier was tested by Realtek and Intel using
the Realtek 3-in-1 add-on boards connected to CometLake and IceLake
platforms

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Shuming Fan <shumingf@realtek.com>
The driver for this codec was tested by Realtek and Intel using the
Realtek 3-in-1 add-on board connected to CometLake and IceLake
platforms.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Shuming Fan <shumingf@realtek.com>
The driver for this capture device was tested by Realtek and Intel
using the Realtek 3-in-1 add-on boards connected to CometLake and
IceLake platforms

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Jack Yu <jack.yu@realtek.com>
Don't push upstream

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Somehow rt711_set_jack_detect() will not be called again even we
return -EAGAIN.
It is better to do jack init in rt711_io_init() if
rt711_set_jack_detect() is called before rt711 is attached.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
@bardliao

Copy link
Copy Markdown
Collaborator Author

@shumingfan Could you help to review?

Comment thread sound/soc/codecs/rt711.c
/* power off */
if (dapm->bias_level <= SND_SOC_BIAS_STANDBY)
regmap_write(rt711->regmap,
RT711_SET_AUDIO_POWER_STATE, AC_PWRST_D3);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@bardliao Could you move the power on/off to rt711_jack_init function?
It has a chance to be power off state when running rt711_jack_init function.

@plbossart
plbossart force-pushed the integration/realtek-sdw-codecs branch 2 times, most recently from 715f052 to c0b65b4 Compare September 26, 2019 16:09
@shumingfan

Copy link
Copy Markdown

@bardliao I submitted a new PR for this issue. Please check PR#1260.

@plbossart plbossart closed this Sep 27, 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.

3 participants