ASoC: soc-card: remove card check
only snd_soc_card_set_bias_level() and snd_soc_card_set_bias_level_post()
are checking "card" in the function, like below
int snd_soc_card_set_bias_level(...)
{
...
=> if (card && card->xxx)
ret = card->xxxx(...);
...
}
But it should already have been an error if "card" was NULL.
remove "card" NULL check here. Otherwise it will get waring on
smatch without this patch. This is prepare for error check cleanup.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://patch.msgid.link/87msg2knsg.wl-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
committed by
Mark Brown
parent
309caeef43
commit
9c3d68ce5d
@@ -219,7 +219,7 @@ int snd_soc_card_set_bias_level(struct snd_soc_card *card,
|
||||
{
|
||||
int ret = 0;
|
||||
|
||||
if (card && card->set_bias_level)
|
||||
if (card->set_bias_level)
|
||||
ret = card->set_bias_level(card, dapm, level);
|
||||
|
||||
return soc_card_ret(card, ret);
|
||||
@@ -231,7 +231,7 @@ int snd_soc_card_set_bias_level_post(struct snd_soc_card *card,
|
||||
{
|
||||
int ret = 0;
|
||||
|
||||
if (card && card->set_bias_level_post)
|
||||
if (card->set_bias_level_post)
|
||||
ret = card->set_bias_level_post(card, dapm, level);
|
||||
|
||||
return soc_card_ret(card, ret);
|
||||
|
||||
Reference in New Issue
Block a user