[U-Boot] should denx wiki page on bootcount mention "failbootcmd"?

[Robert P. J. Day]

  it would seem that this wiki page:

    http://www.denx.de/wiki/view/DULG/UBootBootCountLimit

could use a fair bit of updating, since it concludes with:

"At the moment, the Boot Count Limit feature is available only for
MPC8xx, MPC82xx and MPC5200 Power Architecture® processors."

which i suspect was written when it referred only to *hardware*
support, since saving bootcount in the environment would seem to be
available to pretty much everyone at this point.

  in addition, i notice that if you've configured POST support,
there's also this from common/autobooot.c, the function
bootdelay_process():

  ...
  #ifdef CONFIG_POST
        if (gd->flags & GD_FLG_POSTFAIL) {
                s = getenv("failbootcmd");
        } else
  #endif /* CONFIG_POST */
  #ifdef CONFIG_BOOTCOUNT_LIMIT
        if (bootlimit && (bootcount > bootlimit)) {
                printf("Warning: Bootlimit (%u) exceeded. Using altbootcmd.\n",
                       (unsigned)bootlimit);
                s = getenv("altbootcmd");
        } else
  #endif /* CONFIG_BOOTCOUNT_LIMIT */
                s = getenv("bootcmd");
  ...

which suggests that if you've configured POST support, then that will
override the standard bootcount limit use of "altbootcmd" with the use
of "failbootcmd", correct?

  is that worth mentioning? or is anyone actually taking advantage of
that?

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================