Fixed bug where switching states combinationally rather than using latched logic

fba24c05 · Mike Lyons · 2ecfb943 · fba24c05
Commit fba24c05 authored Mar 07, 2012 by Mike Lyons
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

spiifc.v src/spi_base/spiifc.v +3 -2

No files found.
--- a/src/spi_base/spiifc.v
+++ b/src/spi_base/spiifc.v
@@ -132,8 +132,9 @@ end
 always @(*) begin
  if (Reset || packetStart) begin
    state <= `STATE_GET_CMD;
-  end else if (state_reg == `STATE_GET_CMD && rcByteValid) begin
+// Handled in state_reg logic, should be latched, not immediate.
-    state <= rcByte;
+//  end else if (state_reg == `STATE_GET_CMD && rcByteValid) begin
+//    state <= rcByte;
  end else begin
    state <= state_reg;
  end