[PATCH 1/5] arm: allwinner: dts: a20: add uart2_pa_pins

[Giulio Benetti]

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 at benettiengineering.com> wrote:
> 
> Hi,
> 
>> From: Giulio Benetti <giulio.benetti at micronovasrl.com>
>>
>> Signed-off-by: Giulio Benetti <giulio.benetti at micronovasrl.com>
>> Signed-off-by: Giulio Benetti <giulio.benetti at 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";