Skip to content

Commit dc75925

Browse files
Benoit CoussonPaul Walmsley
Benoit Cousson
authored and
Paul Walmsley
committed
OMAP: hwmod: Fix the missing braces
As reported by Sergei, a couple of braces were missing after the WARN removal patch. [07/22] OMAP: hwmod: Replace WARN by pr_warning if clock lookup failed https://patchwork.kernel.org/patch/100756/ Signed-off-by: Benoit Cousson <[email protected]> [[email protected]: fixed patch description per Anand's E-mail] Signed-off-by: Paul Walmsley <[email protected]> Cc: Sergei Shtylyov <[email protected]> Cc: Anand Gadiyar <[email protected]>
1 parent 7e27d6e commit dc75925

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

arch/arm/mach-omap2/omap_hwmod.c

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -409,10 +409,11 @@ static int _init_main_clk(struct omap_hwmod *oh)
409409
return 0;
410410

411411
oh->_clk = omap_clk_get_by_name(oh->main_clk);
412-
if (!oh->_clk)
412+
if (!oh->_clk) {
413413
pr_warning("omap_hwmod: %s: cannot clk_get main_clk %s\n",
414414
oh->name, oh->main_clk);
415415
return -EINVAL;
416+
}
416417

417418
if (!oh->_clk->clkdm)
418419
pr_warning("omap_hwmod: %s: missing clockdomain for %s.\n",
@@ -444,10 +445,11 @@ static int _init_interface_clks(struct omap_hwmod *oh)
444445
continue;
445446

446447
c = omap_clk_get_by_name(os->clk);
447-
if (!c)
448+
if (!c) {
448449
pr_warning("omap_hwmod: %s: cannot clk_get interface_clk %s\n",
449450
oh->name, os->clk);
450451
ret = -EINVAL;
452+
}
451453
os->_clk = c;
452454
}
453455

@@ -470,10 +472,11 @@ static int _init_opt_clks(struct omap_hwmod *oh)
470472

471473
for (i = oh->opt_clks_cnt, oc = oh->opt_clks; i > 0; i--, oc++) {
472474
c = omap_clk_get_by_name(oc->clk);
473-
if (!c)
475+
if (!c) {
474476
pr_warning("omap_hwmod: %s: cannot clk_get opt_clk %s\n",
475477
oh->name, oc->clk);
476478
ret = -EINVAL;
479+
}
477480
oc->_clk = c;
478481
}
479482

0 commit comments

Comments
 (0)