Re: [PATCH 0/2] net: dsa: b53: mdio: add support for BCM53134

From: Álvaro Fernández Rojas
Date: Thu Mar 23 2023 - 18:10:43 EST


El jue, 23 mar 2023 a las 20:58, Paul Geurts
(<paul.geurts@xxxxxxxxxxxxxxxxxxxxxxxxx>) escribió:
>
> > -----Original Message-----
> > From: Álvaro Fernández Rojas <noltari@xxxxxxxxx>
> > Sent: donderdag 23 maart 2023 13:18
> > To: Paul Geurts <paul.geurts@xxxxxxxxxxxxxxxxxxxxxxxxx>;
> > f.fainelli@xxxxxxxxx; jonas.gorski@xxxxxxxxx; andrew@xxxxxxx;
> > olteanv@xxxxxxxxx; davem@xxxxxxxxxxxxx; edumazet@xxxxxxxxxx;
> > kuba@xxxxxxxxxx; pabeni@xxxxxxxxxx; robh+dt@xxxxxxxxxx;
> > krzysztof.kozlowski+dt@xxxxxxxxxx; netdev@xxxxxxxxxxxxxxx;
> > devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> > Cc: Álvaro Fernández Rojas <noltari@xxxxxxxxx>
> > Subject: [PATCH 0/2] net: dsa: b53: mdio: add support for BCM53134
> >
> > This is based on the initial work from Paul Geurts that was sent to the
> > incorrect linux development lists and recipients.
> > I've modified it by removing BCM53134_DEVICE_ID from is531x5() and
> > therefore adding is53134() where needed.
> > I also added a separate RGMII handling block for is53134() since according to
> > Paul, BCM53134 doesn't support RGMII_CTRL_TIMING_SEL as opposed to
> > is531x5().
> >
> > Paul Geurts (1):
> > net: dsa: b53: mdio: add support for BCM53134
> >
> > Álvaro Fernández Rojas (1):
> > dt-bindings: net: dsa: b53: add BCM53134 support
> >
> > .../devicetree/bindings/net/dsa/brcm,b53.yaml | 1 +
> > drivers/net/dsa/b53/b53_common.c | 53 ++++++++++++++++++-
> > drivers/net/dsa/b53/b53_mdio.c | 5 +-
> > drivers/net/dsa/b53/b53_priv.h | 9 +++-
> > 4 files changed, 65 insertions(+), 3 deletions(-)
> >
> > --
> > 2.30.2
>
> Thank you for resending my patches! I didn't get to it yet. Any particular reason you didn't include the optional GPIO patch I had in my set?

I'm using it for a Sercomm H500-s which doesn't seem to need it.
However, I've just realized that it's documented here as GPIO 18:
https://openwrt.org/toh/sercomm/h500-s#other
Anyway, I think that the GPIO patch can be added later...

> ---
> Paul

--
Álvaro