NetBSD Problem Report #56095

From www@netbsd.org  Tue Apr  6 20:57:00 2021
Return-Path: <www@netbsd.org>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))
	(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id F19441A923E
	for <gnats-bugs@gnats.NetBSD.org>; Tue,  6 Apr 2021 20:56:59 +0000 (UTC)
Message-Id: <20210406205658.8331B1A9242@mollari.NetBSD.org>
Date: Tue,  6 Apr 2021 20:56:58 +0000 (UTC)
From: snow@datagirl.xyz
Reply-To: snow@datagirl.xyz
To: gnats-bugs@NetBSD.org
Subject: SOPINE Clusterboard unable to transmit network data
X-Send-Pr-Version: www-1.0

>Number:         56095
>Category:       port-evbarm
>Synopsis:       SOPINE Clusterboard unable to transmit network data
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    port-evbarm-maintainer
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Tue Apr 06 21:00:00 +0000 2021
>Closed-Date:    
>Last-Modified:  Wed Jun 30 06:24:47 +0000 2021
>Originator:     snow flurry
>Release:        current, 202104041700Z
>Organization:
>Environment:
NetBSD fairy01 9.99.81 NetBSD 9.99.81 (GENERIC64) #0: Sat Apr  3 07:38:11 UTC 2021  mkrepro@mkrepro.NetBSD.org:/usr/src/sys/arch/evbarm/compile/GENERIC64 evbarm
>Description:
With the current sun50i-a64-sopine-baseboard dtb, SOPINE units connected to a clusterboard are unable to transmit network data over emac0. Cursory testing with tcpdump shows packets are being received and show as sent on the SOPINE, but said sent packets never show up on the other end.
>How-To-Repeat:
1. Connect an ethernet cable between the clusterboard and a secondary device (ex. laptop)
2. Attempt to access the SOPINE via the secondary device; or, using the SOPINE, try to access the secondary device
>Fix:
The following patch to the sun50i-a64-sopine-baseboard dts (also used by the SOPINE in clusterboard configuration) seems to fix the issue.

--- sys/external/gpl2/dts/dist/arch/arm64/boot/dts/allwinner/sun50i-a64-sopine-baseboard.dts.orig	2021-04-06 20:37:08.252335236 +0000
+++ sys/external/gpl2/dts/dist/arch/arm64/boot/dts/allwinner/sun50i-a64-sopine-baseboard.dts	2021-04-06 00:26:16.422894811 +0000
@@ -122,6 +122,7 @@
 	phy-mode = "rgmii";
 	phy-handle = <&ext_rgmii_phy>;
 	phy-supply = <&reg_dc1sw>;
+	allwinner,tx-delay-ps = <500>;
 	status = "okay";
 };


>Release-Note:

>Audit-Trail:
From: Tobias Nygren <tnn@NetBSD.org>
To: gnats-bugs@netbsd.org, snow@datagirl.xyz
Cc: 
Subject: Re: port-evbarm/56095: SOPINE Clusterboard unable to transmit
 network data
Date: Tue, 6 Apr 2021 23:49:31 +0200

 Hi,

 On Tue,  6 Apr 2021 21:00:01 +0000 (UTC)
 snow@datagirl.xyz wrote:

 > The following patch to the sun50i-a64-sopine-baseboard dts (also used by the SOPINE in clusterboard configuration) seems to fix the issue.
 > 
 > +++ sys/external/gpl2/dts/dist/arch/arm64/boot/dts/allwinner/sun50i-a64-sopine-baseboard.dts	

 We generally try to avoid patching the upstream dts files.
 Can you try to patch our local src/sys/arch/arm/dts/sun50i-a64-sopine-baseboard.dts instead?
 It should suffice to add the &emac node with the same override.
 Thanks!

 -Tobias

From: snow flurry <snow@datagirl.xyz>
To: Tobias Nygren <tnn@NetBSD.org>
Cc: gnats-bugs@netbsd.org
Subject: Re: port-evbarm/56095: SOPINE Clusterboard unable to transmit
 network data
