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

SPI and related changes #2681

Merged
merged 11 commits into from
Dec 6, 2024
Merged

SPI and related changes #2681

merged 11 commits into from
Dec 6, 2024

Conversation

bugadani
Copy link
Contributor

@bugadani bugadani commented Dec 4, 2024

Closes #2678
Closes #2679
Closes #2680
Closes #2685

@bugadani bugadani added the skip-changelog No changelog modification needed label Dec 4, 2024
@bugadani bugadani marked this pull request as draft December 4, 2024 16:55
@bugadani bugadani force-pushed the spi-fixes branch 3 times, most recently from 712fdc4 to 273938f Compare December 5, 2024 12:30
@bugadani bugadani marked this pull request as ready for review December 5, 2024 13:51
@@ -115,42 +112,43 @@ const MAX_DMA_SIZE: usize = 32736;
///
/// Used to define specific commands sent over the SPI bus.
/// Can be [Command::None] if command phase should be suppressed.
#[derive(Copy, Clone, Debug, PartialEq, Eq)]
#[non_exhaustive]
#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
#[cfg_attr(feature = "defmt", derive(defmt::Format))]
pub enum Command {

This comment was marked as outdated.

This comment was marked as outdated.

@bugadani bugadani mentioned this pull request Dec 5, 2024
41 tasks
@bugadani bugadani force-pushed the spi-fixes branch 2 times, most recently from f3acdbd to 270ad92 Compare December 5, 2024 14:58
Copy link
Contributor

@playfulFence playfulFence left a comment

Choose a reason for hiding this comment

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

LGTM, should satisfy all the issues mentioned in top level comment 😉

@jessebraham jessebraham added this pull request to the merge queue Dec 6, 2024
Merged via the queue into esp-rs:main with commit 6b01f79 Dec 6, 2024
28 checks passed
@bugadani bugadani deleted the spi-fixes branch December 6, 2024 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog No changelog modification needed
Projects
None yet
3 participants