Re: [PATCH] clk: constify the of_phandle_args argument of of_clk_provider

From: Geert Uytterhoeven
Date: Fri Feb 16 2024 - 06:47:24 EST


On Thu, Feb 8, 2024 at 5:37 PM Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxx> wrote:
> None of the implementations of the get() and get_hw() callbacks of
> "struct of_clk_provider" modify the contents of received of_phandle_args
> pointer. They treat it as read-only variable used to find the clock to
> return. Make obvious that implementations are not supposed to modify
> the of_phandle_args, by making it a pointer to const.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

> drivers/clk/renesas/r9a06g032-clocks.c | 2 +-
> drivers/clk/renesas/renesas-cpg-mssr.c | 2 +-
> drivers/clk/renesas/rzg2l-cpg.c | 2 +-

Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
Acked-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68korg

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds