Re: [PATCH] treewide: drop CONFIG_EMBEDDED

[Randy Dunlap]

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