X-Git-Url: http://git.joshuawise.com/firearm.git/blobdiff_plain/fb529aac5eac18b5f4b10cb8c7265997ac253a65..refs/heads/msater:/Memory.v

diff --git a/Memory.v b/Memory.v
index c709e83..2580d0e 100644
--- a/Memory.v
+++ b/Memory.v
@@ -1,5 +1,23 @@
 `include "ARM_Constants.v"
 
+`define SWP_READING	2'b01
+`define SWP_WRITING	2'b10
+
+`define LSRH_MEMIO	3'b001
+`define LSRH_BASEWB	3'b010
+`define LSRH_WBFLUSH	3'b100
+
+`define LSR_MEMIO	4'b0001
+`define LSR_STRB_WR	4'b0010
+`define LSR_BASEWB	4'b0100
+`define LSR_WBFLUSH	4'b1000
+
+`define LSM_SETUP	4'b0001
+`define LSM_MEMIO	4'b0010
+`define LSM_BASEWB	4'b0100
+`define LSM_WBFLUSH	4'b1000
+
+
 module Memory(
 	input clk,
 	input Nrst,
@@ -7,17 +25,17 @@ module Memory(
 	input flush,
 
 	/* bus interface */
-	output reg [31:0] busaddr,
-	output reg rd_req,
-	output reg wr_req,
-	input rw_wait,
-	output reg [31:0] wr_data,
-	input [31:0] rd_data,
-	output reg [2:0] data_size,
+	output reg [31:0] dc__addr_3a,
+	output reg dc__rd_req_3a,
+	output reg dc__wr_req_3a,
+	input dc__rw_wait_3a,
+	output reg [31:0] dc__wr_data_3a,
+	input [31:0] dc__rd_data_3a,
+	output reg [2:0] dc__data_size_3a,
 
 	/* regfile interface */
-	output reg [3:0] st_read,
-	input [31:0] st_data,
+	output reg [3:0] rf__read_3_3a,
+	input [31:0] rf__rdata_3_3a,
 	
 	/* Coprocessor interface */
 	output reg cp_req,
@@ -28,17 +46,18 @@ module Memory(
 	output reg [31:0] cp_write,
 	
 	/* stage inputs */
-	input inbubble,
-	input [31:0] pc,
-	input [31:0] insn,
-	input [31:0] op0,
-	input [31:0] op1,
-	input [31:0] op2,
-	input [31:0] spsr,
-	input [31:0] cpsr,
-	input write_reg,
-	input [3:0] write_num,
-	input [31:0] write_data,
+	input bubble_3a,
+	input [31:0] pc_3a,
+	input [31:0] insn_3a,
+	input [31:0] op0_3a,
+	input [31:0] op1_3a,
+	input [31:0] op2_3a,
+	input [31:0] spsr_3a,
+	input [31:0] cpsr_3a,
+	input cpsrup_3a,
+	input write_reg_3a,
+	input [3:0] write_num_3a,
+	input [31:0] write_data_3a,
 
 	/* outputs */
 	output reg outstall,
@@ -49,10 +68,12 @@ module Memory(
 	output reg [3:0] out_write_num = 4'bxxxx,
 	output reg [31:0] out_write_data = 32'hxxxxxxxx,
 	output reg [31:0] outspsr = 32'hxxxxxxxx,
-	output reg [31:0] outcpsr = 32'hxxxxxxxx
+	output reg [31:0] outcpsr = 32'hxxxxxxxx,
+	output reg outcpsrup = 1'hx
 	);
 
 	reg [31:0] addr, raddr, prev_raddr, next_regdata, next_outcpsr;
+	reg next_outcpsrup;
 	reg [31:0] prevaddr;
 	reg [3:0] next_regsel, cur_reg, prev_reg;
 	reg next_writeback;
@@ -62,40 +83,45 @@ module Memory(
 	reg [3:0] next_write_num;
 	reg [31:0] next_write_data;
 
-	reg [1:0] lsr_state = 2'b01, next_lsr_state;
+	reg [3:0] lsr_state = 4'b0001, next_lsr_state;
 	reg [31:0] align_s1, align_s2, align_rddata;
 
-	reg [1:0] lsrh_state = 2'b01, next_lsrh_state;
+	reg [2:0] lsrh_state = 3'b001, next_lsrh_state;
 	reg [31:0] lsrh_rddata;
 	reg [15:0] lsrh_rddata_s1;
 	reg [7:0] lsrh_rddata_s2;
 
 	reg [15:0] regs, next_regs;
-	reg [2:0] lsm_state = 3'b001, next_lsm_state;
+	reg [3:0] lsm_state = 4'b0001, next_lsm_state;
 	reg [5:0] offset, prev_offset, offset_sel;
 
 	reg [31:0] swp_oldval, next_swp_oldval;
 	reg [1:0] swp_state = 2'b01, next_swp_state;
+	
+	reg do_rd_data_latch;
+	reg [31:0] rd_data_latch = 32'hxxxxxxxx;
 
 	always @(posedge clk)
 	begin
-		outpc <= pc;
-		outinsn <= insn;
+		outpc <= pc_3a;
+		outinsn <= insn_3a;
 		outbubble <= next_outbubble;
 		out_write_reg <= next_write_reg;
 		out_write_num <= next_write_num;
 		out_write_data <= next_write_data;
-		regs <= next_regs;
-		prev_reg <= cur_reg;
-		if (!rw_wait)
+		if (!dc__rw_wait_3a)
 			prev_offset <= offset;
 		prev_raddr <= raddr;
 		outcpsr <= next_outcpsr;
-		outspsr <= spsr;
+		outspsr <= spsr_3a;
+		outcpsrup <= next_outcpsrup;
 		swp_state <= next_swp_state;
 		lsm_state <= next_lsm_state;
 		lsr_state <= next_lsr_state;
 		lsrh_state <= next_lsrh_state;
+		if (do_rd_data_latch)
+			rd_data_latch <= dc__rd_data_3a;
+		swp_oldval <= next_swp_oldval;
 		prevaddr <= addr;
 	end
 	
@@ -105,180 +131,490 @@ module Memory(
 			delayedflush <= 1;
 		else if (!outstall /* anything has been handled this time around */)
 			delayedflush <= 0;
-
+	
+	/* Drive the state machines and stall. */
 	always @(*)
 	begin
-		addr = prevaddr;
-		raddr = 32'hxxxxxxxx;
-		rd_req = 1'b0;
-		wr_req = 1'b0;
-		wr_data = 32'hxxxxxxxx;
-		busaddr = 32'hxxxxxxxx;
-		data_size = 3'bxxx;
 		outstall = 1'b0;
-		next_write_reg = write_reg;
-		next_write_num = write_num;
-		next_write_data = write_data;
-		next_outbubble = inbubble;
-		next_regs = regs;
-		cp_req = 1'b0;
-		cp_rnw = 1'bx;
-		cp_write = 32'hxxxxxxxx;
-		offset = prev_offset;
-		next_outcpsr = lsm_state == 3'b010 ? outcpsr : cpsr;
-		lsrh_rddata = 32'hxxxxxxxx;
-		lsrh_rddata_s1 = 16'hxxxx;
-		lsrh_rddata_s2 = 8'hxx;
 		next_lsm_state = lsm_state;
 		next_lsr_state = lsr_state;
 		next_lsrh_state = lsrh_state;
-		next_swp_oldval = swp_oldval;
 		next_swp_state = swp_state;
-		cur_reg = prev_reg;
-
-		/* XXX shit not given about endianness */
-		casez(insn)
-		`DECODE_ALU_SWP: if(!inbubble) begin
-			outstall = rw_wait;
-			next_outbubble = rw_wait;
-			busaddr = {op0[31:2], 2'b0};
-			data_size = insn[22] ? 3'b001 : 3'b100;
+		casez(insn_3a)
+		`DECODE_ALU_SWP: if(!bubble_3a) begin
 			case(swp_state)
-			2'b01: begin
-				rd_req = 1'b1;
+			`SWP_READING: begin
 				outstall = 1'b1;
-				if(!rw_wait) begin
-					next_swp_state = 2'b10;
-					next_swp_oldval = rd_data;
-				end
+				if (!dc__rw_wait_3a)
+					next_swp_state = `SWP_WRITING;
 				$display("SWP: read stage");
 			end
-			2'b10: begin
-				wr_req = 1'b1;
-				wr_data = insn[22] ? {4{op1[7:0]}} : op1;
-				next_write_reg = 1'b1;
-				next_write_num = insn[15:12];
-				next_write_data = insn[22] ? {24'b0, swp_oldval[7:0]} : swp_oldval;
-				if(!rw_wait)
-					next_swp_state = 2'b01;
+			`SWP_WRITING: begin
+				outstall = dc__rw_wait_3a;
+				if(!dc__rw_wait_3a)
+					next_swp_state = `SWP_READING;
 				$display("SWP: write stage");
 			end
-			default: begin end
+			default: begin
+				outstall = 1'bx;
+				next_swp_state = 2'bxx;
+			end
 			endcase
 		end
-		`DECODE_ALU_MULT: begin end
+		`DECODE_ALU_MULT: begin
+			outstall = 1'b0;	/* XXX work around for Xilinx bug */
+			next_lsrh_state = lsrh_state;
+		end
 		`DECODE_ALU_HDATA_REG,
-		`DECODE_ALU_HDATA_IMM: if(!inbubble) begin
-			next_outbubble = rw_wait;
-			outstall = rw_wait;
-			addr = insn[23] ? op0 + op1 : op0 - op1; /* up/down select */
-			raddr = insn[24] ? op0 : addr; /* pre/post increment */
-			busaddr = raddr;
-			/* rotate to correct position */
-			case(insn[6:5])
-			2'b00: begin end /* swp */
-			2'b01: begin /* unsigned half */
-				wr_data = {2{op2[15:0]}}; /* XXX need to store halfword */
-				data_size = 3'b010;
-				lsrh_rddata = {16'b0, raddr[1] ? rd_data[31:16] : rd_data[15:0]};
+		`DECODE_ALU_HDATA_IMM: if(!bubble_3a) begin
+			case(lsrh_state)
+			`LSRH_MEMIO: begin
+				outstall = dc__rw_wait_3a;
+				if(insn_3a[21] | !insn_3a[24]) begin
+					outstall = 1'b1;
+					if(!dc__rw_wait_3a)
+						next_lsrh_state = `LSRH_BASEWB;
+				end
+				
+				if (flush) /* special case! */ begin
+					outstall = 1'b0;
+					next_lsrh_state = `LSRH_MEMIO;
+				end
+				
+				$display("ALU_LDRSTRH: rd_req %d, wr_req %d", dc__rd_req_3a, dc__wr_req_3a);
 			end
-			2'b10: begin /* signed byte */
-				wr_data = {4{op2[7:0]}};
-				data_size = 3'b001;
-				lsrh_rddata_s1 = raddr[1] ? rd_data[31:16] : rd_data[15:0];
-				lsrh_rddata_s2 = raddr[0] ? lsrh_rddata_s1[15:8] : lsrh_rddata_s1[7:0];
-				lsrh_rddata = {{24{lsrh_rddata_s2[7]}}, lsrh_rddata_s2};
+			`LSRH_BASEWB: begin
+				outstall = 1'b1;
+				next_lsrh_state = `LSRH_WBFLUSH;
 			end
-			2'b11: begin /* signed half */
-				wr_data = {2{op2[15:0]}};
-				data_size = 3'b010;
-				lsrh_rddata = raddr[1] ? {{16{rd_data[31]}}, rd_data[31:16]} : {{16{rd_data[15]}}, rd_data[15:0]};
+			`LSRH_WBFLUSH: begin
+				outstall = 1'b0;
+				next_lsrh_state = `LSRH_MEMIO;
+			end
+			default: begin
+				outstall = 1'bx;
+				next_lsrh_state = 3'bxxx;
 			end
 			endcase
-
+		end
+		`DECODE_LDRSTR_UNDEFINED: begin end
+		`DECODE_LDRSTR: if(!bubble_3a) begin
+			outstall = dc__rw_wait_3a;
+			case(lsr_state)
+			`LSR_MEMIO: begin
+				outstall = dc__rw_wait_3a;
+				next_lsr_state = `LSR_MEMIO;
+				if (insn_3a[22] /* B */ && !insn_3a[20] /* L */) begin	/* i.e., strb */
+					outstall = 1'b1;
+					if (!dc__rw_wait_3a)
+						next_lsr_state = `LSR_STRB_WR;
+				end else if (insn_3a[21] /* W */ || !insn_3a[24] /* P */) begin	/* writeback needed */
+					outstall = 1'b1;
+					if (!dc__rw_wait_3a)
+						next_lsr_state = `LSR_BASEWB;
+				end
+				
+				if (flush) begin
+					outstall = 1'b0;
+					next_lsr_state = `LSR_MEMIO;
+				end
+				$display("LDRSTR: rd_req %d, wr_req %d, raddr %08x, wait %d", dc__rd_req_3a, dc__wr_req_3a, raddr, dc__rw_wait_3a);
+			end
+			`LSR_STRB_WR: begin
+				outstall = 1;
+				if(insn_3a[21] /* W */ | !insn_3a[24] /* P */) begin
+					if(!dc__rw_wait_3a)
+						next_lsr_state = `LSR_BASEWB;
+				end else if (!dc__rw_wait_3a)
+					next_lsr_state = `LSR_WBFLUSH;
+				$display("LDRSTR: Handling STRB");
+			end
+			`LSR_BASEWB: begin
+				outstall = 1;
+				next_lsr_state = `LSR_WBFLUSH;
+			end
+			`LSR_WBFLUSH: begin
+				outstall = 0;
+				next_lsr_state = `LSR_MEMIO;
+			end
+			default: begin
+				outstall = 1'bx;
+				next_lsr_state = 4'bxxxx;
+			end
+			endcase
+			$display("LDRSTR: Decoded, bubble %d, insn %08x, lsm state %b -> %b, stall %d", bubble_3a, insn_3a, lsr_state, next_lsr_state, outstall);
+		end
+		`DECODE_LDMSTM: if(!bubble_3a) begin
+			outstall = dc__rw_wait_3a;
+			case(lsm_state)
+			`LSM_SETUP: begin
+				outstall = 1'b1;
+				next_lsm_state = `LSM_MEMIO;
+				if (flush) begin
+					outstall = 1'b0;
+					next_lsm_state = `LSM_SETUP;
+				end
+				$display("LDMSTM: Round 1: base register: %08x, reg list %b", op0_3a, op1_3a[15:0]);
+			end
+			`LSM_MEMIO: begin
+				outstall = 1'b1;
+				if(next_regs == 16'b0 && !dc__rw_wait_3a) begin
+					next_lsm_state = `LSM_BASEWB;
+				end
+				
+				$display("LDMSTM: Stage 2: Writing: regs %b, next_regs %b, reg %d, wr_data %08x, addr %08x", regs, next_regs, cur_reg, rf__rdata_3_3a, dc__addr_3a);
+			end
+			`LSM_BASEWB: begin
+				outstall = 1;
+				next_lsm_state = `LSM_WBFLUSH;
+				$display("LDMSTM: Stage 3: Writing back");
+			end
+			`LSM_WBFLUSH: begin
+				outstall = 0;
+				next_lsm_state = `LSM_SETUP;
+			end
+			default: begin
+				outstall = 1'bx;
+				next_lsm_state = 4'bxxxx;
+			end
+			endcase
+			$display("LDMSTM: Decoded, bubble %d, insn %08x, lsm state %b -> %b, stall %d", bubble_3a, insn_3a, lsm_state, next_lsm_state, outstall);
+		end
+		`DECODE_LDCSTC: if(!bubble_3a) begin
+			$display("WARNING: Unimplemented LDCSTC");
+		end
+		`DECODE_CDP: if (!bubble_3a) begin
+			if (cp_busy) begin
+				outstall = 1;
+			end
+			if (!cp_ack) begin
+				/* XXX undefined instruction trap */
+				$display("WARNING: Possible CDP undefined instruction");
+			end
+		end
+		`DECODE_MRCMCR: if (!bubble_3a) begin
+			if (cp_busy) begin
+				outstall = 1;
+			end
+			if (!cp_ack) begin
+				$display("WARNING: Possible MRCMCR undefined instruction: cp_ack %d, cp_busy %d",cp_ack, cp_busy);
+			end
+			$display("MRCMCR: ack %d, busy %d", cp_ack, cp_busy);
+		end
+		default: begin end
+		endcase
+	end
+	
+	/* Coprocessor input. */
+	always @(*)
+	begin
+		cp_req = 0;
+		cp_rnw = 1'bx;
+		cp_write = 32'hxxxxxxxx;
+		casez (insn_3a)
+		`DECODE_CDP: if(!bubble_3a) begin
+			cp_req = 1;
+		end
+		`DECODE_MRCMCR: if(!bubble_3a) begin
+			cp_req = 1;
+			cp_rnw = insn_3a[20] /* L */;
+			if (insn_3a[20] == 0 /* store to coprocessor */)
+				cp_write = op0_3a;
+		end
+		endcase
+	end
+	
+	/* Register output logic. */
+	always @(*)
+	begin
+		next_write_reg = write_reg_3a;
+		next_write_num = write_num_3a;
+		next_write_data = write_data_3a;
+		next_outcpsr = lsm_state == 4'b0010 ? outcpsr : cpsr_3a;
+		next_outcpsrup = cpsrup_3a;
+		
+		casez(insn_3a)
+		`DECODE_ALU_SWP: if (!bubble_3a) begin
+			next_write_reg = 1'bx;
+			next_write_num = 4'bxxxx;
+			next_write_data = 32'hxxxxxxxx;
+			case(swp_state)
+			`SWP_READING:
+				next_write_reg = 1'b0;
+			`SWP_WRITING: begin
+				next_write_reg = 1'b1;
+				next_write_num = insn_3a[15:12];
+				next_write_data = insn_3a[22] ? {24'b0, swp_oldval[7:0]} : swp_oldval;
+			end
+			default: begin end
+			endcase
+		end
+		`DECODE_ALU_MULT: begin
+			next_write_reg = write_reg_3a;	/* XXX workaround for ISE 10.1 bug */
+			next_write_num = write_num_3a;
+			next_write_data = write_data_3a;
+			next_outcpsr = lsm_state == 4'b0010 ? outcpsr : cpsr_3a;
+			next_outcpsrup = cpsrup_3a;
+		end
+		`DECODE_ALU_HDATA_REG,
+		`DECODE_ALU_HDATA_IMM: if(!bubble_3a) begin
+			next_write_reg = 1'bx;
+			next_write_num = 4'bxxxx;
+			next_write_data = 32'hxxxxxxxx;
 			case(lsrh_state)
-			2'b01: begin
-				rd_req = insn[20];
-				wr_req = ~insn[20];
-				next_write_num = insn[15:12];
+			`LSRH_MEMIO: begin
+				next_write_num = insn_3a[15:12];
 				next_write_data = lsrh_rddata;
-				if(insn[20]) begin
+				if(insn_3a[20]) begin
 					next_write_reg = 1'b1;
 				end
-				if(insn[21] | !insn[24]) begin
-					outstall = 1'b1;
-					if(!rw_wait)
-						next_lsrh_state = 2'b10;
-				end
-				$display("ALU_LDRSTRH: rd_req %d, wr_req %d", rd_req, wr_req);
 			end
-			2'b10: begin
+			`LSRH_BASEWB: begin
 				next_write_reg = 1'b1;
-				next_write_num = insn[19:16];
+				next_write_num = insn_3a[19:16];
 				next_write_data = addr;
-				next_lsrh_state = 2'b10;
 			end
+			`LSRH_WBFLUSH:
+				next_write_reg = 1'b0;
 			default: begin end
 			endcase
 		end
 		`DECODE_LDRSTR_UNDEFINED: begin end
-		`DECODE_LDRSTR: if(!inbubble) begin
-			next_outbubble = rw_wait;
-			outstall = rw_wait;
-			addr = insn[23] ? op0 + op1 : op0 - op1; /* up/down select */
-			raddr = insn[24] ? addr : op0; /* pre/post increment */
-			busaddr = raddr;
-			/* rotate to correct position */
-			align_s1 = raddr[1] ? {rd_data[15:0], rd_data[31:16]} : rd_data;
-			align_s2 = raddr[0] ? {align_s1[7:0], align_s1[31:8]} : align_s1;
-			/* select byte or word */
-			align_rddata = insn[22] ? {24'b0, align_s2[7:0]} : align_s2;
-			wr_data = insn[22] ? {4{op2[7:0]}} : op2; /* XXX need to actually store just a byte */
-			data_size = insn[22] ? 3'b001 : 3'b100;
+		`DECODE_LDRSTR: if(!bubble_3a) begin
+			next_write_reg = 1'bx;
+			next_write_num = 4'bxxxx;
+			next_write_data = 32'hxxxxxxxx;
 			case(lsr_state)
-			2'b01: begin
-				rd_req = insn[20] /* L */;
-				wr_req = ~insn[20] /* L */;
-				next_write_reg = insn[20] /* L */;
-				next_write_num = insn[15:12];
-				if(insn[20] /* L */) begin
-					next_write_data = align_rddata;
-				end
-				if(insn[21] /* W */ | !insn[24] /* P */) begin
-					outstall = 1'b1;
-					if(!rw_wait)
-						next_lsr_state = 2'b10;
+			`LSR_MEMIO: begin
+				next_write_reg = insn_3a[20] /* L */;
+				next_write_num = insn_3a[15:12];
+				if(insn_3a[20] /* L */) begin
+					next_write_data = insn_3a[22] /* B */ ? {24'h0, align_rddata[7:0]} : align_rddata;
 				end
-				$display("LDRSTR: rd_req %d, wr_req %d, raddr %08x, wait %d", rd_req, wr_req, raddr, rw_wait);
 			end
-			2'b10: begin
+			`LSR_STRB_WR:
+				next_write_reg = 1'b0;
+			`LSR_BASEWB: begin
 				next_write_reg = 1'b1;
-				next_write_num = insn[19:16];
+				next_write_num = insn_3a[19:16];
 				next_write_data = addr;
-				next_lsr_state = 2'b01;
 			end
+			`LSR_WBFLUSH:
+				next_write_reg = 1'b0;
 			default: begin end
 			endcase
 		end
-		/* XXX ldm/stm incorrect in that stupid case where one of the listed regs is the base reg */
-		`DECODE_LDMSTM: if(!inbubble) begin
-			outstall = rw_wait;
-			next_outbubble = rw_wait;
-			data_size = 3'b100;
+		`DECODE_LDMSTM: if(!bubble_3a) begin
+			next_write_reg = 1'bx;
+			next_write_num = 4'bxxxx;
+			next_write_data = 32'hxxxxxxxx;
 			case(lsm_state)
-			3'b001: begin
-//				next_regs = insn[23] ? op1[15:0] : op1[0:15];
-				/** verilator can suck my dick */
-				$display("LDMSTM: Round 1: base register: %08x, reg list %b", op0, op1[15:0]);
-				next_regs = insn[23] /* U */ ? op1[15:0] : {op1[0], op1[1], op1[2], op1[3], op1[4], op1[5], op1[6], op1[7],
-				                                            op1[8], op1[9], op1[10], op1[11], op1[12], op1[13], op1[14], op1[15]};
+			`LSM_SETUP:
+				next_write_reg = 1'b0;
+			`LSM_MEMIO: begin
+				if(insn_3a[20] /* L */) begin
+					next_write_reg = !dc__rw_wait_3a;
+					next_write_num = cur_reg;
+					next_write_data = dc__rd_data_3a;
+				end else
+					next_write_reg = 1'b0;
+			end
+			`LSM_BASEWB: begin
+				next_write_reg = insn_3a[21] /* writeback */;
+				next_write_num = insn_3a[19:16];
+				next_write_data = insn_3a[23] ? op0_3a + {26'b0, prev_offset} : op0_3a - {26'b0, prev_offset};
+				if(cur_reg == 4'hF && insn_3a[22]) begin
+					next_outcpsr = spsr_3a;
+					next_outcpsrup = 1;
+				end
+			end
+			`LSM_WBFLUSH:
+				next_write_reg = 1'b0;
+			default: begin end
+			endcase
+		end
+		`DECODE_MRCMCR: if(!bubble_3a) begin
+			next_write_reg = 1'bx;
+			next_write_num = 4'bxxxx;
+			next_write_data = 32'hxxxxxxxx;
+			next_outcpsr = 32'hxxxxxxxx;
+			next_outcpsrup = 1'bx;
+			if (insn_3a[20] == 1 /* load from coprocessor */)
+				if (insn_3a[15:12] != 4'hF /* Fuck you ARM */) begin
+					next_write_reg = 1'b1;
+					next_write_num = insn_3a[15:12];
+					next_write_data = cp_read;
+				end else begin
+					next_outcpsr = {cp_read[31:28], cpsr_3a[27:0]};
+					next_outcpsrup = 1;
+				end
+		end
+		endcase
+	end
+	
+	/* Bus/address control logic. */
+	always @(*)
+	begin
+		dc__rd_req_3a = 1'b0;
+		dc__wr_req_3a = 1'b0;
+		offset = prev_offset;
+		addr = prevaddr;
+		raddr = 32'hxxxxxxxx;
+		dc__addr_3a = 32'hxxxxxxxx;
+		dc__data_size_3a = 3'bxxx;
+		
+		casez(insn_3a)
+		`DECODE_ALU_SWP: if(!bubble_3a) begin
+			dc__addr_3a = {op0_3a[31:2], 2'b0};
+			dc__data_size_3a = insn_3a[22] ? 3'b001 : 3'b100;
+			case(swp_state)
+			`SWP_READING:
+				dc__rd_req_3a = 1'b1;
+			`SWP_WRITING:
+				dc__wr_req_3a = 1'b1;
+			default: begin end
+			endcase
+		end
+		`DECODE_ALU_MULT: begin
+			dc__rd_req_3a = 1'b0;	/* XXX workaround for Xilinx bug */
+			dc__wr_req_3a = 1'b0;
+			offset = prev_offset;
+			addr = prevaddr;
+		end
+		`DECODE_ALU_HDATA_REG,
+		`DECODE_ALU_HDATA_IMM: if(!bubble_3a) begin
+			addr = insn_3a[23] ? op0_3a + op1_3a : op0_3a - op1_3a; /* up/down select */
+			raddr = insn_3a[24] ? op0_3a : addr; /* pre/post increment */
+			dc__addr_3a = raddr;
+			/* rotate to correct position */
+			case(insn_3a[6:5])
+			2'b01: /* unsigned half */
+				dc__data_size_3a = 3'b010;
+			2'b10: /* signed byte */
+				dc__data_size_3a = 3'b001;
+			2'b11: /* signed half */
+				dc__data_size_3a = 3'b010;
+			default: begin
+				dc__data_size_3a = 3'bxxx;
+			end
+			endcase
+			
+			case(lsrh_state)
+			`LSRH_MEMIO: begin
+				dc__rd_req_3a = insn_3a[20];
+				dc__wr_req_3a = ~insn_3a[20];
+			end
+			`LSRH_BASEWB: begin end
+			`LSRH_WBFLUSH: begin end
+			default: begin end
+			endcase
+		end
+		`DECODE_LDRSTR_UNDEFINED: begin end
+		`DECODE_LDRSTR: if(!bubble_3a) begin
+			addr = insn_3a[23] ? op0_3a + op1_3a : op0_3a - op1_3a; /* up/down select */
+			raddr = insn_3a[24] ? addr : op0_3a; /* pre/post increment */
+			dc__addr_3a = raddr;
+			dc__data_size_3a = insn_3a[22] ? 3'b001 : 3'b100;
+			case (lsr_state)
+			`LSR_MEMIO: begin
+				dc__rd_req_3a = insn_3a[20] /* L */ || insn_3a[22] /* B */;
+				dc__wr_req_3a = !insn_3a[20] /* L */ && !insn_3a[22]/* B */;
+			end
+			`LSR_STRB_WR:
+				dc__wr_req_3a = 1;
+			`LSR_BASEWB: begin end
+			`LSR_WBFLUSH: begin end
+			default: begin end
+			endcase
+		end
+		`DECODE_LDMSTM: if (!bubble_3a) begin
+			dc__data_size_3a = 3'b100;
+			case (lsm_state)
+			`LSM_SETUP:
 				offset = 6'b0;
-				outstall = 1'b1;
-				next_lsm_state = 3'b010;
+			`LSM_MEMIO: begin
+				dc__rd_req_3a = insn_3a[20];
+				dc__wr_req_3a = ~insn_3a[20];
+				offset = prev_offset + 6'h4;
+				offset_sel = insn_3a[24] ? offset : prev_offset;
+				raddr = insn_3a[23] ? op0_3a + {26'b0, offset_sel} : op0_3a - {26'b0, offset_sel};
+				dc__addr_3a = raddr;
 			end
-			3'b010: begin
-				rd_req = insn[20];
-				wr_req = ~insn[20];
+			`LSM_BASEWB: begin end
+			`LSM_WBFLUSH: begin end
+			default: begin end
+			endcase
+		end
+		`DECODE_LDCSTC: begin end
+		`DECODE_CDP: begin end
+		`DECODE_MRCMCR: begin end
+		default: begin end
+		endcase
+	end
+	
+	/* Bus data control logic. */
+	always @(*)
+	begin
+		dc__wr_data_3a = 32'hxxxxxxxx;
+		
+		casez(insn_3a)
+		`DECODE_ALU_SWP: if(!bubble_3a)
+			if (swp_state == `SWP_WRITING)
+				dc__wr_data_3a = insn_3a[22] ? {4{op1_3a[7:0]}} : op1_3a;
+		`DECODE_ALU_MULT: begin end
+		`DECODE_ALU_HDATA_REG,
+		`DECODE_ALU_HDATA_IMM: if(!bubble_3a)
+			case(insn_3a[6:5])
+			2'b01: /* unsigned half */
+				dc__wr_data_3a = {2{op2_3a[15:0]}}; /* XXX need to store halfword */
+			2'b10: /* signed byte */
+				dc__wr_data_3a = {4{op2_3a[7:0]}};
+			2'b11: /* signed half */
+				dc__wr_data_3a = {2{op2_3a[15:0]}};
+			default: begin end
+			endcase
+		`DECODE_LDRSTR_UNDEFINED: begin end
+		`DECODE_LDRSTR: if(!bubble_3a) begin
+			dc__wr_data_3a = insn_3a[22] ? {24'h0, {op2_3a[7:0]}} : op2_3a;
+			if (lsr_state == `LSR_STRB_WR)
+				case (dc__addr_3a[1:0])
+				2'b00: dc__wr_data_3a = {rd_data_latch[31:8], op2_3a[7:0]};
+				2'b01: dc__wr_data_3a = {rd_data_latch[31:16], op2_3a[7:0], rd_data_latch[7:0]};
+				2'b10: dc__wr_data_3a = {rd_data_latch[31:24], op2_3a[7:0], rd_data_latch[15:0]};
+				2'b11: dc__wr_data_3a = {op2_3a[7:0], rd_data_latch[23:0]};
+				endcase
+		end
+		`DECODE_LDMSTM: if (!bubble_3a)
+			if (lsm_state == `LSM_MEMIO)
+				dc__wr_data_3a = (cur_reg == 4'hF) ? (pc_3a + 12) : rf__rdata_3_3a;
+		`DECODE_LDCSTC: begin end
+		`DECODE_CDP: begin end
+		`DECODE_MRCMCR: begin end
+		default: begin end
+		endcase
+	end
+	
+	/* LDM/STM register control logic. */
+	always @(posedge clk)
+		if (!dc__rw_wait_3a || lsm_state != `LSM_MEMIO)
+		begin
+			prev_reg <= cur_reg;
+			regs <= next_regs;
+		end
+	
+	always @(*)
+	begin
+		rf__read_3_3a = 4'hx;
+		cur_reg = prev_reg;
+		next_regs = regs;
+		
+		casez(insn_3a)
+		`DECODE_LDMSTM: if(!bubble_3a) begin
+			case(lsm_state)
+			`LSM_SETUP:
+				next_regs = insn_3a[23] /* U */ ? op1_3a[15:0] : {op1_3a[0], op1_3a[1], op1_3a[2], op1_3a[3], op1_3a[4], op1_3a[5], op1_3a[6], op1_3a[7],
+				                                               op1_3a[8], op1_3a[9], op1_3a[10], op1_3a[11], op1_3a[12], op1_3a[13], op1_3a[14], op1_3a[15]};
+			`LSM_MEMIO: begin
 				casez(regs)
 				16'b???????????????1: begin
 					cur_reg = 4'h0;
@@ -349,82 +685,119 @@ module Memory(
 					next_regs = 16'b0;
 				end
 				endcase
-				cur_reg = insn[23] ? cur_reg : 4'hF - cur_reg;
-				if(cur_reg == 4'hF && insn[22]) begin
-					next_outcpsr = spsr;
-				end
-
-				offset = prev_offset + 6'h4;
-				offset_sel = insn[24] ? offset : prev_offset;
-				raddr = insn[23] ? op0 + {26'b0, offset_sel} : op0 - {26'b0, offset_sel};
-				if(insn[20]) begin
-					next_write_reg = !rw_wait;
-					next_write_num = cur_reg;
-					next_write_data = rd_data;
-				end
-				if (rw_wait) begin
-					next_regs = regs;
-					cur_reg = prev_reg;	/* whoops, do this one again */
-				end
-
-				st_read = cur_reg;
-				wr_data = (cur_reg == 4'hF) ? (pc + 12) : st_data;
-				busaddr = raddr;
+				cur_reg = insn_3a[23] ? cur_reg : 4'hF - cur_reg;
 				
-				$display("LDMSTM: Stage 2: Writing: regs %b, next_regs %b, reg %d, wr_data %08x, addr %08x", regs, next_regs, cur_reg, wr_data, busaddr);
-
-				outstall = 1'b1;
+				rf__read_3_3a = cur_reg;
+			end
+			`LSM_BASEWB: begin end
+			`LSM_WBFLUSH: begin end
+			default: begin end
+			endcase
+		end
+		endcase
+	end
+	
+	always @(*)
+	begin
+		do_rd_data_latch = 0;
+		
+		next_outbubble = bubble_3a;
+		
+		lsrh_rddata = 32'hxxxxxxxx;
+		lsrh_rddata_s1 = 16'hxxxx;
+		lsrh_rddata_s2 = 8'hxx;
+		next_swp_oldval = swp_oldval;
+		
+		align_s1 = 32'hxxxxxxxx;
+		align_s2 = 32'hxxxxxxxx;
+		align_rddata = 32'hxxxxxxxx;
 
-				if(next_regs == 16'b0) begin
-					next_lsm_state = 3'b100;
-				end
+		/* XXX shit not given about endianness */
+		casez(insn_3a)
+		`DECODE_ALU_SWP: if(!bubble_3a) begin
+			next_outbubble = dc__rw_wait_3a;
+			case(swp_state)
+			`SWP_READING:
+				if(!dc__rw_wait_3a)
+					next_swp_oldval = dc__rd_data_3a;
+			`SWP_WRITING: begin end
+			default: begin end
+			endcase
+		end
+		`DECODE_ALU_MULT: begin
+			next_outbubble = bubble_3a;	/* XXX workaround for Xilinx bug */
+		end
+		`DECODE_ALU_HDATA_REG,
+		`DECODE_ALU_HDATA_IMM: if(!bubble_3a) begin
+			next_outbubble = dc__rw_wait_3a;
+			
+			/* rotate to correct position */
+			case(insn_3a[6:5])
+			2'b01: begin /* unsigned half */
+				lsrh_rddata = {16'b0, raddr[1] ? dc__rd_data_3a[31:16] : dc__rd_data_3a[15:0]};
 			end
-			3'b100: begin
-				next_write_reg = insn[21] /* writeback */;
-				next_write_num = insn[19:16];
-				next_write_data = insn[23] ? op0 + {26'b0, prev_offset} : op0 - {26'b0, prev_offset};
-				next_lsm_state = 3'b001;
-				$display("LDMSTM: Stage 3: Writing back");
+			2'b10: begin /* signed byte */
+				lsrh_rddata_s1 = raddr[1] ? dc__rd_data_3a[31:16] : dc__rd_data_3a[15:0];
+				lsrh_rddata_s2 = raddr[0] ? lsrh_rddata_s1[15:8] : lsrh_rddata_s1[7:0];
+				lsrh_rddata = {{24{lsrh_rddata_s2[7]}}, lsrh_rddata_s2};
 			end
-			default: $stop;
+			2'b11: begin /* signed half */
+				lsrh_rddata = raddr[1] ? {{16{dc__rd_data_3a[31]}}, dc__rd_data_3a[31:16]} : {{16{dc__rd_data_3a[15]}}, dc__rd_data_3a[15:0]};
+			end
+			default: begin
+				lsrh_rddata = 32'hxxxxxxxx;
+			end
+			endcase
+
+			case(lsrh_state)
+			`LSRH_MEMIO: begin end
+			`LSRH_BASEWB:
+				next_outbubble = 1'b0;
+			`LSRH_WBFLUSH: begin end
+			default: begin end
 			endcase
-			$display("LDMSTM: Decoded, bubble %d, insn %08x, lsm state %b -> %b, stall %d", inbubble, insn, lsm_state, next_lsm_state, outstall);
 		end
-		`DECODE_LDCSTC: if(!inbubble) begin
-			$display("WARNING: Unimplemented LDCSTC");
+		`DECODE_LDRSTR_UNDEFINED: begin end
+		`DECODE_LDRSTR: if(!bubble_3a) begin
+			next_outbubble = dc__rw_wait_3a;
+			/* rotate to correct position */
+			align_s1 = raddr[1] ? {dc__rd_data_3a[15:0], dc__rd_data_3a[31:16]} : dc__rd_data_3a;
+			align_s2 = raddr[0] ? {align_s1[7:0], align_s1[31:8]} : align_s1;
+			/* select byte or word */
+			align_rddata = insn_3a[22] ? {24'b0, align_s2[7:0]} : align_s2;
+			case(lsr_state)
+			`LSR_MEMIO:
+				if (insn_3a[22] /* B */ && !insn_3a[20] /* L */)
+					do_rd_data_latch = 1;
+			`LSR_STRB_WR: begin end
+			`LSR_BASEWB:
+				next_outbubble = 0;
+			`LSR_WBFLUSH: begin end
+			default: begin end
+			endcase
 		end
-		`DECODE_CDP: if(!inbubble) begin
-			cp_req = 1;
+		/* XXX ldm/stm incorrect in that stupid case where one of the listed regs is the base reg */
+		`DECODE_LDMSTM: if(!bubble_3a) begin
+			next_outbubble = dc__rw_wait_3a;
+			case(lsm_state)
+			`LSM_SETUP: begin end
+			`LSM_MEMIO: begin end
+			`LSM_BASEWB:
+				next_outbubble = 0;
+			`LSM_WBFLUSH: begin end
+			default: $stop;
+			endcase
+		end
+		`DECODE_LDCSTC: begin end
+		`DECODE_CDP: if(!bubble_3a) begin
 			if (cp_busy) begin
-				outstall = 1;
 				next_outbubble = 1;
 			end
-			if (!cp_ack) begin
-				/* XXX undefined instruction trap */
-				$display("WARNING: Possible CDP undefined instruction");
-			end
 		end
-		`DECODE_MRCMCR: if(!inbubble) begin
-			cp_req = 1;
-			cp_rnw = insn[20] /* L */;
-			if (insn[20] == 0 /* store to coprocessor */)
-				cp_write = op0;
-			else begin
-				if (insn[15:12] != 4'hF /* Fuck you ARM */) begin
-					next_write_reg = 1'b1;
-					next_write_num = insn[15:12];
-					next_write_data = cp_read;
-				end else
-					next_outcpsr = {cp_read[31:28], cpsr[27:0]};
-			end
+		`DECODE_MRCMCR: if(!bubble_3a) begin
 			if (cp_busy) begin
-				outstall = 1;
 				next_outbubble = 1;
 			end
-			if (!cp_ack) begin
-				$display("WARNING: Possible MRCMCR undefined instruction: cp_ack %d, cp_busy %d",cp_ack, cp_busy);
-			end
-			$display("MRCMCR: ack %d, busy %d", cp_ack, cp_busy);
 		end
 		default: begin end
 		endcase