On Thu, Jun 12, 2025 at 04:51:42PM +0200, Tommaso Merciai wrote: > 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. Previous Krzysztof give some feedback about clock-names, they don't perfer clk surfix, because clock-names already provide information, which is clock. Some hardware manual use it as signal name, for example, pclk which is apb bus clock. I am fine with align signal name. you can wait for DT team's feedback about this. Frank > > Thanks & Regards, > Tommaso
