Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46842 )
Change subject: soc/intel/jasperlake: Correct GPIO pad sequence for community pad group ......................................................................
Patch Set 7:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46842/6/src/soc/intel/jasperlake/gp... File src/soc/intel/jasperlake/gpio.c:
https://review.coreboot.org/c/coreboot/+/46842/6/src/soc/intel/jasperlake/gp... PS6, Line 37: INTEL_GPP(GPP_F0, GPIO_RSVD_0, GPIO_RSVD_8),
Ack […]
Can you please raise a partner issue as well for this?
https://review.coreboot.org/c/coreboot/+/46842/7/src/soc/intel/jasperlake/in... File src/soc/intel/jasperlake/include/soc/gpio_soc_defs.h:
https://review.coreboot.org/c/coreboot/+/46842/7/src/soc/intel/jasperlake/in... PS7, Line 306: #define GPIO_RSVD_24 240 : #define GPIO_RSVD_25 241 : #define GPIO_RSVD_26 242 : #define GPIO_RSVD_27 243 : #define GPIO_RSVD_28 244 : #define GPIO_RSVD_29 245 : #define GPIO_RSVD_30 246 : #define GPIO_RSVD_31 247 : #define GPIO_RSVD_32 248 : #define GPIO_RSVD_33 249 : #define GPIO_RSVD_34 250 : #define GPIO_RSVD_35 251 : #define GPIO_RSVD_36 252 Why did these get dropped?