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

added Mbed compatibilty #30

Merged

Conversation

JojoS62
Copy link
Contributor

@JojoS62 JojoS62 commented Jul 4, 2020

This is a PR for making the MD_MAX72XX compatible with Mbed-OS: https://github.com/ARMmbed/mbed-os

I like the MD_xx Libraries, but I'm using mainly the MBed-os and wanted to make it compatible. So it can be used with a lot of Cortex-M that are supported by Mbed.

It adds macros for Arduino compatibility and uses Mbed Objects for SPI and DigitalOut (for chip select). Differences with Arduino are solved by adding conditional compiling, using the presence of the __MBED__ define.
It has been tested with Mbed-os 6.1.0, but should work also with earlier releases.

This first release uses only the hardware SPI object from Mbed.

Don't know if Mbed compatibility was ever requested, I don't worry when you don't want to maintain this addition.

add .mbedignore to prevent non Mbed files from compiling
added Arduino compatibilty macros for Mbed
first release, supports only HW SPI for now
tested with Mbed 6.1.0
@MajicDesigns
Copy link
Owner

There has not been a request for this at all. I will put this into the main branch but I am concerned that I will not be able to test or update in future versions.

@MajicDesigns MajicDesigns merged commit 5ec1b38 into MajicDesigns:master Jul 6, 2020
@JojoS62
Copy link
Contributor Author

JojoS62 commented Jul 6, 2020

Great, thanks for your support!
I will provide some Example in my Repo. I've added the Mbed stuff also to the MD_Parola Lib and it is working already. I want to do some more testing and add examples for this too.

@MajicDesigns
Copy link
Owner

@JojoS62 Can you please review the error that is identified in issue #38 and suggest a solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants