[coreboot-gerrit] Change in coreboot[master]: mb/facebook/fbg1701/devicetree.cb: Use 64MB framebuffer size

[Frans Hendriks (Code Review)]

Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35064 )

Change subject: mb/facebook/fbg1701/devicetree.cb: Use 64MB framebuffer size
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/35064/1/src/mainboard/facebook/fbg1... 
File src/mainboard/facebook/fbg1701/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/35064/1/src/mainboard/facebook/fbg1... 
PS1, Line 12: 	register "PcdIgdDvmt50PreAlloc" = "2"
> Update to IGD_MEMSIZE_64MB

Done

https://review.coreboot.org/c/coreboot/+/35064/2/src/soc/intel/braswell/incl... 
File src/soc/intel/braswell/include/soc/gfx.h:

https://review.coreboot.org/c/coreboot/+/35064/2/src/soc/intel/braswell/incl... 
PS2, Line 20: #define IGD_MEMSIZE_32MB	0x01
            : #define IGD_MEMSIZE_64MB	0x02
            : #define IGD_MEMSIZE_96MB	0x03
            : #define IGD_MEMSIZE_128MB	0x04
> maybe add a comment that those values are for the configuration in the devicetree? or should these r […]

Will move the SoC modification to separate patch set.