Removed unnecessary separate if statement.

openhwgroup · Nov 16, 2024 · dda3cd6 · dda3cd6
1 parent 2ee4525
commit dda3cd6
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/uncore/spi_apb.sv b/src/uncore/spi_apb.sv
@@ -174,16 +174,12 @@ module spi_apb import cvw::*; #(parameter cvw_t P) (
           SPI_CSMODE:  ChipSelectMode <= Din[1:0];
           SPI_DELAY0:  Delay0 <= {Din[23:16], Din[7:0]};
           SPI_DELAY1:  Delay1 <= {Din[23:16], Din[7:0]};
-          SPI_FMT:     Format <= {Din[19:16], Din[2]};                    
+          SPI_FMT:     Format <= {Din[19:16], Din[2]};
+          SPI_TXDATA:  if (~TransmitFIFOFull) TransmitData[7:0] <= Din[7:0];
           SPI_TXMARK:  TransmitWatermark <= Din[2:0];
           SPI_RXMARK:  ReceiveWatermark <= Din[2:0];
           SPI_IE:      InterruptEnable <= Din[1:0];
         endcase
-
-      if (Memwrite)
-        case(Entry)
-          SPI_TXDATA:  if (~TransmitFIFOFull) TransmitData[7:0] <= Din[7:0];
-        endcase
       /* verilator lint_off CASEINCOMPLETE */
 
       // According to FU540 spec: Once interrupt is pending, it will remain set until number