[coreboot-gerrit] Change in coreboot[master]: treewide: Drop unused Kconfig symbols

[Elyes Haouas (Code Review)]

Attention is currently required from: Lance Zhao, Tim Wawrzynczak, Patrick Rudolph, Felix Held.
Elyes Haouas has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62362 )

Change subject: treewide: Drop unused Kconfig symbols
......................................................................

Patch Set 4:

(1 comment)

File src/drivers/intel/gma/Kconfig:

https://review.coreboot.org/c/coreboot/+/62362/comment/377a4d3b_c892edc4 
PS2, Line 104: 
             : config INTEL_GMA_OPREGION_2_0
             : 	bool
             : 	default n if INTEL_GMA_OPREGION_2_1
             : 	default y
> Sure, but look at the way these two Kconfigs are structured; OPREGION_2_0 is the default, unless INT […]

what I see is "INTEL_GMA_OPREGION_2_1" set to no by default here : 
src/drivers/intel/gma/Kconfig:

"config INTEL_GMA_OPREGION_2_1
        bool
        default n"

meaning the function 
static struct opregion_version opregion_get_version(void)
{
    if (CONFIG(INTEL_GMA_OPREGION_2_1))
    	return (struct opregion_version) { .major = 2, .minor = 1 };
    return (struct opregion_version) { .major = 2, .minor = 0 };
}

returns "(struct opregion_version) { .major = 2, .minor = 0 }" unless INTEL_GMA_OPREGION_2_1 is selected