[qemu] Enable the tpm_tx_rx test to run with qemu

Signed-off-by: Douglas Reis <[email protected]>

Author: engdoreis

sw/device/tests/BUILD

@@ -3912,6 +3912,8 @@ opentitan_test(
             "//hw/top_earlgrey:sim_verilator": None,
             "//hw/top_earlgrey:fpga_cw340_sival": None,
             "//hw/top_earlgrey:fpga_cw340_sival_rom_ext": None,
+            "//hw/top_earlgrey:sim_qemu_rom_with_fake_keys": None,
+            "//hw/top_earlgrey:sim_qemu_sival_rom_ext": None,
         },
     ),
     fpga = fpga_params(
@@ -3923,6 +3925,12 @@ opentitan_test(
         """,
         test_harness = "//sw/host/tests/chip/spi_device:spi_device_tpm_test",
     ),
+    qemu = qemu_params(
+        test_cmd = """
+            "{firmware:elf}"
+        """,
+        test_harness = "//sw/host/tests/chip/spi_device:spi_device_tpm_test",
+    ),
     silicon = silicon_params(
         test_cmd = """
             --bootstrap="{firmware}"

sw/host/opentitanlib/src/app/config/opentitan_qemu.json

@@ -79,6 +79,9 @@
       // For now we model this strap as a no-op without any pins.
       "pins": [],
     },
+    {
+      "name": "SPI_TPM"
+    }
   ],
   "uarts": [
     {

sw/host/opentitanlib/src/tpm/driver.rs

@@ -277,6 +277,7 @@ impl SpiDriver {
                 if Instant::now().duration_since(start_time) > TIMEOUT {
                     bail!(TpmError::Timeout)
                 }
+                thread::sleep(Duration::from_millis(1));
             }
         }
         Ok(())

sw/host/opentitanlib/src/transport/qemu/spi.rs

@@ -11,7 +11,9 @@ use anyhow::{Context, ensure};
 use serialport::TTYPort;
 
 use crate::io::gpio;
-use crate::io::spi::{AssertChipSelect, MaxSizes, Target, Transfer, TransferMode};
+use crate::io::spi::{
+    AssertChipSelect, MaxSizes, Target, TargetChipDeassert, Transfer, TransferMode,
+};
 use crate::transport::TransportError;
 use crate::util::voltage::Voltage;
 
@@ -205,7 +207,8 @@ impl Target for QemuSpi {
     fn assert_cs(self: Rc<Self>) -> anyhow::Result<AssertChipSelect> {
         // Could potentially be implemented by sending an empty packet that
         // holds the CS and another which deasserts when dropped.
-        Err(TransportError::UnsupportedOperation.into())
+        log::warn!("TODO: Implement CS for Qemu");
+        Ok(AssertChipSelect::new(self))
     }
 
     fn set_voltage(&self, _voltage: Voltage) -> anyhow::Result<()> {
@@ -216,3 +219,7 @@ impl Target for QemuSpi {
         Err(TransportError::UnsupportedOperation.into())
     }
 }
+
+impl TargetChipDeassert for QemuSpi {
+    fn deassert_cs(&self) {}
+}