Re: [PATCH v3 01/18] clk: ingenic: Use const pointer to clk_ops in struct
On 07/13, Ralf Baechle wrote: > On Thu, Jul 13, 2017 at 12:07:25PM +0200, Paul Cercueil wrote: > > > > Sorry I forgot, did you want an ack for these clk patches or for > > > me to take them through clk tree. If it's the ack case, > > > > > > Acked-by: Stephen Boyd > > > > > > for patches 1 through 6. > > > > I think ACK; then Ralf can take them in 4.13 :) > > My pull request for 4.13 is already finalized so it'd be great if this > could make it to 4.13 through the clk tree. If that should be impossible > I'd like to merge this via the MIPS tree for 4.14. > It's too late for v4.13, so you can take it for v4.14. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH v3 01/18] clk: ingenic: Use const pointer to clk_ops in struct
On Thu, Jul 13, 2017 at 12:07:25PM +0200, Paul Cercueil wrote: > > Sorry I forgot, did you want an ack for these clk patches or for > > me to take them through clk tree. If it's the ack case, > > > > Acked-by: Stephen Boyd > > > > for patches 1 through 6. > > I think ACK; then Ralf can take them in 4.13 :) My pull request for 4.13 is already finalized so it'd be great if this could make it to 4.13 through the clk tree. If that should be impossible I'd like to merge this via the MIPS tree for 4.14. Thanks, Ralf
Re: [PATCH v3 01/18] clk: ingenic: Use const pointer to clk_ops in struct
Le 2017-07-13 01:20, Stephen Boyd a écrit : On 07/02, Paul Cercueil wrote: The CGU common code does not modify the pointed clk_ops structure, so it should be marked as const. Signed-off-by: Paul Cercueil --- Sorry I forgot, did you want an ack for these clk patches or for me to take them through clk tree. If it's the ack case, Acked-by: Stephen Boyd for patches 1 through 6. I think ACK; then Ralf can take them in 4.13 :) Thanks, -Paul
Re: [PATCH v3 01/18] clk: ingenic: Use const pointer to clk_ops in struct
On 07/02, Paul Cercueil wrote: > The CGU common code does not modify the pointed clk_ops structure, so it > should be marked as const. > > Signed-off-by: Paul Cercueil > --- Sorry I forgot, did you want an ack for these clk patches or for me to take them through clk tree. If it's the ack case, Acked-by: Stephen Boyd for patches 1 through 6. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
[PATCH v3 01/18] clk: ingenic: Use const pointer to clk_ops in struct
The CGU common code does not modify the pointed clk_ops structure, so it should be marked as const. Signed-off-by: Paul Cercueil --- drivers/clk/ingenic/cgu.h| 2 +- drivers/clk/ingenic/jz4780-cgu.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) v2: New patch in this series v3: No change diff --git a/drivers/clk/ingenic/cgu.h b/drivers/clk/ingenic/cgu.h index 09700b2c555d..da448b0cac18 100644 --- a/drivers/clk/ingenic/cgu.h +++ b/drivers/clk/ingenic/cgu.h @@ -120,7 +120,7 @@ struct ingenic_cgu_gate_info { * @clk_ops: custom clock operation callbacks */ struct ingenic_cgu_custom_info { - struct clk_ops *clk_ops; + const struct clk_ops *clk_ops; }; /** diff --git a/drivers/clk/ingenic/jz4780-cgu.c b/drivers/clk/ingenic/jz4780-cgu.c index b35d6d9dd5aa..a21698fb202c 100644 --- a/drivers/clk/ingenic/jz4780-cgu.c +++ b/drivers/clk/ingenic/jz4780-cgu.c @@ -203,7 +203,7 @@ static int jz4780_otg_phy_set_rate(struct clk_hw *hw, unsigned long req_rate, return 0; } -static struct clk_ops jz4780_otg_phy_ops = { +static const struct clk_ops jz4780_otg_phy_ops = { .get_parent = jz4780_otg_phy_get_parent, .set_parent = jz4780_otg_phy_set_parent, -- 2.11.0