[PATCH] ARM: dts: at91-sama5d27_som1: fix phy address to 7

[Ahmad Fatoum]

Hello Alexander,

On 23.03.21 11:45, Alexander Dahl wrote:
> Hei hei,
> 
> I could not get ethernet to work on SAMA5D27-SOM1-EK1 with kernels v5.10 and v5.11 built by a recent ptxdist based DistroKit BSP, while it used to work with an older v4.19 kernel. Just applying this patch to the tree made ethernet working again, thus:
> 
> Tested-by: Alexander Dahl <ada at thorsis.com>
> 
> Not sure why it worked with that older kernel, though.

Thanks for investigating! Seems that somehow PHY broadcast worked on this
board with older kernels (and current barebox), but no longer does with
newer kernels.

A bisection could shed some light onto what broke this.

As the KSZ8081 driver disables broadcast in the phy config init, this change
looks appropriate regardless. The fixes tag doesn't refer to an upstream
commit though. This should probably read:
Fixes: af690fa37e39 ("ARM: dts: at91: at91-sama5d27_som1: add sama5d27 SoM1 support")

With this addressed:

Reviewed-by: Ahmad Fatoum <a.fatoum at pengutronix.de>

You could send a proper patch and stick your S-o-b under it.

Cheers,
Ahmad

> 
> I added Ahmad to Cc, he added board support to DistroKit for that board, and might want to know. And I added the devicetree list to Cc, I wondered why the patch was not there and get_maintainers.pl proposed it.
> 
> Thanks for fixing this and greetings
> Alex
> 
>> nicolas.ferre at microchip.com hat am 17.02.2021 12:38 geschrieben:
>>
>>  
>> From: Claudiu Beznea <claudiu.beznea at microchip.com>
>>
>> Fix the phy address to 7 for Ethernet PHY on SAMA5D27 SOM1. No
>> connection established if phy address 0 is used.
>>
>> The board uses the 24 pins version of the KSZ8081RNA part, KSZ8081RNA
>> pin 16 REFCLK as PHYAD bit [2] has weak internal pull-down.  But at
>> reset, connected to PD09 of the MPU it's connected with an internal
>> pull-up forming PHYAD[2:0] = 7.
>>
>> Signed-off-by: Claudiu Beznea <claudiu.beznea at microchip.com>
>> Fixes: 2f61929eb10a ("ARM: dts: at91: at91-sama5d27_som1: fix PHY ID")
>> Cc: Ludovic Desroches <ludovic.desroches at microchip.com>
>> Signed-off-by: Nicolas Ferre <nicolas.ferre at microchip.com>
>> Cc: <stable at vger.kernel.org> # 4.14+
>> ---
>>  arch/arm/boot/dts/at91-sama5d27_som1.dtsi | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/at91-sama5d27_som1.dtsi b/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
>> index 1b1163858b1d..e3251f3e3eaa 100644
>> --- a/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
>> +++ b/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
>> @@ -84,8 +84,8 @@ macb0: ethernet at f8008000 {
>>  				pinctrl-0 = <&pinctrl_macb0_default>;
>>  				phy-mode = "rmii";
>>  
>> -				ethernet-phy at 0 {
>> -					reg = <0x0>;
>> +				ethernet-phy at 7 {
>> +					reg = <0x7>;
>>  					interrupt-parent = <&pioA>;
>>  					interrupts = <PIN_PD31 IRQ_TYPE_LEVEL_LOW>;
>>  					pinctrl-names = "default";
>> -- 
>> 2.30.0
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |