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

add alignment to high-level flash clients #24489

Closed
wants to merge 2 commits into from

Conversation

pabigot
Copy link
Collaborator

@pabigot pabigot commented Apr 19, 2020

The USB mass storage infrastructure uses the disk-access API with alternative underlying drivers, one of which can be flash, which can be instantiated as Nordic QSPI, which requires that data buffers be 4-byte aligned. Add the buffer alignments necessary to allow export of QSPI-based flash file systems through the USB mass storage class.

@pabigot pabigot added area: USB Universal Serial Bus area: Flash DNM This PR should not be merged (Do Not Merge) labels Apr 19, 2020
pabigot added 2 commits April 20, 2020 17:16
Flash drivers may impose alignment requirements on the destination
buffers due to use of DMA transfers.  In particular Nordic QSPI flash
API requires that addresses, sizes, and buffers all by 4-byte aligned.
Align the page buffer to satisfy this requirement.

Signed-off-by: Peter A. Bigot <pab@pabigot.com>
Flash drivers may impose alignment requirements on the destination
buffers due to use of DMA transfers.  In particular Nordic QSPI flash
API requires that addresses, sizes, and buffers all be 4-byte aligned.
Align the ready/copy buffer to satisfy this requirement.

Signed-off-by: Peter A. Bigot <pab@pabigot.com>
@pabigot pabigot removed the DNM This PR should not be merged (Do Not Merge) label Apr 20, 2020
@pabigot pabigot force-pushed the pabigot/20200418b branch from e502a27 to 3626fd2 Compare April 20, 2020 22:17
@pabigot pabigot marked this pull request as ready for review April 20, 2020 22:18
@pabigot
Copy link
Collaborator Author

pabigot commented Apr 28, 2020

This isn't getting attention, so close the separate PR and have it reviewed in the context of #24696

@pabigot pabigot closed this Apr 28, 2020
@pabigot pabigot deleted the pabigot/20200418b branch May 4, 2020 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Flash area: USB Universal Serial Bus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant