[PATCH RFC 3/7] dt-bindings: i3c: renesas,i3c: Add binding for Renesas I3C controller
Tommaso Merciai
tommaso.merciai.xr at bp.renesas.com
Thu Jun 12 07:51:42 PDT 2025
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
More information about the linux-i3c
mailing list