Thread (14 messages) 14 messages, 4 authors, 2021-04-17

Re: [PATCH 0/2] fdt: translate address if #size-cells = <0>

From: Tony Lindgren <tony@atomide.com>
Date: 2021-04-17 08:37:18
Also in: linux-clk, linux-omap, lkml

* Dario Binacchi [off-list ref] [210414 20:40]:
quoted
Il 12/04/2021 09:41 Tero Kristo [off-list ref] ha scritto:
The change on the DT itself would be pretty large, removing all clock 
nodes and modifying any existing handles towards the clock nodes, and 
this would impact all OMAP architectures.

Anyways, it is mostly up-to Tony how he wants to see the DT change, as 
he is the maintainer for the OMAP family DT data.
While I think all the clocks should use a similar binding to the clkctrl
binding, I don't know if it makes sense to start changing things around
at such a large scale.

Certainly if somebody does the patches and they can be tested to not cause
regressions, sure why not :)
quoted
I am just raising the opinion here that from kernel point of view, 
adding the missing size cells seems unnecessary, and I can't see why 
u-boot can't be changed to support the existing broken DT. It is broken 
now, and it will be broken with the addition of the size cells in place, 
and the actual "neat" end result would be to get rid of the clock nodes 
completely.
I'll fix U-boot.
Thanks for your explanations.
Hope for SSC patch review from you and/or some TI MAINTAINER.
Best to fix the issues first, then make any clean-up patches a separate
series.

Regards,

Tony
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help