Re: [PATCH] treewide: drop CONFIG_EMBEDDED

[Randy Dunlap <rdunlap@infradead.org>]

Hi Jesse,

I replied to your comment a few days ago, but for some reason
your email to me contains:
Reply-To: 20230816055010.31534-1-rdunlap@infradead.org
so it wasn't sent directly to you.

My former reply is below.

On 8/16/23 20:15, Randy Dunlap wrote:
> Hi Jesse,
> 
> On 8/16/23 15:45, Jesse Taube wrote:
>> Hi, Randy
>>
>>> diff -- a/init/Kconfig b/init/Kconfig
>>> --- a/init/Kconfig
>>> +++ b/init/Kconfig
>>> @@ -1790,14 +1790,6 @@ config DEBUG_RSEQ
>>>
>>>         If unsure, say N.
>>>
>>> -config EMBEDDED
>>> -    bool "Embedded system"
>>> -    select EXPERT
>>> -    help
>>> -      This option should be enabled if compiling the kernel for
>>> -      an embedded system so certain expert options are available
>>> -      for configuration.
>>
>> Wouldn't removing this break many out of tree configs?
> 
> I'm not familiar with out-of-tree configs.
> Do you have some examples of some that use CONFIG_EMBEDDED?
> (not distros)
> 
>> Should there be a warning here to update change it instead of removal?
> 
> kconfig doesn't have a warning mechanism AFAIK.
> Do you have an idea of how this would work?
> 
> We could make a smaller change to init/Kconfig, like so:
> 
>  config EMBEDDED
> -	bool "Embedded system"
> +	bool "Embedded system (DEPRECATED)"
>  	select EXPERT
>  	help
> -	  This option should be enabled if compiling the kernel for
> -	  an embedded system so certain expert options are available
> -	  for configuration.
> +	  This option is being removed after Linux 6.6.
> +	  Use EXPERT instead of EMBEDDED.
> 
> but there is no way to produce a warning message. I.e., even with this
> change, the message will probably be overlooked.
> 
> ---
> ~Randy

-- 
~Randy