Hi Rob,
On Fri, 9 May 2025 at 20:59, Rob Herring <robh@xxxxxxxxxx> wrote:
> On Tue, Apr 29, 2025 at 10:19:43AM +0200, Thierry Bultel wrote:
> > At boot, the default clock is the PCLKM core lock (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.
> >
> > Signed-off-by: Thierry Bultel <thierry.bultel.yh@xxxxxxxxxxxxxx>
> > ---
> > .../bindings/serial/renesas,rsci.yaml | 18 +++++++++---------
> > 1 file changed, 9 insertions(+), 9 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/serial/renesas,rsci.yaml b/Documentation/devicetree/bindings/serial/renesas,rsci.yaml
> > index ea879db5f485..aa2428837a2f 100644
> > --- a/Documentation/devicetree/bindings/serial/renesas,rsci.yaml
> > +++ b/Documentation/devicetree/bindings/serial/renesas,rsci.yaml
> > @@ -35,10 +35,14 @@ properties:
> > - const: tei
> >
> > clocks:
> > - maxItems: 1
> > + items:
> > + - description: serial functional clock
> > + - description: default core clock
> >
> > clock-names:
> > - const: fck # UART functional clock
> > + items:
> > + - const: async
> > + - const: bus
>
> This is an ABI change. You can't just drop 'fck' without good reasons.
This is fine, as there are no users yet.
The initial DT bindings were written based on a limited understanding of
the device. Let's hope our current understanding is less limited ;-)
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
