diff mbox series

[v2] ASoC: cs42l56: Fix the error handling of cs42l56_i2c_probe()

Message ID 20220511015514.1777923-1-zheyuma97@gmail.com (mailing list archive)
State Accepted
Commit c1ce4ba5021a9730c00bab6f8122702deb69d37e
Headers show
Series [v2] ASoC: cs42l56: Fix the error handling of cs42l56_i2c_probe() | expand

Commit Message

Zheyu Ma May 11, 2022, 1:55 a.m. UTC 
The driver should goto label 'err_enable' when failing at regmap_read().

Signed-off-by: Zheyu Ma <zheyuma97@gmail.com>
---
Changes in v2:
    - Fix the typo in the subject
---
 sound/soc/codecs/cs42l56.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Charles Keepax May 11, 2022, 8:19 a.m. UTC | #1 
On Wed, May 11, 2022 at 09:55:14AM +0800, Zheyu Ma wrote:
> The driver should goto label 'err_enable' when failing at regmap_read().
> 
> Signed-off-by: Zheyu Ma <zheyuma97@gmail.com>
> ---
> Changes in v2:
>     - Fix the typo in the subject
> ---

Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>

Thanks,
Charles
diff mbox series

Patch

diff --git a/sound/soc/codecs/cs42l56.c b/sound/soc/codecs/cs42l56.c
index 2c4e09b43199..dc23007336c5 100644
--- a/sound/soc/codecs/cs42l56.c
+++ b/sound/soc/codecs/cs42l56.c
@@ -1245,7 +1245,7 @@  static int cs42l56_i2c_probe(struct i2c_client *i2c_client)
 	ret = regmap_read(cs42l56->regmap, CS42L56_CHIP_ID_1, &reg);
 	if (ret) {
 		dev_err(&i2c_client->dev, "Failed to read chip ID: %d\n", ret);
-		return ret;
+		goto err_enable;
 	}
 
 	devid = reg & CS42L56_CHIP_ID_MASK;