Re: [PATCH] Debug: add Kconfig option to abort autoboot

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

Hi,

On 2/25/26 10:52 AM, Jonas Rebmann wrote:
> On 2026-02-25 08:54, Sascha Hauer wrote:
>> On Tue, Feb 24, 2026 at 04:59:00PM +0100, Jonas Rebmann wrote:
>>> Hi Sascha,
>>>
>>> On 2026-02-23 16:39, Sascha Hauer wrote:
>>>> For debugging purposes it's sometimes useful when barebox doesn't boot
>>>> through, but stops at the prompt.
>>>
>>> Isn't this already compile-time configurable by setting the defaultenv
>>> autoboot to "abort"?
>>
>> Yes, it is. It's just easier sometimes to just change a config, at least
>> for Claude.
> 
> I think for the human, managing one option in two systems in different
> ways makes things confusing. If the change is really needed, I suggest
> the help text could explain that this overrides global.autoboot and that
> configuration via defaultenv has further options for boot behavior.
> Maybe also the autoboot global help text could be extended: "Autoboot
> state. Possible values: countdown (default), abort, halt, menu, boot.
> Ineffective if overwritten by CONFIG_DEBUG_AUTOBOOT_ABORT."

Or alternatively, we add defaultenv/defaultenv-2-devel/nv/autoboot ==
abort and then have a config option that enables this environment overlay.

Cheers,
Ahmad

> 
> Regards,
> Jonas
> 

-- 
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 |