[coreboot-gerrit] Change in coreboot[master]: mb/google/guybrush/var/guybrush: Add FPMCU configration

[Ivy Jian (Code Review)]

Attention is currently required from: Furquan Shaikh, Bhanu Prakash Maiya, EricR Lai.
Ivy Jian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52181 )

Change subject: mb/google/guybrush/var/guybrush: Add FPMCU configration
......................................................................

Patch Set 9:

(1 comment)

File src/mainboard/google/guybrush/variants/guybrush/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/52181/comment/92df2074_9e3828bc 
PS9, Line 113: register "has_power_resource" = "1"
> > I tried configure GPIO_32 to high and GPIO_11 to low in "early_gpio_table" then de-assert reset in […]

Are the signals GPIO_11(reset) and GPIO_32(enable) driven low at reset? 
=> GPIO_11 is default PU from PPR, GPIO_32 shows n/a.
GPIO_11 should not be driven low in early_gpio_table. Else it will break the case of S3 resume.
=> FP seems not working if not configuring GPIO_11(reset) to low in early_gpio_table