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

New Support for ODROID-M1S #739

Merged
merged 1 commit into from
Nov 15, 2023
Merged

Conversation

how2flow
Copy link
Contributor

i2c0, spi and uart are not available as default values because they are provided as aliases.

i2c0, spi and uart are not available as default values
because they are provided as aliases.

Signed-off-by: Steve Jeong <steve@how2flow.net>
@how2flow
Copy link
Contributor Author

how2flow commented Nov 14, 2023

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

@how2flow
Copy link
Contributor Author

how2flow commented Nov 15, 2023

Because you added a new microcontroller, they should be also added to: https://github.com/adafruit/Adafruit_Blinka/blob/main/src/microcontroller/__init__.py https://github.com/adafruit/Adafruit_Blinka/blob/main/src/microcontroller/pin.py

ODROID-M1S is rk3566 SBC.
And rk3566 is already included in the code.

from adafruit_blinka.microcontroller.rockchip.rk3566 import *

from adafruit_blinka.microcontroller.rockchip.rk3566.pin import *

@how2flow
Copy link
Contributor Author

Request review is my mistake..

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Oh you're right. I didn't notice you only added pins. Looks good then.

@makermelissa makermelissa merged commit b676b2e into adafruit:main Nov 15, 2023
1 check passed
@how2flow how2flow deleted the odroidm1s branch November 16, 2023 01:07
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.

2 participants