Improvement: i2c_top/i2c_ctrl.sv Module

Description

We need to enhance the i2c_top.sv and i2c_ctrl.sv modules by addressing several critical issues and optimizing its performance. The following tasks have been identified for this improvement:

Tasks

[x] Remove unnecessary flip-flops that are unused or redundant in the i2c_top.sv.
[ ] Remove unnecessary flip-flops that are unused or redundant in the i2c_ctrl.sv.
[ ] Fix the clock 'runt' issue occurring immediately after activation.
[ ] Conduct a detailed simulation to observe state transitions, paying special attention to clock behavior during these transitions.
[ ] Eliminate one of the states in the i2c_ctrl.sv module, as it appears to be unnecessary and redundant.

Details

1. Remove Unnecessary Flip-Flops

There are several flip-flops in the current design that are either not used or are redundant. These need to be identified and removed to streamline the module.

2. Fix Clock 'Runt' Issue

There is a known issue where a 'runt' clock pulse appears immediately after activation. This needs to be fixed to ensure stable clock signals throughout the module's operation.

3. Detailed Simulation of State Transitions

A thorough simulation is required to closely observe the transitions between states. Particular focus should be on the behavior of the clock signal during these transitions to ensure smooth and error-free operation.

4. Eliminate Redundant State

There appears to be a redundant state in the state machine. This state should be identified and removed to simplify the state machine and improve the efficiency of the module.

chili-chips-ba / openeye-CamSI

issue#3 - Optimizations of I2C Master controller #16