mirror of
https://github.com/torvalds/linux.git
synced 2024-11-13 15:41:39 +00:00
ASoC: Intel: bdw_rt286: Improve codec_init() quality
Drop redundant 'ret' assignemnt, stop ignoring set_jack() return value and reword local 'component' variable to 'codec' to improve readability. Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com> Reviewed-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> Tested-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Link: https://lore.kernel.org/r/20220620101402.2684366-16-cezary.rojewski@intel.com Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
423cc2d0e8
commit
8fe4709962
@ -61,8 +61,8 @@ static const struct snd_soc_dapm_route card_routes[] = {
|
||||
|
||||
static int codec_link_init(struct snd_soc_pcm_runtime *rtd)
|
||||
{
|
||||
struct snd_soc_component *component = asoc_rtd_to_codec(rtd, 0)->component;
|
||||
int ret = 0;
|
||||
struct snd_soc_component *codec = asoc_rtd_to_codec(rtd, 0)->component;
|
||||
int ret;
|
||||
|
||||
ret = snd_soc_card_jack_new_pins(rtd->card, "Headset", SND_JACK_HEADSET | SND_JACK_BTN_0,
|
||||
&card_headset, card_headset_pins,
|
||||
@ -70,8 +70,7 @@ static int codec_link_init(struct snd_soc_pcm_runtime *rtd)
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
snd_soc_component_set_jack(component, &card_headset, NULL);
|
||||
return 0;
|
||||
return snd_soc_component_set_jack(codec, &card_headset, NULL);
|
||||
}
|
||||
|
||||
static int codec_link_hw_params_fixup(struct snd_soc_pcm_runtime *rtd,
|
||||
|
Loading…
Reference in New Issue
Block a user