Change in ...coreboot[master]: arch/x86/Kconfig: move MAX_REBOOT_CNT option

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29979 )

Change subject: arch/x86/Kconfig: move MAX_REBOOT_CNT option
......................................................................

Patch Set 2:

(1 comment)

Currently, you can't set wrong values. That's a feature I'd like to
have maintained. Everything below the currently defined defaults is
wrong.

In other words, this option was hidden on purpose. You can add another
option for instance and take the minimum of both or something like
that. But you mustn't lower it.

https://review.coreboot.org/#/c/29979/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/29979/1//COMMIT_MSG@11 
PS1, Line 11:
> It only adds the entry, so you don't have to change it in the config file by hand. […]

It's not about complete reboots. If you'd set it to 2 you
might get false fallbacks (because a single reboot can result
in coreboot resetting again, without anything failing).