The `cs_change_delay` backwards compatibility value could be moved outside
of the switch statement.
The only reason to do it, is to make the next patches easier to diff.

Signed-off-by: Alexandru Ardelean <alexandru.ardel...@analog.com>
---
 drivers/spi/spi.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c
index 75ac046cae52..c90e02e6d62f 100644
--- a/drivers/spi/spi.c
+++ b/drivers/spi/spi.c
@@ -1114,16 +1114,15 @@ static void _spi_transfer_cs_change_delay(struct 
spi_message *msg,
        u32 hz;
 
        /* return early on "fast" mode - for everything but USECS */
-       if (!delay && unit != SPI_DELAY_UNIT_USECS)
+       if (!delay) {
+               if (unit == SPI_DELAY_UNIT_USECS)
+                       _spi_transfer_delay_ns(10000);
                return;
+       }
 
        switch (unit) {
        case SPI_DELAY_UNIT_USECS:
-               /* for compatibility use default of 10us */
-               if (!delay)
-                       delay = 10000;
-               else
-                       delay *= 1000;
+               delay *= 1000;
                break;
        case SPI_DELAY_UNIT_NSECS: /* nothing to do here */
                break;
-- 
2.20.1

Reply via email to