[coreboot-gerrit] Change in ...coreboot[master]: drivers/apple: Add hybrid graphics driver

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32719 )

Change subject: drivers/apple: Add hybrid graphics driver
......................................................................

Patch Set 8:

(2 comments)

https://review.coreboot.org/#/c/32719/7/src/drivers/apple/hybrid_graphics/hy... 
File src/drivers/apple/hybrid_graphics/hybrid_graphics.c:

https://review.coreboot.org/#/c/32719/7/src/drivers/apple/hybrid_graphics/hy... 
PS7, Line 35: 	config = dev->chip_info;
> but it is used just the line below

just check for dev->chip_info directly

https://review.coreboot.org/#/c/32719/7/src/drivers/apple/hybrid_graphics/hy... 
PS7, Line 37: 		printk(BIOS_INFO, "Hybrid graphics: Not installed\n");
that should probably BIOS_ERR, as this is a miss configuration