GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd/common/block/spi/fch_spi_ctrl: improve printk messages 

Replace FCH_SC with FCH SPI in the printk messages to make those a bit
clearer and also remove an unneeded line break in another printk call.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I6ff02163e6a48a2cc8b7fe89b15826e154715d29
Reviewed-on: https://review.coreboot.org/c/coreboot/+/60120
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
This commit is contained in:
Felix Held 2021-12-10 18:38:16 +01:00
parent 1105fe8913
commit 856d6bc6d3
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/soc/amd/common/block/spi/fch_spi_ctrl.c
View File

@ -98,7 +98,7 @@ static int execute_command(void)
spi_write8(SPI_CMD_TRIGGER, SPI_CMD_TRIGGER_EXECUTE);
if (wait_for_ready()) {
printk(BIOS_ERR, "FCH_SC Error: Timeout executing command\n");
printk(BIOS_ERR, "FCH SPI Error: Timeout executing command\n");
return -1;
}
@ -121,8 +121,7 @@ static int spi_ctrlr_xfer(const struct spi_slave *slave, const void *dout,
const uint8_t *bufout = dout;
if (CONFIG(SOC_AMD_COMMON_BLOCK_SPI_DEBUG))
printk(BIOS_DEBUG, "%s(%zx, %zx)\n", __func__, bytesout,
bytesin);
printk(BIOS_DEBUG, "%s(%zx, %zx)\n", __func__, bytesout, bytesin);
/* First byte is cmd which cannot be sent through FIFO */
cmd = bufout[0];
@ -136,7 +135,7 @@ static int spi_ctrlr_xfer(const struct spi_slave *slave, const void *dout,
* and followed by other SPI commands.
*/
if (bytesout + bytesin > SPI_FIFO_DEPTH) {
printk(BIOS_WARNING, "FCH_SC: Too much to transfer, code error!\n");
printk(BIOS_WARNING, "FCH SPI: Too much to transfer, code error!\n");
return -1;
}