bslathi19/taxi
1
0
Fork 0
mirror of https://github.com/fpganinja/taxi.git synced 2025-12-09 00:48:40 -08:00
Code Issues Packages Projects Releases Wiki Activity

lss: Optimize delay implementation in I2C master module

Browse Source 
Signed-off-by: Alex Forencich <alex@alexforencich.com>
This commit is contained in:
Alex Forencich
2025-08-24 11:30:03 -07:00
parent c2c4f5316d
commit 06e6f3e1b4
1 changed files with 42 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

66
src/lss/rtl/taxi_i2c_master.sv
View File

@@ -212,7 +212,8 @@ logic mode_read_reg = 1'b0, mode_read_next;
logic mode_write_multiple_reg = 1'b0, mode_write_multiple_next; logic mode_write_multiple_reg = 1'b0, mode_write_multiple_next;
logic mode_stop_reg = 1'b0, mode_stop_next; logic mode_stop_reg = 1'b0, mode_stop_next;
logic [16:0] delay_reg = '0, delay_next; logic [15:0] delay_count_reg = '0, delay_count_next;
logic delay_run_reg = 1'b0, delay_run_next;
logic delay_scl_reg = 1'b0, delay_scl_next; logic delay_scl_reg = 1'b0, delay_scl_next;
logic delay_sda_reg = 1'b0, delay_sda_next; logic delay_sda_reg = 1'b0, delay_sda_next;
@@ -589,7 +590,8 @@ always_comb begin
phy_rx_data_next = phy_rx_data_reg; phy_rx_data_next = phy_rx_data_reg;
delay_next = delay_reg; delay_count_next = delay_count_reg;
delay_run_next = delay_run_reg;
delay_scl_next = delay_scl_reg; delay_scl_next = delay_scl_reg;
delay_sda_next = delay_sda_reg; delay_sda_next = delay_sda_reg;
@@ -598,25 +600,39 @@ always_comb begin
bus_control_next = bus_control_reg; bus_control_next = bus_control_reg;
if (delay_scl_reg) begin
// wait for SCL to match command
delay_scl_next = scl_o_reg && !scl_i_reg;
end else if (delay_sda_reg) begin
// wait for SDA to match command
delay_sda_next = sda_o_reg && !sda_i_reg;
end else if (delay_run_reg) begin
// time delay
if (delay_count_reg != 0) begin
delay_count_next = delay_count_reg - 1;
end else begin
delay_run_next = 1'b0;
end
end else begin
delay_count_next = prescale;
end
if (phy_release_bus) begin if (phy_release_bus) begin
// release bus and return to idle state // release bus and return to idle state
sda_o_next = 1'b1; sda_o_next = 1'b1;
scl_o_next = 1'b1; scl_o_next = 1'b1;
delay_scl_next = 1'b0; delay_scl_next = 1'b0;
delay_sda_next = 1'b0; delay_sda_next = 1'b0;
delay_next = '0; delay_run_next = '0;
phy_state_next = PHY_STATE_IDLE; phy_state_next = PHY_STATE_IDLE;
end else if (delay_scl_reg) begin end else if (delay_scl_reg) begin
// wait for SCL to match command // wait for SCL to match command
delay_scl_next = scl_o_reg && !scl_i_reg;
phy_state_next = phy_state_reg; phy_state_next = phy_state_reg;
end else if (delay_sda_reg) begin end else if (delay_sda_reg) begin
// wait for SDA to match command // wait for SDA to match command
delay_sda_next = sda_o_reg && !sda_i_reg;
phy_state_next = phy_state_reg; phy_state_next = phy_state_reg;
end else if (delay_reg != 0) begin end else if (delay_run_reg) begin
// time delay // time delay
delay_next = delay_reg - 1;
phy_state_next = phy_state_reg; phy_state_next = phy_state_reg;
end else begin end else begin
case (phy_state_reg) case (phy_state_reg)
@@ -626,7 +642,7 @@ always_comb begin
scl_o_next = 1'b1; scl_o_next = 1'b1;
if (phy_start_bit) begin if (phy_start_bit) begin
sda_o_next = 1'b0; sda_o_next = 1'b0;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_START_1; phy_state_next = PHY_STATE_START_1;
end else begin end else begin
phy_state_next = PHY_STATE_IDLE; phy_state_next = PHY_STATE_IDLE;
@@ -636,19 +652,19 @@ always_comb begin
// bus active // bus active
if (phy_start_bit) begin if (phy_start_bit) begin
sda_o_next = 1'b1; sda_o_next = 1'b1;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_REPEATED_START_1; phy_state_next = PHY_STATE_REPEATED_START_1;
end else if (phy_write_bit) begin end else if (phy_write_bit) begin
sda_o_next = phy_tx_data; sda_o_next = phy_tx_data;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_WRITE_BIT_1; phy_state_next = PHY_STATE_WRITE_BIT_1;
end else if (phy_read_bit) begin end else if (phy_read_bit) begin
sda_o_next = 1'b1; sda_o_next = 1'b1;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_READ_BIT_1; phy_state_next = PHY_STATE_READ_BIT_1;
end else if (phy_stop_bit) begin end else if (phy_stop_bit) begin
sda_o_next = 1'b0; sda_o_next = 1'b0;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_STOP_1; phy_state_next = PHY_STATE_STOP_1;
end else begin end else begin
phy_state_next = PHY_STATE_ACTIVE; phy_state_next = PHY_STATE_ACTIVE;
@@ -664,7 +680,7 @@ always_comb begin
scl_o_next = 1'b1; scl_o_next = 1'b1;
delay_scl_next = 1'b1; delay_scl_next = 1'b1;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_REPEATED_START_2; phy_state_next = PHY_STATE_REPEATED_START_2;
end end
PHY_STATE_REPEATED_START_2: begin PHY_STATE_REPEATED_START_2: begin
@@ -676,7 +692,7 @@ always_comb begin
// //
sda_o_next = 1'b0; sda_o_next = 1'b0;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_START_1; phy_state_next = PHY_STATE_START_1;
end end
PHY_STATE_START_1: begin PHY_STATE_START_1: begin
@@ -688,7 +704,7 @@ always_comb begin
// //
scl_o_next = 1'b0; scl_o_next = 1'b0;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_START_2; phy_state_next = PHY_STATE_START_2;
end end
PHY_STATE_START_2: begin PHY_STATE_START_2: begin
@@ -711,7 +727,7 @@ always_comb begin
scl_o_next = 1'b1; scl_o_next = 1'b1;
delay_scl_next = 1'b1; delay_scl_next = 1'b1;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_WRITE_BIT_2; phy_state_next = PHY_STATE_WRITE_BIT_2;
end end
PHY_STATE_WRITE_BIT_2: begin PHY_STATE_WRITE_BIT_2: begin
@@ -722,7 +738,7 @@ always_comb begin
// scl __/ \__ // scl __/ \__
scl_o_next = 1'b0; scl_o_next = 1'b0;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_WRITE_BIT_3; phy_state_next = PHY_STATE_WRITE_BIT_3;
end end
PHY_STATE_WRITE_BIT_3: begin PHY_STATE_WRITE_BIT_3: begin
@@ -743,7 +759,7 @@ always_comb begin
scl_o_next = 1'b1; scl_o_next = 1'b1;
delay_scl_next = 1'b1; delay_scl_next = 1'b1;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_READ_BIT_2; phy_state_next = PHY_STATE_READ_BIT_2;
end end
PHY_STATE_READ_BIT_2: begin PHY_STATE_READ_BIT_2: begin
@@ -754,7 +770,7 @@ always_comb begin
// scl __/ \__ // scl __/ \__
phy_rx_data_next = sda_i_reg; phy_rx_data_next = sda_i_reg;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_READ_BIT_3; phy_state_next = PHY_STATE_READ_BIT_3;
end end
PHY_STATE_READ_BIT_3: begin PHY_STATE_READ_BIT_3: begin
@@ -765,7 +781,7 @@ always_comb begin
// scl __/ \__ // scl __/ \__
scl_o_next = 1'b0; scl_o_next = 1'b0;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_READ_BIT_4; phy_state_next = PHY_STATE_READ_BIT_4;
end end
PHY_STATE_READ_BIT_4: begin PHY_STATE_READ_BIT_4: begin
@@ -786,7 +802,7 @@ always_comb begin
scl_o_next = 1'b1; scl_o_next = 1'b1;
delay_scl_next = 1'b1; delay_scl_next = 1'b1;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_STOP_2; phy_state_next = PHY_STATE_STOP_2;
end end
PHY_STATE_STOP_2: begin PHY_STATE_STOP_2: begin
@@ -797,7 +813,7 @@ always_comb begin
// scl _______/ // scl _______/
sda_o_next = 1'b1; sda_o_next = 1'b1;
delay_next = 17'(prescale); delay_run_next = 1'b1;
phy_state_next = PHY_STATE_STOP_3; phy_state_next = PHY_STATE_STOP_3;
end end
PHY_STATE_STOP_3: begin PHY_STATE_STOP_3: begin
@@ -831,7 +847,8 @@ always_ff @(posedge clk) begin
mode_write_multiple_reg <= mode_write_multiple_next; mode_write_multiple_reg <= mode_write_multiple_next;
mode_stop_reg <= mode_stop_next; mode_stop_reg <= mode_stop_next;
delay_reg <= delay_next; delay_count_reg <= delay_count_next;
delay_run_reg <= delay_run_next;
delay_scl_reg <= delay_scl_next; delay_scl_reg <= delay_scl_next;
delay_sda_reg <= delay_sda_next; delay_sda_reg <= delay_sda_next;
@@ -870,7 +887,8 @@ always_ff @(posedge clk) begin
if (rst) begin if (rst) begin
state_reg <= STATE_IDLE; state_reg <= STATE_IDLE;
phy_state_reg <= PHY_STATE_IDLE; phy_state_reg <= PHY_STATE_IDLE;
delay_reg <= '0; delay_count_reg <= '0;
delay_run_reg <= 1'b0;
delay_scl_reg <= 1'b0; delay_scl_reg <= 1'b0;
delay_sda_reg <= 1'b0; delay_sda_reg <= 1'b0;
s_axis_cmd_ready_reg <= 1'b0; s_axis_cmd_ready_reg <= 1'b0;