Re: PR/56095 CVS commit: src/sys/arch/arm/dts

[Jared McNeill <jmcneill%invisible.ca@localhost>]

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@localhost>
> To: gnats-bugs%gnats.NetBSD.org@localhost
> 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.