[PATCH v2 3/4] Documentation: bootchooser: add information about attempts_locked
Ahmad Fatoum
a.fatoum at pengutronix.de
Mon Jun 16 08:34:52 PDT 2025
On 6/16/25 17:06, Lars Schmidt wrote:
> The new attempts_locked variable does have influence on
> remaining_attempts counter.
This commit message feels incomplete.
>
> While at it, also make mention in the bootstate.dtsi.
>
> Signed-off-by: Lars Schmidt <l.schmidt at pengutronix.de>
The diff looks fine though:
Reviewed-by: Ahmad Fatoum <a.fatoum at pengutronix.de>
> ---
> Documentation/user/bootchooser.rst | 23 +++++++++++++++++++++++
> arch/arm/dts/bootstate.dtsi | 5 +++++
> 2 files changed, 28 insertions(+)
>
> diff --git a/Documentation/user/bootchooser.rst b/Documentation/user/bootchooser.rst
> index 351e1d14e1ead6ba8d329c03c0bc7ed28b523df0..ab0b6c3fc895226f5aa4590944a2cd675a56d985 100644
> --- a/Documentation/user/bootchooser.rst
> +++ b/Documentation/user/bootchooser.rst
> @@ -77,6 +77,16 @@ no remaining attempts left.
> To prevent ending up in an unbootable system after a number of failed boot
> attempts, there is also a built-in mechanism to reset the counters to their defaults,
> controlled by the ``global.bootchooser.reset_attempts`` variable.
> +Alternatively, counting down the remaining attempts can be disabled by
> +locking bootchooser boot attempts.
> +This is done by defining a (32-bit) ``attempts_locked`` variable in the
> +bootstate and setting its value to ``1`` (usually from userspace).
> +
> +The variable affects all targets, is optional and its absence is
> +interpreted as ``0``, meaning that attempts are decremented normally.
> +
> +The ``attempts_locked`` value does not influence the decision on which target
> +to boot if any, only whether to decrement the attempts when booting.
>
> If ``global.bootchooser.retry`` is enabled (set to ``1``), the bootchooser
> algorithm will iterate through all valid boot targets (and decrease their
> @@ -107,6 +117,19 @@ on the :ref:`reset reason <reset_reason>` (i.e. != WDG) using the
> This will reset the ``remaining_attempts`` counter of the *last chosen* slot to
> its default value (``reset_attempts``).
>
> +Another option is to use ``attempts_locked``. Normally this should be controlled from
> +Linux userspace using the *barebox-state* tool, i.e.::
> +
> + barebox-state -s bootstate.attempts_locked=1
> +
> +It can also be locked via the :ref:`bootchooser command <command_bootchooser>`::
> +
> + bootchooser -l
> +
> +or unlocked::
> +
> + bootchooser -L
> +
>
> .. _dt-utils: https://git.pengutronix.de/cgit/tools/dt-utils
>
> diff --git a/arch/arm/dts/bootstate.dtsi b/arch/arm/dts/bootstate.dtsi
> index aa767d4e3b966244b01702bbc2bdcd5c95ef7009..733441fb2697b283dc784b00b012e11ad671bb15 100644
> --- a/arch/arm/dts/bootstate.dtsi
> +++ b/arch/arm/dts/bootstate.dtsi
> @@ -42,4 +42,9 @@ last_chosen at 10 {
> reg = <0x10 0x4>;
> type = "uint32";
> };
> +
> + attempts_locked at 14 {
> + reg = <0x14 0x4>;
> + type = "uint32";
> + };
> };
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list