Hi Tomasz, On 17 October 2013 22:00, Tomasz Figa <t.figa@xxxxxxxxxxx> wrote: > The function can simply return 0, without jumping to a separate label, > which does exactly the same. This patch does not introduce any > functional change, just a clean-up. > > Signed-off-by: Tomasz Figa <t.figa@xxxxxxxxxxx> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > --- > drivers/clk/clk-max77686.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/clk/clk-max77686.c b/drivers/clk/clk-max77686.c > index e40ecdb..5d78460 100644 > --- a/drivers/clk/clk-max77686.c > +++ b/drivers/clk/clk-max77686.c > @@ -178,7 +178,7 @@ static int max77686_clk_probe(struct platform_device *pdev) > > platform_set_drvdata(pdev, max77686_clks); > > - goto out; > + return 0; > > err_clk_pmic: > clkdev_drop(max77686_clks[MAX77686_CLK_CP]->lookup); > @@ -187,7 +187,6 @@ err_clk_cp: > clkdev_drop(max77686_clks[MAX77686_CLK_AP]->lookup); > kfree(max77686_clks[MAX77686_CLK_AP]->hw.clk); > err_clk_ap: You could probably do the same for this label too. > -out: > return ret; > } > -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
