| Message ID | 20170702163016.6714-2-paul@crapouillou.net (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
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 <paul@crapouillou.net> > --- 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 <sboyd@codeaurora.org> for patches 1 through 6.
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 <paul@crapouillou.net> >> --- > > 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 <sboyd@codeaurora.org> > > for patches 1 through 6. I think ACK; then Ralf can take them in 4.13 :) Thanks, -Paul -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 <sboyd@codeaurora.org> > > > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 <sboyd@codeaurora.org> > > > > > > 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.
Hi, This is my v4 of my patch series to support the JZ4770 SoC from Ingenic as well as the GCW Zero handheld console. Not much changed since v3, I dropped the three serial-related patches (07-08-09/18) as I will submit them separately. The only other change is that I rebased the patch series on top of v4.15-rc5. Regards, -Paul Cercueil -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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,
The CGU common code does not modify the pointed clk_ops structure, so it should be marked as const. Signed-off-by: Paul Cercueil <paul@crapouillou.net> --- 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