Skip to content

Conversation

antonok-edm
Copy link
Contributor

Followup to jkristell/sdio-host#10, which will also be required before this PR can be merged (cc @jkristell)

This PR adds support for eMMC peripherals using the SDIO module. Sdio is now generic over an SdioPeripheral, which can either be a SdCard or Emmc. Many of the methods are generic enough to be shared between both, although some (like initialization) need to be specialized for each.

@burrbull
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Feb 28, 2022
458: Support eMMC in SDIO module r=burrbull a=antonok-edm

Followup to jkristell/sdio-host#10, which will also be required before this PR can be merged (cc `@jkristell)`

This PR adds support for eMMC peripherals using the SDIO module. `Sdio` is now generic over an `SdioPeripheral`, which can either be a `SdCard` or `Emmc`. Many of the methods are generic enough to be shared between both, although some (like initialization) need to be specialized for each.

Co-authored-by: Anton Lazarev <[email protected]>
@bors
Copy link
Contributor

bors bot commented Feb 28, 2022

This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried.

Additional information:

{"message":"All comments must be resolved.","documentation_url":"https://docs.github.com/articles/about-protected-branches"}

@burrbull
Copy link
Member

bors merge

bors bot added a commit that referenced this pull request Feb 28, 2022
458: Support eMMC in SDIO module r=burrbull a=antonok-edm

Followup to jkristell/sdio-host#10, which will also be required before this PR can be merged (cc `@jkristell)`

This PR adds support for eMMC peripherals using the SDIO module. `Sdio` is now generic over an `SdioPeripheral`, which can either be a `SdCard` or `Emmc`. Many of the methods are generic enough to be shared between both, although some (like initialization) need to be specialized for each.

Co-authored-by: Anton Lazarev <[email protected]>
@bors
Copy link
Contributor

bors bot commented Feb 28, 2022

This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried.

Additional information:

{"message":"All comments must be resolved.","documentation_url":"https://docs.github.com/articles/about-protected-branches"}

@burrbull
Copy link
Member

bors merge

@bors bors bot merged commit be4d08d into stm32-rs:master Feb 28, 2022
@antonok-edm antonok-edm deleted the emmc branch February 28, 2022 18:03
richardeoin added a commit to richardeoin/stm32h7xx-hal that referenced this pull request Mar 27, 2022
richardeoin added a commit to richardeoin/stm32h7xx-hal that referenced this pull request Mar 27, 2022
richardeoin added a commit to richardeoin/stm32h7xx-hal that referenced this pull request Mar 27, 2022
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.

3 participants