diff mbox series

[meta-rockchip] nanopi-r4s: refresh device-tree patch for linux-6.16.y

Message ID 20250904132717.8973-1-twoerner@gmail.com
State New
Headers show
Series [meta-rockchip] nanopi-r4s: refresh device-tree patch for linux-6.16.y | expand

Commit Message

Trevor Woerner Sept. 4, 2025, 1:27 p.m. UTC
The device-tree organization of the nanopi-r4s device changed slightly in
the linux-6.16.y release, therefore refresh the nanopi-r4s device-tree
patch to match.

Signed-off-by: Trevor Woerner <twoerner@gmail.com>
---
 .../linux/linux-yocto/nanopi-r4s.scc          |  2 +-
 ...opi-r4s.dtsi-enable-second-ethernet.patch} | 21 +++++++++----------
 2 files changed, 11 insertions(+), 12 deletions(-)
 rename recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/{0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch => 0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch} (56%)

Comments

Quentin Schulz Sept. 4, 2025, 1:43 p.m. UTC | #1
Hi Trevor,

On 9/4/25 3:27 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> The device-tree organization of the nanopi-r4s device changed slightly in
> the linux-6.16.y release, therefore refresh the nanopi-r4s device-tree
> patch to match.
> 
> Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> ---
>   .../linux/linux-yocto/nanopi-r4s.scc          |  2 +-
>   ...opi-r4s.dtsi-enable-second-ethernet.patch} | 21 +++++++++----------
>   2 files changed, 11 insertions(+), 12 deletions(-)
>   rename recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/{0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch => 0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch} (56%)
> 
> diff --git a/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc b/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
> index 266b77c63fe5..3d1601e28d13 100644
> --- a/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
> +++ b/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
> @@ -3,4 +3,4 @@ define KMACHINE nanopi-r4s
>   define KTYPE standard
>   define KARCH arm64
>   
> -include bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
> +include bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> diff --git a/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch b/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> similarity index 56%
> rename from recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
> rename to recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> index e1f2b92665dd..a3dc23afae7e 100644
> --- a/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
> +++ b/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> @@ -1,9 +1,8 @@
> -From 4c867907b68b417da0711cabdde30a969e368596 Mon Sep 17 00:00:00 2001
> +From de2c28076a50e94659ea9693f9d0a307ef475132 Mon Sep 17 00:00:00 2001
>   From: Trevor Woerner <twoerner@gmail.com>
> -Date: Sun, 17 Sep 2023 17:30:43 -0400
> -Subject: [PATCH] rk3399-nanopi-r4s.dts: enable second ethernet
> +Date: Thu, 4 Sep 2025 09:11:33 -0400
> +Subject: [PATCH] rk3399-nanopi-r4s.dtsi: enable second ethernet
>   
> -# lspci

I think this isn't supposed to have disappeared from the commit log.

>   00:00.0 PCI bridge: Rockchip Electronics Co., Ltd RK3399 PCI Express Root Port
>   01:00.0 Ethernet controller: Realtek Semiconductor Co., Ltd. RTL8111/8168/8411 PCI Express Gigabit Ether)
>   
> @@ -11,14 +10,14 @@ Link: https://github.com/armbian/build/pull/2415
>   Upstream-Status: Submitted [by someone else]

Where was it submitted? I cannot see it on linux-rockchip mailing list 
archive the little I searched for it?

It'd be nice to have this upstreamed so you don't have to carry it around?

It does seem odd that every address and range is 0 though...

The change does look reasonable though :)

