2023-10-03 12:07:01

by Andy Shevchenko

Subject: [PATCH v1 1/1] pinctrl: core: Remove unneeded {} around one line conditional body

The one line conditional body doesn't require {} surrounding it.
Remove unneeded {}.

Signed-off-by: Andy Shevchenko <[email protected]>
---
drivers/pinctrl/core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

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;
--
2.40.0.1.gaa8946217a0b

2023-10-06 08:28:37

by Andy Shevchenko

Subject: Re: [PATCH v1 1/1] pinctrl: core: Remove unneeded {} around one line conditional body

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.

--
With Best Regards,
Andy Shevchenko

2023-10-10 12:06:24

by Linus Walleij

Subject: Re: [PATCH v1 1/1] pinctrl: core: Remove unneeded {} around one line conditional body

On Tue, Oct 3, 2023 at 2:06 PM Andy Shevchenko
<[email protected]> wrote:


> The one line conditional body doesn't require {} surrounding it.
> Remove unneeded {}.
>
> Signed-off-by: Andy Shevchenko <[email protected]>

Patch applied!

Yours,
Linus Walleij