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

risc-v/mpfs: add CoreMMC support #9548

Merged
merged 1 commit into from Jun 19, 2023

Conversation

eenurkka
Copy link
Contributor

This adds support for the CoreMMC v3.1 FPGA driver as described in Microchip Handbook HB0510. The driver doesn't support DMA.

Summary

This adds the driver for CoreMMC FPGA block. This may be added to the Polarfire or related FPGA.

Impact

Testing

Hardware with the CoreMMC IP configured properly. FAT32 file system testing with file access (read / writes / copying / appending files).

*
****************************************************************************/

static void mpfs_modifyreg8(uintptr_t addr, uint8_t clearbits,
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it is time for void modifyreg8(unsigned int addr, uint8_t clearbits, uint8_t setbits) interface placed in arch/risc-v/src/common? I see mpfs_modifyreg8 in arch/risc-v/src/mpfs/mpfs_usb.c that duplicates 95% of this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, fixed all the others but will come back to this, as it'd affect multiple drivers / files.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sure. That is more an improvement and should be done in a separate PR.

arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
arch/risc-v/src/mpfs/mpfs_coremmc.c Outdated Show resolved Hide resolved
This adds support for the CoreMMC v3.1 FPGA driver as described
in Microchip Handbook HB0510. The driver doesn't support DMA.

Signed-off-by: Eero Nurkkala <eero.nurkkala@offcode.fi>
@xiaoxiang781216 xiaoxiang781216 merged commit 4494e75 into apache:master Jun 19, 2023
26 checks passed
@jerpelea jerpelea added this to To-Add in Release Notes - 12.2.0 Jun 26, 2023
@jerpelea jerpelea moved this from To-Add to In Progress in Release Notes - 12.2.0 Jun 26, 2023
@eenurkka eenurkka deleted the mpfs-coremmc branch November 21, 2023 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants