Hi Andre,
On 2/19/21 12:04 PM, Andre Przywara wrote:
On Fri, 19 Feb 2021 00:26:16 +0100 Giulio Benetti giulio.benetti@benettiengineering.com wrote:
Hi,
From: Giulio Benetti giulio.benetti@micronovasrl.com
Signed-off-by: Giulio Benetti giulio.benetti@micronovasrl.com Signed-off-by: Giulio Benetti giulio.benetti@benettiengineering.com
arch/arm/dts/sun7i-a20.dtsi | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/arch/arm/dts/sun7i-a20.dtsi b/arch/arm/dts/sun7i-a20.dtsi index a013805831..8bf1921500 100644 --- a/arch/arm/dts/sun7i-a20.dtsi +++ b/arch/arm/dts/sun7i-a20.dtsi @@ -943,6 +943,11 @@ function = "uart1"; };
uart2_pa_pins: uart2-pa-pins {pins = "PA2", "PA3";function = "uart2";};This is already in the Linux .dtsi, so we just need to sync that (and probably all the sun7i-a20 .dts files) to U-Boot. There are some changes around the PHY nodes, so we need to check if that affects the operation of the U-Boot GMAC driver.
You're right, instead of only reset-delay-us also other properties have been introduced like: - reset-assert-us - reset-deassert-us
https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bind...
So basically I could add a patch for it where I add those 2 delays to already present reset_delays array like: (pseudocode): reset_delays[1] = "reset-assert-us"
reset_delays[2] = "reset-deassert-us"
That way we both accept reset delays array and new mdio properties. I need some time for that and possibly someone to test if possible/available.
Kind regards