Re: [PATCH] Convert CONFIG_SKIP_LOWLEVEL_INIT et al to Kconfig

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1316 bytes --]

On Fri, Apr 08, 2022 at 10:58:28AM +0200, Heinrich Schuchardt wrote:
> 
> 
> On 8/28/21 03:18, Tom Rini wrote:
> > This converts the following to Kconfig:
> >     CONFIG_SKIP_LOWLEVEL_INIT
> >     CONFIG_SKIP_LOWLEVEL_INIT_ONLY
> > 
> > In order to do this, we need to introduce SPL and TPL variants of these
> > options so that we can clearly disable these options only in SPL in some
> > cases, and both instances in other cases.
> > 
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> 
> 
> Since this patch series 'make menuconfig' does not look good anymore:
> 
>     *** Compiler: gcc (Ubuntu 11.2.0-18ubuntu1) 11.2.0 ***
>     Architecture select (ARM architecture)  --->
> [ ] Skip the calls to certain low level initialization functions
> [ ] Skip the calls to certain low level initialization functions
> [ ] Skip the calls to certain low level initialization functions
> [ ] Skip the call to lowlevel_init during early boot ONLY
> [ ] Skip the call to lowlevel_init during early boot ONLY
>     ARM architecture  --->
>     General setup  --->
> 
> Who would be able to understand three entries with the same short text?
> 
> Why wouldn't these entries live in "General setup" or some other sub-menu?

Please send patches to change some of the wording, thanks.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]