[PATCH] net: mv88e6xxx: add missing SMI address init
YOUR NAME
username at gmail.com
Fri Nov 24 14:04:05 CET 2023
On Wed, Nov 22, 2023 at 04:38:41PM +0100, Lukasz Majewski wrote:
> Hi Marek,
>
> > Driver does not currently initialize the smi_addr field, but instead
> > keeps the default value. This leads to issues on systems with
> > different hardware configuration. We can fix this problem by reading
> > the SMI address from device tree.
> >
> > Signed-off-by: Marek Mojík <marek.mojik at nic.cz>
> > ---
> > drivers/net/mv88e6xxx.c | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/drivers/net/mv88e6xxx.c b/drivers/net/mv88e6xxx.c
> > index c073f81e72..4636dbf156 100644
> > --- a/drivers/net/mv88e6xxx.c
> > +++ b/drivers/net/mv88e6xxx.c
> > @@ -753,6 +753,12 @@ static int mv88e6xxx_probe(struct udevice *dev)
> > return -ENODEV;
> > }
> >
> > + priv->smi_addr = dev_read_addr(dev);
> > + if (priv->smi_addr == FDT_ADDR_T_NONE) {
> > + dev_err(dev, "Invalid or missing SMI address\n");
> > + return -EINVAL;
> > + }
> > +
> > /* probe internal mdio bus */
> > ret = mv88e6xxx_probe_mdio(dev);
> > if (ret)
>
> I've posted some time ago patches for this driver:
>
> https://patchwork.ozlabs.org/project/uboot/cover/20230601100005.2216345-1-lukma@denx.de/
>
> Unfortunately, those were not pulled...
>
> Maybe you would find fix for your issue, or prepare new version of it?
I looked at your patches, but they don't fix the issue with smi_addr
initialization. But i may try preparing a new version for them in the
future.
Regards,
Marek
More information about the U-Boot
mailing list