[PATCH 3/3] ARM: dts: Add ixp4xx Actiontec MI424WR device trees

[Linus Walleij]

On Thu, Aug 14, 2025 at 11:13 PM Florian Fainelli
<florian.fainelli at broadcom.com> wrote:
> On 8/14/25 09:40, Linus Walleij wrote:
> > The Actiontex MI424WR is a pretty widespread home router, made
> > in many different revisions.
> >
> > Revisions A, C and D are based on IXP42x. We add a device tree
> > for the A/C variant and one for the D variant as these differ in
> > the location of the WAN PHY on the MDIO bus, and the ethernet
> > interfaces for the WAN PHY and the DSA switch are switched around.
> >
> > Signed-off-by: Linus Walleij <linus.walleij at linaro.org>
> > ---
>
> [snip]
>
> > +                     gpio1: gpio at 1,0 {
> > +                             /* MMIO GPIO at CS1 */
> > +                             compatible = "brcm,bcm6345-gpio";
>
> Is that really the compatible string we want to use here?

Yes as a matter of fact ... I got slapped for not just using compatible strings
that are ... compatible and piling up new ones :/

Maybe I should try adding a new one, I suppose just "basic-mmio"
would not be an acceptable cover-all compatible string?

I guess you're right, I should poke the DT binding people as to what
to do about this.

Yours,
Linus Walleij