On Mon, 2019-06-17 at 14:02 +0200, Geert Uytterhoeven wrote: > A boolean expression already evaluates to true or false, so there is no > need to check the result and return true or false explicitly. > > Signed-off-by: Geert Uytterhoeven <[email protected]> > --- > drivers/clk/clk.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c > index aa51756fd4d695b3..c68bc5f695912bf5 100644 > --- a/drivers/clk/clk.c > +++ b/drivers/clk/clk.c > @@ -1324,10 +1324,7 @@ static void clk_core_init_rate_req(struct clk_core * > const core, > > static bool clk_core_can_round(struct clk_core * const core) > { > - if (core->ops->determine_rate || core->ops->round_rate) > - return true; > - > - return false; > + return core->ops->determine_rate || core->ops->round_rate; > } > > static int clk_core_round_rate_nolock(struct clk_core *core,
Reviewed-by: Jerome Brunet <[email protected]>
