Re: [PATCH] clk: ti: dflt: remove redundant unlikely

2016-04-15 Thread Stephen Boyd
On 04/05, Joe Perches wrote: > > There are several of these: [...] > > Maybe a new checkpatch test like: Looks good to me. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: ti: dflt: remove redundant unlikely

2016-04-15 Thread Stephen Boyd
On 04/05, Suman Anna wrote: > Commit 7aba4f5201d1 ("clk: ti: dflt: fix enable_reg validity check") > fixed a validation check by using an IS_ERR() macro within the > existing unlikely expression, but IS_ERR() macro already has an > unlikely inside it, so get rid of the redundant unlikely macro > fr

Re: [PATCH] clk: ti: dflt: remove redundant unlikely

2016-04-05 Thread Joe Perches
On Tue, 2016-04-05 at 13:28 -0500, Suman Anna wrote: > Commit 7aba4f5201d1 ("clk: ti: dflt: fix enable_reg validity check") > fixed a validation check by using an IS_ERR() macro within the > existing unlikely expression, but IS_ERR() macro already has an > unlikely inside it, so get rid of the redu