| Message ID | 20231003120648.2838047-1-andriy.shevchenko@linux.intel.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v1,1/1] pinctrl: core: Remove unneeded {} around one line conditional body | expand |
On Tue, Oct 03, 2023 at 03:06:48PM +0300, Andy Shevchenko wrote: > The one line conditional body doesn't require {} surrounding it. > Remove unneeded {}. Linus, if you are okay with the change, I can include in my usual PR. Whatever you prefer, though.
On Tue, Oct 3, 2023 at 2:06 PM Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > The one line conditional body doesn't require {} surrounding it. > Remove unneeded {}. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Patch applied! Yours, Linus Walleij
diff --git a/drivers/pinctrl/core.c b/drivers/pinctrl/core.c index e2f7519bef04..5a15583bf4ae 100644 --- a/drivers/pinctrl/core.c +++ b/drivers/pinctrl/core.c @@ -445,9 +445,9 @@ struct pinctrl_dev *pinctrl_find_and_add_gpio_range(const char *devname, * it has not probed yet, so the driver trying to register this * range need to defer probing. */ - if (!pctldev) { + if (!pctldev) return ERR_PTR(-EPROBE_DEFER); - } + pinctrl_add_gpio_range(pctldev, range); return pctldev;
The one line conditional body doesn't require {} surrounding it. Remove unneeded {}. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- drivers/pinctrl/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)