build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30934 )
Change subject: qualcomm/qcs405: enable SPI bus 4 for TPM ......................................................................
Patch Set 11:
(15 comments)
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c File src/soc/qualcomm/qcs405/spi.c:
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@354 PS11, Line 354: gpio_configure(GPIO(37), 2, GPIO_PULL_DOWN, GPIO_6MA, GPIO_INPUT); // MOSI line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@355 PS11, Line 355: gpio_configure(GPIO(38), 2, GPIO_PULL_DOWN, GPIO_6MA, GPIO_OUTPUT); // MISO line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@356 PS11, Line 356: gpio_configure(GPIO(117), 2, GPIO_NO_PULL, GPIO_6MA, GPIO_OUTPUT);// CS line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@357 PS11, Line 357: gpio_configure(GPIO(118), 2, GPIO_PULL_DOWN, GPIO_6MA, GPIO_OUTPUT);// CLK line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@360 PS11, Line 360: gpio_configure(GPIO(26), 3, GPIO_NO_PULL, GPIO_16MA, GPIO_INPUT); // MOSI line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@361 PS11, Line 361: gpio_configure(GPIO(27), 3, GPIO_NO_PULL, GPIO_16MA, GPIO_INPUT); // MISO line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@362 PS11, Line 362: gpio_configure(GPIO(28), 4, GPIO_PULL_UP, GPIO_16MA, GPIO_INPUT); // CS line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@363 PS11, Line 363: gpio_configure(GPIO(29), 4, GPIO_NO_PULL, GPIO_16MA, GPIO_INPUT); // CLK line over 80 characters
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@514 PS11, Line 514: printk(BIOS_ERR, "SPI FIFO read timeout \n"); unnecessary whitespace before a quoted newline
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@518 PS11, Line 518: }while(!(val & MAX_INPUT_DONE_FLAG)); space required after that close brace '}'
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@518 PS11, Line 518: }while(!(val & MAX_INPUT_DONE_FLAG)); space required before the open parenthesis '('
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@640 PS11, Line 640: printk(BIOS_ERR, "SPI FIFO write timeout \n"); unnecessary whitespace before a quoted newline
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@644 PS11, Line 644: }while(!(val & MAX_OUTPUT_DONE_FLAG)); space required after that close brace '}'
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@644 PS11, Line 644: }while(!(val & MAX_OUTPUT_DONE_FLAG)); space required before the open parenthesis '('
https://review.coreboot.org/#/c/30934/11/src/soc/qualcomm/qcs405/spi.c@666 PS11, Line 666: printk(BIOS_ERR, "SPI:DBG write not success \n"); unnecessary whitespace before a quoted newline