On Sat, Aug 3, 2019 at 4:57 AM <[email protected]> wrote: > > From: Nevo Hed <[email protected]> > > phy_dev ptr is set from return of phy_connect() and is used before > test to see if NULL. Obviously since the test already sxists someoen Spelling. > made the determination that this NULL is possible. > > Signed-off-by: Nevo Hed <[email protected]> > --- > drivers/net/mvpp2.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/mvpp2.c b/drivers/net/mvpp2.c > index fae7090121..5f908113f2 100644 > --- a/drivers/net/mvpp2.c > +++ b/drivers/net/mvpp2.c > @@ -4494,7 +4494,8 @@ static void mvpp2_phy_connect(struct udevice *dev, > struct mvpp2_port *port) > * an option because it is required for the phy_fw_down > * procedure. > */ > - if (phy_dev->drv->uid == 0xffffffff) {/* Generic phy */ > + if (phy_dev && > + phy_dev->drv->uid == 0xffffffff) {/* Generic phy */ > netdev_warn(port->dev, > "Marking phy as invalid, link will not be > checked\n"); > /* set phy_addr to invalid value */ > -- > 2.21.0 > > _______________________________________________ > U-Boot mailing list > [email protected] > https://lists.denx.de/listinfo/u-boot
Reviewed-By: Ramon Fried <[email protected]> _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
