Hi Stephen Boyd, > Subject: Re: [PATCH v3 1/4] dt-bindings: clock: versaclock3: Document > clock-output-names > > Quoting Biju Das (2023-08-17 07:22:08) > > Document clock-output-names property and 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. > > > > Suggested-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > Closes: > > Fixes: a03d23f860eb ("dt-bindings: clock: Add Renesas versa3 clock > > generator bindings") > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > > --- > > Applied to clk-next As per Geert's suggestion [1], I am about to send a patch for dropping "clock-output-names", as there is no validation for it and people can get it wrong. Is it ok,if I send a patch dropping clock-output-names? Please let me know your opinion on this. [1] https://patchwork.kernel.org/project/linux-renesas-soc/patch/20230817090810.203900-2-biju.das.jz@xxxxxxxxxxxxxx/ Cheers, Biju