Hello Jerome, On Sun, 13 Nov 2022 20:29:43 +0100 Jerome Brunet <jbrunet@xxxxxxxxxxxx> wrote: > On Sun 13 Nov 2022 at 09:48, Heiko Stübner <heiko@xxxxxxxxx> wrote: > > > Am Samstag, 12. November 2022, 22:43:03 CET schrieb Christophe JAILLET: > >> <linux/rational.h> is not needed for these drivers. Remove the > >> corresponding #include. > >> > >> Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > >> --- > >> drivers/clk/clk-versaclock5.c | 1 - > >> drivers/clk/meson/clk-pll.c | 1 - > >> drivers/clk/rockchip/clk.c | 1 - > > > > Clock-trees for soc families normally have different sub-maintainers, > > so depending on how we want to merge this, you might need to split > > this into 3 patches. > > > > But at least for the Rockchip part, I'm also fine with the patch going > > in as a whole, so (for the Rockchip part): > > > > Acked-by: Heiko Stuebner <heiko@xxxxxxxxx> > > > > > > Heiko > > Same for the Amlogic part > > Acked-by: Jerome Brunet <jbrunet@xxxxxxxxxxxx> And the same for Versaclock: Reviewed-by: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com