Hi Biju, On Thu, Aug 24, 2023 at 10:25 AM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > Fix the "assigned-clock-rates" for each clock output in the > example based on Table 3. ("Output Source") in the 5P35023 > datasheet(ie: {REF,SE1,SE2,SE3,DIFF1,DIFF2}). > > While at it, replace clocks phandle in the example from x1_x2->x1 as > X2 is a different 32768 kHz crystal. Thanks for your patch! > Suggested-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> This is not what I suggested. The bindings should document the meaning of the clock indices. > Fixes: a03d23f860eb ("dt-bindings: clock: Add Renesas versa3 clock generator bindings") This patch is not a critical fix. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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