On 19.02.2024 08:59, Krzysztof Kozlowski wrote: > On 19/02/2024 07:25, claudiu beznea wrote: >> >> >> On 08.02.2024 18:37, Krzysztof Kozlowski 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/at91/pmc.c | 3 +- >>> drivers/clk/at91/pmc.h | 3 +- >> >> Reviewed-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxx> > > You understand there is no review for "part of patch"? You probably > meant Acked-by. Sure... Here it is: Acked-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxx> > > https://elixir.bootlin.com/linux/v6.8-rc5/source/Documentation/process/submitting-patches.rst#L544 > > Best regards, > Krzysztof >
