diff options
| author | Miquel Raynal <miquel.raynal@bootlin.com> | 2021-10-15 11:14:30 +0300 |
|---|---|---|
| committer | Lee Jones <lee.jones@linaro.org> | 2021-10-20 19:17:31 +0300 |
| commit | 287ee127bf0bc6ede07426a57de1475d0a60b503 (patch) | |
| tree | 8b52db431d80254d8eb5095035bef7f249eb70af /drivers/mfd | |
| parent | 243e3cb9c093cab2e999e738304f1a23770b3717 (diff) | |
| download | linux-287ee127bf0bc6ede07426a57de1475d0a60b503.tar.xz | |
mfd: ti_am335x_tscadc: Get rid of useless gotos
Goto's jumping to a return statement are not really useful, drop them.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Link: https://lore.kernel.org/r/20211015081506.933180-13-miquel.raynal@bootlin.com
Diffstat (limited to 'drivers/mfd')
| -rw-r--r-- | drivers/mfd/ti_am335x_tscadc.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/mfd/ti_am335x_tscadc.c b/drivers/mfd/ti_am335x_tscadc.c index 00428d12e6b8..3dfa9af7e6ab 100644 --- a/drivers/mfd/ti_am335x_tscadc.c +++ b/drivers/mfd/ti_am335x_tscadc.c @@ -174,7 +174,7 @@ static int ti_tscadc_probe(struct platform_device *pdev) err = platform_get_irq(pdev, 0); if (err < 0) - goto ret; + return err; else tscadc->irq = err; @@ -189,8 +189,7 @@ static int ti_tscadc_probe(struct platform_device *pdev) &tscadc_regmap_config); if (IS_ERR(tscadc->regmap)) { dev_err(&pdev->dev, "regmap init failed\n"); - err = PTR_ERR(tscadc->regmap); - goto ret; + return PTR_ERR(tscadc->regmap); } spin_lock_init(&tscadc->reg_lock); @@ -274,7 +273,7 @@ static int ti_tscadc_probe(struct platform_device *pdev) err_disable_clk: pm_runtime_put_sync(&pdev->dev); pm_runtime_disable(&pdev->dev); -ret: + return err; } |