-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Wio 3G] Modified pin map #9556
Conversation
@ytsuboi, thank you for your changes. |
// Not connected | ||
NC = (int)0xFFFFFFFF, | ||
|
||
PA_0 = 0x00, |
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.
misaligned code
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.
Approach changed.
LED4 = PB_14, | ||
LED_RED = LED2, | ||
USER_BUTTON = PC_13, | ||
LED1 = NC, |
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.
Why are these being changed to NC? Can you add this to the tcommit message?
Why and how are this being modified, how this fixes the bug?
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.
The LED on Wio 3G is not traditional LED, so LED1-4 doesn't have any meaning. However, we need to have definition of LED1-4. This is the reason why we tried to change like this. ( Finally, we changed approach for this issue. )
* Remove unused Peripheral Pins definitions and add UART_1 TXD/RXD pins (fix #9026) for the Wio 3G board * Remove/Fix unused PinName definitions for the Wio 3G board from the PinNames enum
CI started |
Test run: SUCCESSSummary: 12 of 12 test jobs passed |
Description
Fixing issue #9026
Pull request type
Reviewers