On 10/02/2015 01:21 PM, Stephen Boyd wrote: > On 09/29, Suman Anna wrote: >> diff --git a/drivers/clk/ti/clkt_dflt.c b/drivers/clk/ti/clkt_dflt.c >> index 90d7d8a21c49..1ddc288fce4e 100644 >> --- a/drivers/clk/ti/clkt_dflt.c >> +++ b/drivers/clk/ti/clkt_dflt.c >> @@ -222,7 +222,7 @@ int omap2_dflt_clk_enable(struct clk_hw *hw) >> } >> } >> >> - if (unlikely(!clk->enable_reg)) { >> + if (unlikely(IS_ERR(clk->enable_reg))) { > > IS_ERR() already has an unlikely inside it, so the unlikely is > redundant here. Thanks Stephen, didn't realize that. I will fix this up in a subsequent patch, Tero has already staged it and sent a PULL request. regards Suman -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html