| Message ID | 1503410786-25519-1-git-send-email-bhumirks@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
On Tue, Aug 22, 2017 at 07:36:26PM +0530, Bhumika Goyal wrote: > Make this const as it is only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> Acked-by: Liviu Dudau <Liviu.Dudau@arm.com> Thanks for the patch! Sudeep, can you going to pick this up for 4.14? Best regards, Liviu > --- > drivers/clk/versatile/clk-vexpress-osc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/versatile/clk-vexpress-osc.c b/drivers/clk/versatile/clk-vexpress-osc.c > index 7e5add7..e7a868b 100644 > --- a/drivers/clk/versatile/clk-vexpress-osc.c > +++ b/drivers/clk/versatile/clk-vexpress-osc.c > @@ -61,7 +61,7 @@ static int vexpress_osc_set_rate(struct clk_hw *hw, unsigned long rate, > return regmap_write(osc->reg, 0, rate); > } > > -static struct clk_ops vexpress_osc_ops = { > +static const struct clk_ops vexpress_osc_ops = { > .recalc_rate = vexpress_osc_recalc_rate, > .round_rate = vexpress_osc_round_rate, > .set_rate = vexpress_osc_set_rate, > -- > 1.9.1 >
On 24/08/17 10:54, Liviu Dudau wrote: > On Tue, Aug 22, 2017 at 07:36:26PM +0530, Bhumika Goyal wrote: >> Make this const as it is only stored in the const field of a >> clk_init_data structure. >> >> Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> > > Acked-by: Liviu Dudau <Liviu.Dudau@arm.com> > > Thanks for the patch! > > Sudeep, can you going to pick this up for 4.14? > IIUC anything under drivers/clk is routed via clk tree. I assume Mike/Stephen will pick this up.
On 08/22, Bhumika Goyal wrote: > Make this const as it is only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> > --- Applied to clk-next
diff --git a/drivers/clk/versatile/clk-vexpress-osc.c b/drivers/clk/versatile/clk-vexpress-osc.c index 7e5add7..e7a868b 100644 --- a/drivers/clk/versatile/clk-vexpress-osc.c +++ b/drivers/clk/versatile/clk-vexpress-osc.c @@ -61,7 +61,7 @@ static int vexpress_osc_set_rate(struct clk_hw *hw, unsigned long rate, return regmap_write(osc->reg, 0, rate); } -static struct clk_ops vexpress_osc_ops = { +static const struct clk_ops vexpress_osc_ops = { .recalc_rate = vexpress_osc_recalc_rate, .round_rate = vexpress_osc_round_rate, .set_rate = vexpress_osc_set_rate,
Make this const as it is only stored in the const field of a clk_init_data structure. Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> --- drivers/clk/versatile/clk-vexpress-osc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)