Update housekeeping_spi.v

[NouranAbdelaziz]

No description provided.

[RTimothyEdwards]

Nobody is going to merge a pull request that comes without a description, discussion, or justification.

[NouranAbdelaziz]

Restructuring the housekeeping spi FSM to be more clean so that each always block handle only one register and there is no register updated in more than one always block.

[marwaneltoukhy]

@NouranAbdelaziz What verification have you done on these changes?

cc: @DavidRLindley

[DavidRLindley]

@NouranAbdelaziz What verification have you done?

[d-m-bailey]

@NouranAbdelaziz will this change require a re-synthesis of caravel_core/caravan_core?

[NouranAbdelaziz]

This RTL change was requested by @shalan when I was implementing Caravel on FPGA two years ago, in an attempt to make the housekeeping SPI function work correctly. However, I believe the RTL structure wasn’t the actual issue, and both implementations would likely synthesize to the same result. The change primarily makes the RTL more readable. I’m not sure if it’s still required. I believe I partially verified it on FPGA, but I cannot confirm that it has been fully verified.