[PATCH] ARM: mx28: Skip OCOTP FEC MAC setup if in DT
Marek Vasut
marex at denx.de
Tue Sep 25 07:27:51 EDT 2012
Dear Shawn Guo,
> On Tue, Sep 25, 2012 at 01:01:46PM +0200, Marek Vasut wrote:
> > > > np = of_find_compatible_node(from, NULL, "fsl,imx28-
fec");
> > > > if (!np)
> > > >
> > > > return;
> > > >
> > > > +
> > > > + if (of_get_property(np, "local-mac-address", NULL))
> > > > + continue;
> > > > +
> > >
> > > The block should be put after "from" assignment below.
> >
> > Can you please elaborate why would that be necessary?
>
> Without assigning 'np' to 'from', the next iteration of 'for' loop
> will still find the first fec node. Then what's point of 'continue'?
Ewww, you're right.
> Shawn
Best regards,
Marek Vasut
More information about the linux-arm-kernel
mailing list