[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