[coreboot-gerrit] Change in coreboot[master]: drivers/intel/gma: Restructure Opregion version info code

[Subrata Banik (Code Review)]

Attention is currently required from: Furquan Shaikh, Maulik V Vaghela, Tim Wawrzynczak, Meera Ravindranath, Angel Pons, Patrick Rudolph.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56168 )

Change subject: drivers/intel/gma: Restructure Opregion version info code
......................................................................

Patch Set 5:

(3 comments)

File src/drivers/intel/gma/opregion.h:

https://review.coreboot.org/c/coreboot/+/56168/comment/abb53aa8_e5ac0852 
PS5, Line 21: 	
one tab less?

https://review.coreboot.org/c/coreboot/+/56168/comment/23c70603_512090b3 
PS5, Line 25: __packed
wondering if we need __packed here ? isn't it already 32-bit variable ?

File src/drivers/intel/gma/opregion.c:

https://review.coreboot.org/c/coreboot/+/56168/comment/c8ea1259_15844228 
PS5, Line 262: 0
how would you take care of INTEL_GMA_OPREGION_2_1 and INTEL_GMA_OPREGION_2_0, can we use those Kconfig rather hardcoding the value here ?