From mboxrd@z Thu Jan  1 00:00:00 1970
From: Giulio Benetti <giulio.benetti@benettiengineering.com>
Date: Fri, 19 Feb 2021 12:45:34 +0100
Subject: [PATCH 1/5] arm: allwinner: dts: a20: add uart2_pa_pins
In-Reply-To: <20210219110441.44c12fc1@slackpad.fritz.box>
References: <20210218232622.883509-1-giulio.benetti@benettiengineering.com>
 <20210218232622.883509-2-giulio.benetti@benettiengineering.com>
 <20210219110441.44c12fc1@slackpad.fritz.box>
Message-ID: <7f0fc7e8-c52b-b5c3-ad17-c23114df9b26@benettiengineering.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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/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";