2014-04-01 10:46:51

by Daeseok Youn

[permalink] [raw]

Subject: [PATCH] regmap: adds missing braces in regmap_init()

---

It need to add curly braces because the inner for "if" has
two statements.

coccicheck says:
drivers/base/regmap/regmap.c:765:2-44:
code aligned with following code on line 766

Signed-off-by: Daeseok Youn <[email protected]>
---
drivers/base/regmap/regmap.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c
index d0a0724..63e30ef 100644
--- a/drivers/base/regmap/regmap.c
+++ b/drivers/base/regmap/regmap.c
@@ -761,10 +761,11 @@ skip_format_initialization:
if (ret != 0)
goto err_range;

- if (dev)
+ if (dev) {
ret = regmap_attach_dev(dev, map, config);
if (ret != 0)
goto err_regcache;
+ }

return map;

--
1.7.4.4

2014-04-01 11:09:59

by Mark Brown

[permalink] [raw]

Subject: Re: [PATCH] regmap: adds missing braces in regmap_init()

---

On Tue, Apr 01, 2014 at 07:46:43PM +0900, Daeseok Youn wrote:
>
> It need to add curly braces because the inner for "if" has
> two statements.

Applied, thanks.

---

Attachments:

(No filename) (162.00 B)
signature.asc (836.00 B)
Digital signature Download all attachments