[U-Boot] [PATCH v6 4/5] Put common autoload code into auto_load() function

Wolfgang Denk wd at denx.de
Thu May 5 21:48:07 CEST 2011


Dear Mike Frysinger,

In message <BANLkTinaPhYEY48wsKJ4PEZY6sXwdcrLwA at mail.gmail.com> you wrote:
> On Thu, May 5, 2011 at 12:52, Simon Glass wrote:
> > +static void auto_load(void)
> > +{
> > +       char *s = getenv("autoload");
>
> const char *s

Agreed.

> > +       if (s != NULL) {
> > +               if (*s == 'n') {
> > +                       /*
> > +                        * Just use BOOTP to configure system;
> > +                        * Do not use TFTP to load the bootfile.
> > +                        */
> > +                       NetState = NETLOOP_SUCCES> S;
> > +                       return;
> > +#if defined(CONFIG_CMD_NFS)
> > +               } else if (strcmp(s, "NFS") == 0) {
> > +                       /*
> > +                        * Use NFS to load the bo> otfile.
> > +                        */
> > +                       NfsStart();
> > +                       return;
> > +#endif
> > +               }
> > +       }
> > +
> > +       TftpStart();
>
> put the TftpStart into a new else, then there's no need for the inline
> "return" ...

I disagree here.  As is, we save one level of nesting, which is
always a good thing.

On contrary, the "else" in the CONFIG_CMD_NFS should be dropped as
well, i. e. let's make this:


+       if (s != NULL) {
+               if (*s == 'n') {
+                       /*
+                        * Just use BOOTP to configure system;
+                        * Do not use TFTP to load the bootfile.
+                        */
+                       NetState = NETLOOP_SUCCES> S;
+                       return;
+               }
+#if defined(CONFIG_CMD_NFS)
+               if (strcmp(s, "NFS") == 0) {
+                       /*
+                        * Use NFS to load the bootfile.
+                        */
+                       NfsStart();
+                       return;
		}
+#endif
+       }
+
+       TftpStart();

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"We learn from history that we learn nothing from history."
- George Bernard Shaw


More information about the U-Boot mailing list