Hi Andre,
On 2/19/21 12:04 PM, Andre Przywara wrote:
On Fri, 19 Feb 2021 00:26:16 +0100
Giulio Benetti <giulio.bene...@benettiengineering.com> wrote:
Hi,
From: Giulio Benetti <giulio.bene...@micronovasrl.com>
Signed-off-by: Giulio Benetti <giulio.bene...@micronovasrl.com>
Signed-off-by: Giulio Benetti <giulio.bene...@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/bindings/net/mdio.yaml#L90
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
--
Giulio Benetti
Benetti Engineering sas
Cheers,
Andre
uart2_pi_pins: uart2-pi-pins {
pins = "PI18", "PI19";
function = "uart2";
