Renamed state in buscachefsm to match AHB phases.

This commit is contained in:
Ross Thompson 2022-09-02 17:17:40 -05:00
parent 6f2acf678c
commit 00cffb0aa5

View File

@ -59,15 +59,15 @@ module buscachefsm #(parameter integer WordCountThreshold,
output logic [2:0] HBURST output logic [2:0] HBURST
); );
typedef enum logic [2:0] {STATE_READY, typedef enum logic [2:0] {ADR_PHASE,
STATE_CAPTURE, DATA_PHASE,
STATE_DELAY, MEM3,
STATE_CACHE_FETCH, CACHE_FETCH,
STATE_CACHE_EVICT} busstatetype; CACHE_EVICT} busstatetype;
typedef enum logic [1:0] {AHB_IDLE = 2'b00, AHB_BUSY = 2'b01, AHB_NONSEQ = 2'b10, AHB_SEQ = 2'b11} ahbtranstype; typedef enum logic [1:0] {AHB_IDLE = 2'b00, AHB_BUSY = 2'b01, AHB_NONSEQ = 2'b10, AHB_SEQ = 2'b11} ahbtranstype;
(* mark_debug = "true" *) busstatetype BusCurrState, BusNextState; (* mark_debug = "true" *) busstatetype CurrState, NextState;
logic [LOGWPL-1:0] NextWordCount; logic [LOGWPL-1:0] NextWordCount;
logic FinalWordCount; logic FinalWordCount;
@ -77,24 +77,24 @@ module buscachefsm #(parameter integer WordCountThreshold,
logic CacheAccess; logic CacheAccess;
always_ff @(posedge HCLK) always_ff @(posedge HCLK)
if (~HRESETn) BusCurrState <= #1 STATE_READY; if (~HRESETn) CurrState <= #1 ADR_PHASE;
else BusCurrState <= #1 BusNextState; else CurrState <= #1 NextState;
always_comb begin always_comb begin
case(BusCurrState) case(CurrState)
STATE_READY: if(HREADY & |RW) BusNextState = STATE_CAPTURE; ADR_PHASE: if(HREADY & |RW) NextState = DATA_PHASE;
else if (HREADY & CacheRW[0]) BusNextState = STATE_CACHE_EVICT; else if (HREADY & CacheRW[0]) NextState = CACHE_EVICT;
else if (HREADY & CacheRW[1]) BusNextState = STATE_CACHE_FETCH; else if (HREADY & CacheRW[1]) NextState = CACHE_FETCH;
else BusNextState = STATE_READY; else NextState = ADR_PHASE;
STATE_CAPTURE: if(HREADY) BusNextState = STATE_DELAY; DATA_PHASE: if(HREADY) NextState = MEM3;
else BusNextState = STATE_CAPTURE; else NextState = DATA_PHASE;
STATE_DELAY: if(CPUBusy) BusNextState = STATE_DELAY; MEM3: if(CPUBusy) NextState = MEM3;
else BusNextState = STATE_READY; else NextState = ADR_PHASE;
STATE_CACHE_FETCH: if(HREADY & FinalWordCount) BusNextState = STATE_READY; CACHE_FETCH: if(HREADY & FinalWordCount) NextState = ADR_PHASE;
else BusNextState = STATE_CACHE_FETCH; else NextState = CACHE_FETCH;
STATE_CACHE_EVICT: if(HREADY & FinalWordCount) BusNextState = STATE_READY; CACHE_EVICT: if(HREADY & FinalWordCount) NextState = ADR_PHASE;
else BusNextState = STATE_CACHE_EVICT; else NextState = CACHE_EVICT;
default: BusNextState = STATE_READY; default: NextState = ADR_PHASE;
endcase endcase
end end
@ -116,26 +116,26 @@ module buscachefsm #(parameter integer WordCountThreshold,
assign NextWordCount = WordCount + 1'b1; assign NextWordCount = WordCount + 1'b1;
assign FinalWordCount = WordCountDelayed == WordCountThreshold[LOGWPL-1:0]; assign FinalWordCount = WordCountDelayed == WordCountThreshold[LOGWPL-1:0];
assign WordCntEn = ((BusNextState == STATE_CACHE_EVICT | BusNextState == STATE_CACHE_FETCH) & HREADY) | assign WordCntEn = ((NextState == CACHE_EVICT | NextState == CACHE_FETCH) & HREADY) |
(BusNextState == STATE_READY & |CacheRW & HREADY); (NextState == ADR_PHASE & |CacheRW & HREADY);
assign WordCntReset = BusNextState == STATE_READY; assign WordCntReset = NextState == ADR_PHASE;
assign CaptureEn = (BusCurrState == STATE_CAPTURE & RW[1]) | (BusCurrState == STATE_CACHE_FETCH & HREADY); assign CaptureEn = (CurrState == DATA_PHASE & RW[1]) | (CurrState == CACHE_FETCH & HREADY);
assign CacheAccess = BusCurrState == STATE_CACHE_FETCH | BusCurrState == STATE_CACHE_EVICT; assign CacheAccess = CurrState == CACHE_FETCH | CurrState == CACHE_EVICT;
assign BusStall = (BusCurrState == STATE_READY & (|RW | |CacheRW)) | assign BusStall = (CurrState == ADR_PHASE & (|RW | |CacheRW)) |
//(BusCurrState == STATE_CAPTURE & ~RW[0]) | // replace the next line with this. Fails uart test but i think it's a test problem not a hardware problem. //(CurrState == DATA_PHASE & ~RW[0]) | // replace the next line with this. Fails uart test but i think it's a test problem not a hardware problem.
(BusCurrState == STATE_CAPTURE) | (CurrState == DATA_PHASE) |
(BusCurrState == STATE_CACHE_FETCH) | (CurrState == CACHE_FETCH) |
(BusCurrState == STATE_CACHE_EVICT); (CurrState == CACHE_EVICT);
assign BusCommitted = BusCurrState != STATE_READY; assign BusCommitted = CurrState != ADR_PHASE;
assign SelUncachedAdr = (BusCurrState == STATE_READY & |RW) | assign SelUncachedAdr = (CurrState == ADR_PHASE & |RW) |
(BusCurrState == STATE_CAPTURE) | (CurrState == DATA_PHASE) |
(BusCurrState == STATE_DELAY); (CurrState == MEM3);
// AHB bus interface // AHB bus interface
assign HTRANS = (BusCurrState == STATE_READY & HREADY & (|RW | |CacheRW)) | assign HTRANS = (CurrState == ADR_PHASE & HREADY & (|RW | |CacheRW)) |
(BusCurrState == STATE_CAPTURE & ~HREADY) ? AHB_NONSEQ : (CurrState == DATA_PHASE & ~HREADY) ? AHB_NONSEQ :
(CacheAccess & |WordCount) ? AHB_SEQ : AHB_IDLE; (CacheAccess & |WordCount) ? AHB_SEQ : AHB_IDLE;
assign HWRITE = RW[0] | CacheRW[0]; assign HWRITE = RW[0] | CacheRW[0];
@ -153,8 +153,8 @@ module buscachefsm #(parameter integer WordCountThreshold,
// communication to cache // communication to cache
assign CacheBusAck = (CacheAccess & HREADY & FinalWordCount); assign CacheBusAck = (CacheAccess & HREADY & FinalWordCount);
assign SelBusWord = (BusCurrState == STATE_READY & (RW[0] | CacheRW[0])) | assign SelBusWord = (CurrState == ADR_PHASE & (RW[0] | CacheRW[0])) |
(BusCurrState == STATE_CAPTURE & RW[0]) | (CurrState == DATA_PHASE & RW[0]) |
(BusCurrState == STATE_CACHE_EVICT); (CurrState == CACHE_EVICT);
endmodule endmodule