22 comments:
Patch Set #3, Line 7: Northbridge
s/Northbridge/PCH/ ? if any bridge, it would be a southbridge.
Done
File util/inteltool/gpio_groups.c:
VDD2
Done
Patch Set #3, Line 1209: "GPP_A23", "ISH_GP5", "n/a", "n/a", "n/a",
missing GPIO_RSVD_0 (coreboot notion, see soc/intel/cannonlake/include/soc/gpio_soc_defs_cnp_h. […]
Done
When all function 4 values are "n/a" you can leave it out and […]
Done
Patch Set #3, Line 1222: VRALRERT
VRALERT#
Done
Patch Set #3, Line 1243: "GPP_B23", "SML1ALERT#", "PCHHOT#", "n/a", "n/a",
missing GPIO_RSVD_1, GPIO_RSVD_2 […]
Done
`func_count = 3` ?
Done
SPI1
Done
Patch Set #3, Line 1292: I2C2SDA
I2C2_SDA
Done
Patch Set #3, Line 1294: modem
MODEM
Done
GSPI2
Done
Patch Set #3, Line 1334: "GPP_E12", "USB2_OC3#", "n/a", "n/a", "n/a",
`func_count = 3` ?
Done
Patch Set #3, Line 1368: "GPP_F23", "DDPF_CTRLDATA", "n/a", "n/a", "n/a",
`func_count = 3` ?
Done
drop empty line here
Done
Patch Set #3, Line 1383: DATRA2
DATA2
Done
Patch Set #3, Line 1387: "GPP_G7", "SD_WP", "n/a", "n/a", "n/a",
coreboot treats these as a separate group (AZA) after G: GPIO_RSVD_3..10 […]
Done
`func_count = 2`?
Done
Patch Set #3, Line 1421: "GPP_H23", "TIME_SYNC0", "n/a", "n/a", "n/a",
`func_count = 2` ?
Done
Patch Set #3, Line 1446: "GPP_I14", "M2_SKT2_CFG3", "n/a", "n/a", "n/a",
`func_count = 3` ? […]
Done
Patch Set #3, Line 1468: "GPP_J11", "A4WP_PRESENT", "n/a", "n/a", "n/a",
`func_count = 3` ?
Done
Patch Set #3, Line 1502: "GPP_K23", "IMGCLKOUT1", "n/a", "n/a", "n/a",
`func_count = 2` ?
Done
Patch Set #3, Line 1524: "GPD11", "LANPHYPC", "n/a", "n/a", "n/a",
`func_count = 2` ?
Done
To view, visit change 31500. To unsubscribe, or for help writing mail filters, visit settings.