Re: [PATCH] kconfig: untangle EXPERT and EMBEDDED

From: Jerome Marchand
Date: Mon Jan 16 2012 - 10:33:49 EST


On 01/13/2012 10:27 PM, David Rientjes wrote:
> CONFIG_EXPERT does work, there haven't been problems reported with it in
> the year that it has been in the kernel, and CONFIG_EMBEDDED is available
> to be extended to have its logical semantics. Right now, CONFIG_EMBEDDED
> is pretty useless other than setting CONFIG_EXPERT but that could easily
> be extended if someone wants to spend the time to separate options out
> from CONFIG_EXPERT that don't make any sense for embedded devices. I'm
> not from the embedded world, so I don't know the problems they face when
> configuring the kernel.

I don't see what your objections to this patch are. It actually does not
change anything to CONFIG_EMBEDDED and 6a108a14fa35 didn't either.
CONFIG_EMBEDDED still exposes the same options and set the same default
values as it did before or after 6a108a14fa35 was applied.

This patch just makes EXPERT exposes config option as advertised by its
description without changing any default value.
For instance, Why would CONFIG_EXPERT disable by default some HID devices?
I could understand why it is done for CONFIG_EMBEDDED, but certainly not
for an general EXPERT option.

>
>> This patch is actually attempting to make progress in the separation of
>> the semantics for EXPERT and the old, insane EMBEDDED, which had the very
>> loose semantics. That's why it's called "untangle EXPERT and EMBEDDED".
>>

If EMBEDDED was insane before 6a108a14fa35, then it still was after, since
nothing changed. EMBEDDED was just renamed EXPERT, and the "new" EMBEDDED
option just selects EXPERT.

>
> Breaking backwards compatibility for users who aren't defconfigs is a
> non-starter, as I've said. Admitting that your patch does it is almost
> like nacking your own patch.

If people use CONFIG_EXPERT while expecting it to change some default value
useful for embedded device, while at all time there was a CONFIG_EMBEDDED
available that is according to its description made for this purpose, then
the shame on them...
If CONFIG_EMBEDDED is broken, it has to be fixed. As Andrew already said,
this is out of the scope of this patch.

Jerome

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/