-
Notifications
You must be signed in to change notification settings - Fork 614
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
Rockpi S #1110
base: master
Are you sure you want to change the base?
Rockpi S #1110
Conversation
api/mraa/types.h
Outdated
MRAA_ADLINK_IPI = 21, /**< Adlink Industrial PI */ | ||
MRAA_ADLINK_LEC_AL = 22, /**< Adlink LEC-AL*/ | ||
MRAA_ADLINK_LEC_AL_AI = 23, /**< Adlink LEC-AL*/ | ||
MRAA_UPXTREME = 24, /**< The UPXTREME Board */ | ||
MRAA_INTEL_ILK = 25, /**< Intel Learning Kit */ | ||
MRAA_SIEMENS_IOT2050 = 26, /**< Siemens IOT2050 board */ | ||
MRAA_ROCKPIS = 21, /**< Radxa ROCK PI S */ | ||
MRAA_ADLINK_IPI = 22, /**< Adlink Industrial PI */ | ||
MRAA_ADLINK_LEC_AL = 23, /**< Adlink LEC-AL*/ | ||
MRAA_ADLINK_LEC_AL_AI = 24, /**< Adlink LEC-AL*/ | ||
MRAA_UPXTREME = 25, /**< The UPXTREME Board */ | ||
MRAA_INTEL_ILK = 26, /**< Intel Learning Kit */ | ||
MRAA_SIEMENS_IOT2050 = 27, /**< Siemens IOT2050 board */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd suggest AGAINST changing existing constant's value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, Please dont ever change these constants. There might be downstream code that depends on it.
Added support for Rock Pi S platform.