Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45201 )
Change subject: util/inteltool: add missing special function pads for CNL-LP ......................................................................
Patch Set 8:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45201/2/util/inteltool/gpio_names/c... File util/inteltool/gpio_names/cannonlake_lp.h:
https://review.coreboot.org/c/coreboot/+/45201/2/util/inteltool/gpio_names/c... PS2, Line 311: "vUART0_TXD", "mapped", "n/a", "n/a", : "vUART0_RXD", "mapped", "n/a", "n/a", : "vUART0_CTS_B", "mapped", "n/a", "n/a", : "vUART0_RTS_B", "mapped", "n/a", "n/a", : "vISH_UART0_TXD", "mapped", "n/a", "n/a", : "vISH_UART0_RXD", "mapped", "n/a", "n/a", : "vISH_UART0_CTS_B", "mapped", "n/a", "n/a", : "vISH_UART0_RTS_B", "mapped", "n/a", "n/a", : "vISH_UART1_TXD", "mapped", "n/a", "n/a", : "vISH_UART1_RXD", "mapped", "n/a", "n/a", : "vISH_UART1_CTS_B", "mapped", "n/a", "n/a", : "vISH_UART1_RTS_B", "mapped", "n/a", "n/a", : "vCNV_BT_I2S_BCLK", "SSP0", "SSP1", "SSP2", : "vCNV_BT_I2S_WS_SYNC", "SSP0", "SSP1", "SSP2", : "vCNV_BT_I2S_SDO", "SSP0", "SSP1", "SSP2", : "vCNV_BT_I2S_SDI", "SSP0", "SSP1", "SSP2", : "vSSP2_SCLK", "mapped", "n/a", "n/a", : "vSSP2_SFRM", "mapped", "n/a", "n/a", : "vSSP2_TXD", "mapped", "n/a", "n/a", : "vSSP2_RXD", "n/a", "n/a", "n/a", : "vCNV_GNSS_HOST_WAKEB", "n/a", "n/a", "n/a", : "vSD3_CD_B", "n/a", "n/a", "n/a",
this needs further investigation; it seems, they use vGPIOs as mux switches o. […]
will be done in a follow-up later. for dumping the register values it's not that important for now