Ravi, On 27/07/2023 08:36, Ravi Gunasekaran wrote: > > > On 7/26/23 6:22 PM, Nishanth Menon wrote: >> On 14:44-20230726, Ravi Gunasekaran wrote: >> [...] >>> If it is the former, then would a duplicate mdio node help keep the changes >>> to minimum? >> >> That is worse hack. How does it make sense to have two mdio nodes to >> represent the same hardware block? we are trying to get closer to kernel >> dts support not farther away from it. >> > > I know it's not a clean hack. In k3-am625-sk-u-boot.dtsi, as a hack, > the mdio pinctrl gets added to cpsw node. So was wondering if a duplicate > mdio pinctrl node could be added in the same file. Just wanted to know if we > could skip the changes in CPSW driver posted by Maxime. > > But Maxime's approach is much cleaner. We can just sync up kernel dts and not > keeping adding the hack to every K3 SoC's DT. > > Everything is sorted now as far as DT sync is concerned. Please see [1] and its dependencies.
Only thing missing is to have proper MDIO DM support for TI platforms in u-boot. Then we can get rid of the MDIO hack in am65-cpsw driver. [1] - https://lore.kernel.org/all/20230726151027.2517151-1...@ti.com/ -- cheers, -roger