-
Notifications
You must be signed in to change notification settings - Fork 52
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
Adjust the pin definitions for the hardware section of the Datalogger board. #237
Conversation
@@ -96,9 +96,6 @@ extern void wait_for_debug(void); | |||
#define A6 AMB_D9 // PB_3 3.3V CH6 | |||
|
|||
/* LED pin mapping */ | |||
#define LED_G AMB_D3 // Green | |||
#define LED_R AMB_D2 // Red | |||
#define LED_B AMB_D1 // Blue | |||
#define LED_BUILTIN LED_G // Green |
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 will put "NC" for "LED_BUILTIN", as there is no "LED_G" anymore.
#define LED_G AMB_D3 // Green | ||
#define LED_R AMB_D2 // Red | ||
#define LED_B AMB_D1 // Blue | ||
#define LED_BUILTIN LED_G // Green |
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 have changed LED_BUILTIN as NC for the LED_G no more exist.
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.
Please assign AMB_D3 to be used for LED_BUILTIN. It is used in my circuit. I did not notice the issue, sorry.
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.
Please assign AMB_D3 to be used for LED_BUILTIN. It is used in my circuit. I did not notice the issue, sorry.
Lets refer to the latest commit 666c2a9
By completing this PR sufficiently, please help us to review and improve the quality of this PR and Release Notes
Description of Change
Please describe your proposed Pull Request and it's impact. \
Adjust the pin definitions
Tests and Environments
Please describe on what Hardware, SDK, IDE, OS that tested this Pull Request and how if necessary. \