Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[hw,spi_host,rtl] CSB is NumCS wide #25782

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Conversation

Razer6
Copy link
Member

@Razer6 Razer6 commented Jan 7, 2025

This was mistakenly changed to CSW in the last PR. Revert it back.

This was mistakenly changed to CSW in the last PR. Revert it back.

Signed-off-by: Robert Schilling <[email protected]>
@Razer6 Razer6 requested review from a-will and vogelpi January 7, 2025 08:19
@vogelpi
Copy link
Contributor

vogelpi commented Jan 7, 2025

CHANGE AUTHORIZED: hw/ip/spi_host/rtl/spi_host_fsm.sv

This PR reverts an erroneous change of a previous PR. This is okay.

Copy link
Contributor

@vogelpi vogelpi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rswarbrick
Copy link
Contributor

CHANGE AUTHORIZED: hw/ip/spi_host/rtl/spi_host_fsm.sv

Fixes an accidental change: definitely sensible to apply!

@a-will a-will merged commit c97e3fa into lowRISC:master Jan 7, 2025
38 checks passed
@Razer6 Razer6 deleted the num-cs-csb branch January 7, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants