On 2018년 10월 27일 14:47, Julia Lawall wrote: > The clk_ops structure is only stored in the ops fields of > clk_init_data structures. This field is const, so the clk_ops > structure can be const as well. > > Identified and transformed using Coccinelle. > > Signed-off-by: Julia Lawall <[email protected]> > > --- > drivers/clk/clk-s2mps11.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/clk-s2mps11.c b/drivers/clk/clk-s2mps11.c > index 5b419b82f7ca..2ce370c804aa 100644 > --- a/drivers/clk/clk-s2mps11.c > +++ b/drivers/clk/clk-s2mps11.c > @@ -71,7 +71,7 @@ static unsigned long s2mps11_clk_recalc_rate(struct clk_hw > *hw, > return 32768; > } > > -static struct clk_ops s2mps11_clk_ops = { > +static const struct clk_ops s2mps11_clk_ops = { > .prepare = s2mps11_clk_prepare, > .unprepare = s2mps11_clk_unprepare, > .is_prepared = s2mps11_clk_is_prepared, > > >
Looks good to me. Reviewed-by: Chanwoo Choi <[email protected]> -- Best Regards, Chanwoo Choi Samsung Electronics
