[coreboot-gerrit] Change in coreboot[master]: mb/intel/jasperlake_rvp: Configure GPIOs related to UFC

[Varshit B Pandya (Code Review)]

Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44416 )

Change subject: mb/intel/jasperlake_rvp: Configure GPIOs related to UFC
......................................................................

Patch Set 3:

(4 comments)

https://review.coreboot.org/c/coreboot/+/44416/2/src/mainboard/intel/jasperl... 
File src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c:

https://review.coreboot.org/c/coreboot/+/44416/2/src/mainboard/intel/jasperl... 
PS2, Line 83: 0
> shouldn't this be 1? or driver drivers it from PR0?

Drivers drives it

https://review.coreboot.org/c/coreboot/+/44416/2/src/mainboard/intel/jasperl... 
PS2, Line 89: PLTRST
> DEEP

Done

https://review.coreboot.org/c/coreboot/+/44416/2/src/mainboard/intel/jasperl... 
PS2, Line 92: PLTRST
> DEEP

Done

https://review.coreboot.org/c/coreboot/+/44416/2/src/mainboard/intel/jasperl... 
PS2, Line 113: PLTRST
> DEEP

Done