diff options
| author | Stefan Wahren <stefan.wahren@i2se.com> | 2021-05-08 15:36:33 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2021-05-11 00:29:18 +0300 |
| commit | b76078df15935b54c353eb0461c95a6eaf73c7ca (patch) | |
| tree | c16605662796e6f0773a395ca056dcb70975f49c /drivers | |
| parent | 48de7c0c1c9225b478c602184be8e00b92b35c61 (diff) | |
| download | linux-b76078df15935b54c353eb0461c95a6eaf73c7ca.tar.xz | |
net: qca_spi: Avoid reading signature three times in a row
There is no need to read the signature three times. So bail out
in case the second check failed.
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/qualcomm/qca_spi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/qualcomm/qca_spi.c b/drivers/net/ethernet/qualcomm/qca_spi.c index ab9b02574a15..3e2a54c2fc83 100644 --- a/drivers/net/ethernet/qualcomm/qca_spi.c +++ b/drivers/net/ethernet/qualcomm/qca_spi.c @@ -506,6 +506,7 @@ qcaspi_qca7k_sync(struct qcaspi *qca, int event) if (signature != QCASPI_GOOD_SIGNATURE) { qca->sync = QCASPI_SYNC_UNKNOWN; netdev_dbg(qca->net_dev, "sync: got CPU on, but signature was invalid, restart\n"); + return; } else { /* ensure that the WRBUF is empty */ qcaspi_read_register(qca, SPI_REG_WRBUF_SPC_AVA, |