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

Tommaso Merciai
2025-06-24T00:30:53Z

Hi Frank, Wolfram,

Thanks both for your work.

On 12/06/25 16:31, Wolfram Sang wrote:
> 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.
>

Agreed.
Since the name comes from the hw manual I think is better to leave it as is.

Thanks & Regards,
Tommaso