Re: [PATCH RFC 3/7] dt-bindings: i3c: renesas,i3c: Add binding for Renesas I3C controller

Wolfram Sang
2025-06-24T00:30:53Z

[-- Attachment #1.1: Type: text/plain, Size: 617 bytes --]

Hi Frank,

> > + clock-names:
> > + oneOf:
> > + - items:
> > + - const: pclk
> > + - const: tclk
> > + - items:
> > + - const: pclk
> > + - const: pclkrw
> > + - const: tclk
>
> Suggested clock name is, apb, trans, srfs.
> Needn't clk surfix and provide meanful name.

? Why can't we have the name from the datasheets? It will be
unnecessarily hard for users to verify the correctness of the clock
assignments if they need an artifical mapping of these names. If they
are the same as in the datasheet, the same task becomes very easy.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 111 bytes --]