[ot] hw/opentitan: ot_spi_device: rename ot_spi_device_release_cs

Function name was confusing. It may also be called when no CS change occurs.

Signed-off-by: Emmanuel Blot <[email protected]>
(cherry picked from commit 12c6a1a)

Author: rivos-eblot

hw/opentitan/ot_spi_device.c

@@ -787,11 +787,13 @@ static bool ot_spi_device_is_hw_read_command(const OtSPIDeviceState *s)
     }
 }
 
-static void ot_spi_device_release_cs(OtSPIDeviceState *s)
+static void ot_spi_device_release(OtSPIDeviceState *s)
 {
     SpiDeviceFlash *f = &s->flash;
     SpiDeviceBus *bus = &s->bus;
 
+    trace_ot_spi_device_release();
+
     BUS_CHANGE_STATE(bus, IDLE);
     bus->byte_count = 0;
 
@@ -1988,7 +1990,7 @@ static void ot_spi_device_chr_receive(void *opaque, const uint8_t *buf,
 
     if (!bus->byte_count) {
         if (bus->release) {
-            ot_spi_device_release_cs(s);
+            ot_spi_device_release(s);
         } else {
             BUS_CHANGE_STATE(bus, IDLE);
         }
@@ -2008,11 +2010,11 @@ static void ot_spi_device_chr_event_hander(void *opaque, QEMUChrEvent event)
             return;
         }
 
-        ot_spi_device_release_cs(s);
+        ot_spi_device_release(s);
     }
 
     if (event == CHR_EVENT_CLOSED) {
-        ot_spi_device_release_cs(s);
+        ot_spi_device_release(s);
     }
 }
 
@@ -2041,7 +2043,7 @@ static int ot_spi_device_chr_be_change(void *opaque)
 
     fifo8_reset(&bus->chr_fifo);
 
-    ot_spi_device_release_cs(s);
+    ot_spi_device_release(s);
 
     if (s->watch_tag > 0) {
         g_source_remove(s->watch_tag);
@@ -2098,7 +2100,7 @@ static void ot_spi_device_reset(DeviceState *dev)
     fifo8_reset(&f->cmd_fifo);
     ot_fifo32_reset(&f->address_fifo);
 
-    ot_spi_device_release_cs(s);
+    ot_spi_device_release(s);
     f->watermark = false;
     f->new_cmd = false;
     s->spi_regs[R_CONTROL] = 0x10u;

hw/opentitan/trace-events

@@ -459,6 +459,7 @@ ot_spi_device_io_spi_write_in(uint32_t addr, const char * regname, uint32_t val,
 ot_spi_device_set_irq(const char *name, unsigned irq, bool level) "%s [%u]: %u"
 ot_spi_device_io_tpm_read_out(uint32_t addr, const char * regname, uint32_t val, uint32_t pc) "addr=0x%02x (%s), val=0x%x, pc=0x%x"
 ot_spi_device_io_tpm_write_in(uint32_t addr, const char * regname, uint32_t val, uint32_t pc) "addr=0x%02x (%s), val=0x%x, pc=0x%x"
+ot_spi_device_release(void) ""
 ot_spi_device_update_last_read_addr(uint32_t addr) "0x%08x"
 
 # ot_spi_host.c