diff options
| author | Stefan Wahren <stefan.wahren@i2se.com> | 2023-06-14 23:06:56 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2023-06-19 09:37:25 +0100 |
| commit | 92717c2356cb62c89e8a3dc37cbbab2502562524 (patch) | |
| tree | 7f023df6f4f93aa6344d8eb09427a89c77494974 | |
| parent | c938ab4da0eb1620ae3243b0b24c572ddfc318fc (diff) | |
| download | ntb-work-92717c2356cb62c89e8a3dc37cbbab2502562524.tar.gz | |
net: qca_spi: Avoid high load if QCA7000 is not available
In case the QCA7000 is not available via SPI (e.g. in reset),
the driver will cause a high load. The reason for this is
that the synchronization is never finished and schedule()
is never called. Since the synchronization is not timing
critical, it's safe to drop this from the scheduling condition.
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Fixes: 291ab06ecf67 ("net: qualcomm: new Ethernet over SPI driver for QCA7000")
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/ethernet/qualcomm/qca_spi.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/qualcomm/qca_spi.c b/drivers/net/ethernet/qualcomm/qca_spi.c index c865a4be05eec2..4a1b94e5a8ea9d 100644 --- a/drivers/net/ethernet/qualcomm/qca_spi.c +++ b/drivers/net/ethernet/qualcomm/qca_spi.c @@ -582,8 +582,7 @@ qcaspi_spi_thread(void *data) while (!kthread_should_stop()) { set_current_state(TASK_INTERRUPTIBLE); if ((qca->intr_req == qca->intr_svc) && - (qca->txr.skb[qca->txr.head] == NULL) && - (qca->sync == QCASPI_SYNC_READY)) + !qca->txr.skb[qca->txr.head]) schedule(); set_current_state(TASK_RUNNING); |