summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>2024-09-03 18:47:55 +0300
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>2024-09-04 13:31:05 +0300
commitbda2f1c2d8c0fe93306043462cd6f679844e2171 (patch)
tree5f9c1d3246700f3e69677523be3e683e821a0c1f /drivers
parent52c62a3da80bcd17f535db00f5ea84ab16669438 (diff)
downloadlinux-bda2f1c2d8c0fe93306043462cd6f679844e2171.tar.xz
pinctrl: baytrail: Drop duplicate return statement
No need to repeat 'return ret;' inside and outside conditional. Just use one outside conditional for both cases. Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/pinctrl/intel/pinctrl-baytrail.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/pinctrl/intel/pinctrl-baytrail.c b/drivers/pinctrl/intel/pinctrl-baytrail.c
index d39e666a62df..4533c4d0a9e7 100644
--- a/drivers/pinctrl/intel/pinctrl-baytrail.c
+++ b/drivers/pinctrl/intel/pinctrl-baytrail.c
@@ -1542,10 +1542,8 @@ static int byt_gpio_probe(struct intel_pinctrl *vg)
}
ret = devm_gpiochip_add_data(vg->dev, gc, vg);
- if (ret) {
+ if (ret)
dev_err(vg->dev, "failed adding byt-gpio chip\n");
- return ret;
- }
return ret;
}