Date: Tue, 6 Apr 2021 16:56:02 -0700

 Sure. This patch worked for me (produced the same dtb):

 --- sys/arch/arm/dts/sun50i-a64-sopine-baseboard.dts.orig   2021-04-06 21:50:12.873593543 +0000
 +++ sys/arch/arm/dts/sun50i-a64-sopine-baseboard.dts    2021-04-06 21:51:07.919416729 +0000
 @@ -33,6 +33,10 @@
     status = "okay";
  };

 +&emac {
 +   allwinner,tx-delay-ps = <500>;
 +};
 +
  &sound_hdmi {
     status = "okay";
  };

State-Changed-From-To: open->closed
State-Changed-By: skrll@NetBSD.org
State-Changed-When: Fri, 25 Jun 2021 07:04:04 +0000
State-Changed-Why:
Fix is committed.


From: "Nick Hudson" <skrll@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/56095 CVS commit: src/sys/arch/arm/dts
Date: Fri, 25 Jun 2021 07:03:28 +0000

 Module Name:	src
 Committed By:	skrll
 Date:		Fri Jun 25 07:03:28 UTC 2021

 Modified Files:
 	src/sys/arch/arm/dts: sun50i-a64-sopine-baseboard.dts

 Log Message:
 PR/56095 SOPINE Clusterboard unable to transmit network data

 Add allwinner,tx-delay-ps = <500> using the diff supplied by tnn@


 To generate a diff of this commit:
 cvs rdiff -u -r1.2 -r1.3 src/sys/arch/arm/dts/sun50i-a64-sopine-baseboard.dts

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Jared McNeill <jmcneill@invisible.ca>
To: gnats-bugs@netbsd.org, skrll@netbsd.org, tnn@netbsd.org
Cc: gnats-admin@netbsd.org, netbsd-bugs@netbsd.org, snow@datagirl.xyz
Subject: Re: PR/56095 CVS commit: src/sys/arch/arm/dts
Date: Sat, 26 Jun 2021 19:31:01 -0300 (ADT)

 I don't think this change is correct. The .dts file you are changing is 
 for the _SOPine Baseboard Model A_, which has worked fine to date with the 
 old config. If _Clusterboard_ is using the same .dtb and needs a different 
 config, that is a bug that we need to fix in a Clusterboard-specific 
 U-Boot. Either by using a different .dtb or by programming the TX delay in 
 U-Boot up front.

 Take care,
 Jared


 On Fri, 25 Jun 2021, Nick Hudson wrote:

 > The following reply was made to PR port-evbarm/56095; it has been noted by GNATS.
 >
 > From: "Nick Hudson" <skrll@netbsd.org>
 > To: gnats-bugs@gnats.NetBSD.org
 > Cc:
 > Subject: PR/56095 CVS commit: src/sys/arch/arm/dts
 > Date: Fri, 25 Jun 2021 07:03:28 +0000
 >
 > Module Name:	src
 > Committed By:	skrll
 > Date:		Fri Jun 25 07:03:28 UTC 2021
 >
 > Modified Files:
 > 	src/sys/arch/arm/dts: sun50i-a64-sopine-baseboard.dts
 >
 > Log Message:
 > PR/56095 SOPINE Clusterboard unable to transmit network data
 >
 > Add allwinner,tx-delay-ps = <500> using the diff supplied by tnn@
 >
 >
 > To generate a diff of this commit:
 > cvs rdiff -u -r1.2 -r1.3 src/sys/arch/arm/dts/sun50i-a64-sopine-baseboard.dts
 >
 > Please note that diffs are not public domain; they are subject to the
 > copyright notices on the relevant files.
 >
 >
 >

State-Changed-From-To: closed->open
State-Changed-By: skrll@NetBSD.org
State-Changed-When: Wed, 30 Jun 2021 06:24:47 +0000
State-Changed-Why:
re-think required.


>Unformatted:

Home	PR Database Search