From 35b7924431f9ad84b134ee6aed94cfe72e643beb Mon Sep 17 00:00:00 2001
From: Harry Ho <hh@m-labs.hk>
Date: Thu, 3 Jun 2021 11:48:26 +0800
Subject: [PATCH] spi: Add back NSS high delay conditionally based on opcode
 type

---
 src/spi.rs | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/src/spi.rs b/src/spi.rs
index 3cc61a7..d9cd5cf 100644
--- a/src/spi.rs
+++ b/src/spi.rs
@@ -191,25 +191,27 @@ impl <SPI: Transfer<u8>,
     // receiving or sending n*8-bit data.
     // The slice of buffer provided must begin with the 8-bit instruction.
     // If n = 0, the transfer will only involve sending the instruction.
-    fn rw_n<'a>(&mut self, buf: &'a mut [u8], opcode: u8, data_length: usize)
-              -> Result<(), Error> {
+    fn rw_n<'a>(&mut self, buf: &'a mut [u8], opcode: u8, data_length: usize) -> Result<(), Error> {
         assert!(buf.len() > data_length);
         // Enable chip select
         self.nss.set_low();
         // Start writing to SLAVE
         buf[0] = opcode;
-        match self.spi.transfer(&mut buf[..data_length+1]) {
-            Ok(_) => {
+        let result = self.spi.transfer(&mut buf[..data_length+1]);
+        match opcode {
+            opcodes::RCRU | opcodes::WCRU |
+            opcodes::RRXDATA | opcodes::WGPDATA => {
                 // Disable chip select
+                (self.delay_ns)(60);
                 self.nss.set_high();
-                Ok(())
-            },
-            // TODO: Maybe too naive?
-            Err(_) => {
-                // Disable chip select
-                self.nss.set_high();
-                Err(Error::TransferError)
+                (self.delay_ns)(30);
             }
+            _ => { }
+        }
+        match result {
+            Ok(_) => Ok(()),
+            // TODO: Maybe too naive?
+            Err(_) => Err(Error::TransferError),
         }
     }
 }