Commit 6319a680 authored by Nobuteru Hayashi's avatar Nobuteru Hayashi Committed by Mark Brown

spi/fsl-espi: avoid infinite loops on fsl_espi_cpu_irq()

It brought nearly infinite loops, and was possible to be
occurred only if the SPI transaction total size are not
alighed with 4. Loops are here at while (tmp--),
tmp is unsigned, and set it with minus value.

The loops are executed as a result of unexpected RX interrupt
occurrence after that. This interrupt may be hardware eratta
and is not fixed.

Fix mspi->len from minus value to 0 and print warning message.
Signed-off-by: default avatarNobuteru Hayashi <hayashi.nbb@ncos.nec.co.jp>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent aa70e567
...@@ -544,6 +544,7 @@ void fsl_espi_cpu_irq(struct mpc8xxx_spi *mspi, u32 events) ...@@ -544,6 +544,7 @@ void fsl_espi_cpu_irq(struct mpc8xxx_spi *mspi, u32 events)
if (events & SPIE_NE) { if (events & SPIE_NE) {
u32 rx_data, tmp; u32 rx_data, tmp;
u8 rx_data_8; u8 rx_data_8;
int rx_nr_bytes = 4;
int ret; int ret;
/* Spin until RX is done */ /* Spin until RX is done */
...@@ -560,7 +561,14 @@ void fsl_espi_cpu_irq(struct mpc8xxx_spi *mspi, u32 events) ...@@ -560,7 +561,14 @@ void fsl_espi_cpu_irq(struct mpc8xxx_spi *mspi, u32 events)
if (mspi->len >= 4) { if (mspi->len >= 4) {
rx_data = mpc8xxx_spi_read_reg(&reg_base->receive); rx_data = mpc8xxx_spi_read_reg(&reg_base->receive);
} else if (mspi->len <= 0) {
dev_err(mspi->dev,
"unexpected RX(SPIE_NE) interrupt occurred,\n"
"(local rxlen %d bytes, reg rxlen %d bytes)\n",
min(4, mspi->len), SPIE_RXCNT(events));
rx_nr_bytes = 0;
} else { } else {
rx_nr_bytes = mspi->len;
tmp = mspi->len; tmp = mspi->len;
rx_data = 0; rx_data = 0;
while (tmp--) { while (tmp--) {
...@@ -571,7 +579,7 @@ void fsl_espi_cpu_irq(struct mpc8xxx_spi *mspi, u32 events) ...@@ -571,7 +579,7 @@ void fsl_espi_cpu_irq(struct mpc8xxx_spi *mspi, u32 events)
rx_data <<= (4 - mspi->len) * 8; rx_data <<= (4 - mspi->len) * 8;
} }
mspi->len -= 4; mspi->len -= rx_nr_bytes;
if (mspi->rx) if (mspi->rx)
mspi->get_rx(rx_data, mspi); mspi->get_rx(rx_data, mspi);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment