On Mon, Apr 08, 2024 at 10:38:02AM +0300, Dan Carpenter wrote: > ret = snd_soc_register_card(priv->card); > - if (!ret) > + if (ret) > return ret; > > return 0; Clearly a better fix here would just be to remove the conditional entirely.
Attachment:
signature.asc
Description: PGP signature
- Follow-Ups:
- Re: [PATCH] ASoC: soc-card: Fix a reversed if condition
- From: Dan Carpenter
- Re: [PATCH] ASoC: soc-card: Fix a reversed if condition
- References:
- [PATCH] ASoC: soc-card: Fix a reversed if condition
- From: Dan Carpenter
- [PATCH] ASoC: soc-card: Fix a reversed if condition
- Prev by Date: Re: [PATCH 1/4] regmap: Add regmap_read_bypassed()
- Next by Date: Re: [RESEND v3 2/2] ASoC: qcom: sc8280xp: Add support for QCM6490 and QCS6490
- Previous by thread: [PATCH] ASoC: soc-card: Fix a reversed if condition
- Next by thread: Re: [PATCH] ASoC: soc-card: Fix a reversed if condition
- Index(es):