On Thu, May 5, 2011 at 12:29 PM, Mike Frysinger <[email protected]> wrote:

> On Thu, May 5, 2011 at 12:52, Simon Glass wrote:
> > +static void auto_load(void)
> > +{
> > +       char *s = getenv("autoload");
>
> const char *s
>
> > +       if (s != NULL) {
> > +               if (*s == 'n') {
> > +                       /*
> > +                        * Just use BOOTP to configure system;
> > +                        * Do not use TFTP to load the bootfile.
> > +                        */
> > +                       NetState = NETLOOP_SUCCESS;
> > +                       return;
> > +#if defined(CONFIG_CMD_NFS)
> > +               } else if (strcmp(s, "NFS") == 0) {
> > +                       /*
> > +                        * Use NFS to load the bootfile.
> > +                        */
> > +                       NfsStart();
> > +                       return;
> > +#endif
> > +               }
> > +       }
> > +
> > +       TftpStart();
>
> put the TftpStart into a new else, then there's no need for the inline
> "return" ...
> -mike
>

Will do.
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to