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

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
participants (1)
-
Robert P. J. Day