[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Add i915_gpu_controller_info

[Tim Crawford (Code Review)]

Tim Crawford has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43616 )

Change subject: soc/intel/cannonlake: Add i915_gpu_controller_info
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43616/1/src/soc/intel/cannonlake/ch... 
File src/soc/intel/cannonlake/chip.h:

https://review.coreboot.org/c/coreboot/+/43616/1/src/soc/intel/cannonlake/ch... 
PS1, Line 483: backlight control
> It's mostly about the available display ports and only indirectly related to […]

Should the comment just be dropped? Skylake is the only other chip that documents this field (with this comment).