Cheers,
Quentin
Trevor Woerner Sept. 4, 2025, 4:58 p.m. UTC | #2
On Thu 2025-09-04 @ 03:43:45 PM, Quentin Schulz via lists.yoctoproject.org wrote:
> Hi Trevor,
> 
> On 9/4/25 3:27 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> > The device-tree organization of the nanopi-r4s device changed slightly in
> > the linux-6.16.y release, therefore refresh the nanopi-r4s device-tree
> > patch to match.
> > 
> > Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> > ---
> >   .../linux/linux-yocto/nanopi-r4s.scc          |  2 +-
> >   ...opi-r4s.dtsi-enable-second-ethernet.patch} | 21 +++++++++----------
> >   2 files changed, 11 insertions(+), 12 deletions(-)
> >   rename recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/{0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch => 0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch} (56%)
> > 
> > diff --git a/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc b/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
> > index 266b77c63fe5..3d1601e28d13 100644
> > --- a/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
> > +++ b/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
> > @@ -3,4 +3,4 @@ define KMACHINE nanopi-r4s
> >   define KTYPE standard
> >   define KARCH arm64
> > -include bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
> > +include bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> > diff --git a/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch b/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> > similarity index 56%
> > rename from recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
> > rename to recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> > index e1f2b92665dd..a3dc23afae7e 100644
> > --- a/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
> > +++ b/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
> > @@ -1,9 +1,8 @@
> > -From 4c867907b68b417da0711cabdde30a969e368596 Mon Sep 17 00:00:00 2001
> > +From de2c28076a50e94659ea9693f9d0a307ef475132 Mon Sep 17 00:00:00 2001
> >   From: Trevor Woerner <twoerner@gmail.com>
> > -Date: Sun, 17 Sep 2023 17:30:43 -0400
> > -Subject: [PATCH] rk3399-nanopi-r4s.dts: enable second ethernet
> > +Date: Thu, 4 Sep 2025 09:11:33 -0400
> > +Subject: [PATCH] rk3399-nanopi-r4s.dtsi: enable second ethernet
> > -# lspci
> 
> I think this isn't supposed to have disappeared from the commit log.
> 
> >   00:00.0 PCI bridge: Rockchip Electronics Co., Ltd RK3399 PCI Express Root Port
> >   01:00.0 Ethernet controller: Realtek Semiconductor Co., Ltd. RTL8111/8168/8411 PCI Express Gigabit Ether)
> > @@ -11,14 +10,14 @@ Link: https://github.com/armbian/build/pull/2415
> >   Upstream-Status: Submitted [by someone else]
> 
> Where was it submitted? I cannot see it on linux-rockchip mailing list
> archive the little I searched for it?
> 
> It'd be nice to have this upstreamed so you don't have to carry it around?
> 
> It does seem odd that every address and range is 0 though...

Yea, it's a kludge of some sort.

> The change does look reasonable though :)

I just tried this "trick" on real hardware using the latest 6.16.2 linux-yocto
kernel and it does not seem to work anymore. That's unfortunate. So I'll send
a new patch to remove this kludge for now.
diff mbox series

Patch

diff --git a/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc b/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
index 266b77c63fe5..3d1601e28d13 100644
--- a/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
+++ b/recipes-kernel/linux/linux-yocto/nanopi-r4s.scc
@@ -3,4 +3,4 @@  define KMACHINE nanopi-r4s
 define KTYPE standard
 define KARCH arm64
 
-include bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
+include bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
diff --git a/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch b/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
similarity index 56%
rename from recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
rename to recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
index e1f2b92665dd..a3dc23afae7e 100644
--- a/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dts-enable-second-ethernet.patch
+++ b/recipes-kernel/linux/linux-yocto/rockchip-kmeta/bsp/rockchip/0001-rk3399-nanopi-r4s.dtsi-enable-second-ethernet.patch
@@ -1,9 +1,8 @@ 
-From 4c867907b68b417da0711cabdde30a969e368596 Mon Sep 17 00:00:00 2001
+From de2c28076a50e94659ea9693f9d0a307ef475132 Mon Sep 17 00:00:00 2001
 From: Trevor Woerner <twoerner@gmail.com>
-Date: Sun, 17 Sep 2023 17:30:43 -0400
-Subject: [PATCH] rk3399-nanopi-r4s.dts: enable second ethernet
+Date: Thu, 4 Sep 2025 09:11:33 -0400
+Subject: [PATCH] rk3399-nanopi-r4s.dtsi: enable second ethernet
 
-# lspci
 00:00.0 PCI bridge: Rockchip Electronics Co., Ltd RK3399 PCI Express Root Port
 01:00.0 Ethernet controller: Realtek Semiconductor Co., Ltd. RTL8111/8168/8411 PCI Express Gigabit Ether)
 
@@ -11,14 +10,14 @@  Link: https://github.com/armbian/build/pull/2415
 Upstream-Status: Submitted [by someone else]
 Signed-off-by: Trevor Woerner <twoerner@gmail.com>
 ---
- arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts | 11 +++++++++++
+ arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dtsi | 11 +++++++++++
  1 file changed, 11 insertions(+)
 
-diff --git a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
-index fe5b52610010..771615722358 100644
---- a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
-+++ b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
-@@ -76,6 +76,17 @@ &pcie0 {
+diff --git a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dtsi
+index 8d94d9f91a5c..d6dca18e772c 100644
+--- a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dtsi
++++ b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dtsi
+@@ -74,6 +74,17 @@ &pcie0 {
  	max-link-speed = <1>;
  	num-lanes = <1>;
  	vpcie3v3-supply = <&vcc3v3_sys>;
@@ -37,5 +36,5 @@  index fe5b52610010..771615722358 100644
  
  &pinctrl {
 -- 
-2.41.0.327.gaa9166bcc0ba
+2.44.0.501.g19981daefd7c