[coreboot-gerrit] Change in coreboot[master]: drivers/intel/gma/edid: Use byte swap function

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38647 )

Change subject: drivers/intel/gma/edid: Use byte swap function
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38647/1/src/drivers/intel/gma/edid.... 
File src/drivers/intel/gma/edid.c:

https://review.coreboot.org/c/coreboot/+/38647/1/src/drivers/intel/gma/edid.... 
PS1, Line 96: edid
> Call me cursed, buuut... What if you do a pointer cast and use an u32* here? This would make the byteswap unnecessary.

If the type is char or void, then aliasing to a different type does NOT result in nasal demons (undefined behavior, after which anything could happen like demons coming out of everyone's noses). So, it's an option.