Jacob Garber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34523 )
Change subject: soc/qualcomm/qcs405: Handle invalid QUP and BLSP ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/34523/2/src/soc/qualcomm/qcs405/clo... File src/soc/qualcomm/qcs405/clock.c:
https://review.coreboot.org/c/coreboot/+/34523/2/src/soc/qualcomm/qcs405/clo... PS2, Line 218: case 0: : spi_clk = (struct qcs405_clock *) : &gcc->blsp1_qup0_spi_clk; : break; : case 1: : spi_clk = (struct qcs405_clock *) : &gcc->blsp1_qup1_spi_clk; : break; : case 2: : spi_clk = (struct qcs405_clock *) : &gcc->blsp1_qup2_spi_clk; : break; : case 3: : spi_clk = (struct qcs405_clock *) : &gcc->blsp1_qup3_spi_clk; : break; : case 4: : spi_clk = (struct qcs405_clock *) : &gcc->blsp1_qup4_spi_clk;
Not on this patch, but I guess this could fit in one line with the new 96 character line length.
Ack
https://review.coreboot.org/c/coreboot/+/34523/2/src/soc/qualcomm/qcs405/clo... PS2, Line 239: pri
I guess "Invalid QUP %d\n" would be better
Done