Hi Rob, On Fri, Jun 27, 2025 at 12:09 AM Rob Herring <robh@xxxxxxxxxx> wrote: > > On Tue, Jun 17, 2025 at 02:44:58PM +0100, Prabhakar wrote: > > From: Thierry Bultel <thierry.bultel.yh@xxxxxxxxxxxxxx> > > > > At boot, the default clock is the PCLKM core clock (synchronous > > clock, which is enabled by the bootloader). > > For different baudrates, the asynchronous clock input must be used. > > Clock selection is made by an internal register of RCSI. > > > > Add the optional "sck", external clock input. > > > > Also remove the unneeded serial0 alias from the dts example. > > > > Signed-off-by: Thierry Bultel <thierry.bultel.yh@xxxxxxxxxxxxxx> > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > --- > > Hi Rob, > > As mentioned in the thread [1] below there are no users of the RSCI binding > > hence this change doesn not break any ABI. > > > > [1] https://lore.kernel.org/all/CAMuHMdUThuWxxznhjvcn5cOFCWOkb5u-fRYwTOoenDRY=4H6FA@xxxxxxxxxxxxxx/ > > Please state this in the commit message. If you want to break the ABI > you have to say that you are and why it is okay. > Sure, I'll update the commit message and send a new version. Cheers, Prabhakar