[PATCH 0/4] net: ti: am65-cpsw-nuss: Fix DT binding handling of pinctrl

Roger Quadros rogerq at kernel.org
Thu Jul 27 09:10:03 CEST 2023


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-nm@ti.com/

-- 
cheers,
-roger


More information about the U-Boot mailing list