ARM: dts: bcm470x: set MACs where missing

[PATCH 0/5] ARM: dts: bcm470x: set MACs where missing

Posted by Rosen Penev 1 month, 1 week ago

I believe these predate the bcm,nvram driver, which makes this
possible. Downstream OpenWrt currently assigns these in userspace.

Rosen Penev (5):
  ARM: dts: bcm4709-linksys-ea9200: set WAN MAC
  ARM: dts: bcm47094-linksys-panamera: set WAN MAC
  ARM: dts: bcm4709-asus-rt-ac87u: specify switch
  ARM: dts: bcm4708-linksys-ea6300-v1: specify switch
  ARM: dts: bcm47094-phicomm-k3: specify switch

 .../broadcom/bcm4708-linksys-ea6300-v1.dts    | 45 +++++++++++++++++++
 .../dts/broadcom/bcm4709-asus-rt-ac87u.dts    | 37 +++++++++++++++
 .../dts/broadcom/bcm4709-linksys-ea9200.dts   |  7 +++
 .../broadcom/bcm47094-linksys-panamera.dts    |  7 +++
 .../boot/dts/broadcom/bcm47094-phicomm-k3.dts | 43 ++++++++++++++++++
 5 files changed, 139 insertions(+)

--
2.53.0

Re: [PATCH 0/5] ARM: dts: bcm470x: set MACs where missing

Posted by Rafał Miłecki 1 month, 1 week ago

pt., 20 lut 2026 o 03:14 Rosen Penev <rosenp@gmail.com> napisał(a):
> I believe these predate the bcm,nvram driver, which makes this
> possible. Downstream OpenWrt currently assigns these in userspace.

Changes look good, I just posted one comment about commit description.
Thanks for submitting this!

If you check
git log --oneline arch/arm/boot/dts/broadcom/bcm470*dts
git log --oneline -- arch/arm/boot/dts/bcm470*dts
the prefix always used for NS patches was:
"ARM: dts: BCM5301X: "

I guess it may be preferred / required to follow that.

-- 
Rafał

Re: [PATCH 0/5] ARM: dts: bcm470x: set MACs where missing

Posted by Rosen Penev 1 month, 1 week ago

On Fri, Feb 20, 2026 at 3:21 AM Rafał Miłecki <zajec5@gmail.com> wrote:
>
> pt., 20 lut 2026 o 03:14 Rosen Penev <rosenp@gmail.com> napisał(a):
> > I believe these predate the bcm,nvram driver, which makes this
> > possible. Downstream OpenWrt currently assigns these in userspace.
>
> Changes look good, I just posted one comment about commit description.
> Thanks for submitting this!
>
> If you check
> git log --oneline arch/arm/boot/dts/broadcom/bcm470*dts
> git log --oneline -- arch/arm/boot/dts/bcm470*dts
> the prefix always used for NS patches was:
> "ARM: dts: BCM5301X: "
>
> I guess it may be preferred / required to follow that.
I didn't want to make the subject line huge.

Actually in such a case, would it make sense to fold into one commit?
>
> --
> Rafał

Re: [PATCH 0/5] ARM: dts: bcm470x: set MACs where missing

Posted by Rafał Miłecki 1 month, 1 week ago

pt., 20 lut 2026 o 22:20 Rosen Penev <rosenp@gmail.com> napisał(a):
> On Fri, Feb 20, 2026 at 3:21 AM Rafał Miłecki <zajec5@gmail.com> wrote:
> >
> > pt., 20 lut 2026 o 03:14 Rosen Penev <rosenp@gmail.com> napisał(a):
> > > I believe these predate the bcm,nvram driver, which makes this
> > > possible. Downstream OpenWrt currently assigns these in userspace.
> >
> > Changes look good, I just posted one comment about commit description.
> > Thanks for submitting this!
> >
> > If you check
> > git log --oneline arch/arm/boot/dts/broadcom/bcm470*dts
> > git log --oneline -- arch/arm/boot/dts/bcm470*dts
> > the prefix always used for NS patches was:
> > "ARM: dts: BCM5301X: "
> >
> > I guess it may be preferred / required to follow that.
> I didn't want to make the subject line huge.
>
> Actually in such a case, would it make sense to fold into one commit?

I'll leave it to Florian to comment on. He's more in place to decide than me.

--
Rafał