-
Notifications
You must be signed in to change notification settings - Fork 178
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
add initial working m5stack_stamps3 #374
Conversation
* THE SOFTWARE. | ||
*/ | ||
|
||
// #ifndef ADAFRUIT_FEATHER_ESP32S3_TFT_H_ |
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 remove those unused, and add header defie
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.
header define still not added
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.
#ifndef M5STACK_STAMPS3_BOARD_H_
#define M5STACK_STAMPS3_BOARD_H_``
put those under the license
#endif
put this at the end
#define USB_PRODUCT "Stamp S3" | ||
|
||
#define UF2_PRODUCT_NAME USB_MANUFACTURER " " USB_PRODUCT | ||
#define UF2_BOARD_ID "ESP32S3" |
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.
incorrect board-id format
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 updated the board folder from cardputer to stamps3 to better match the 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.
board-id still not in correct format
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.
#define UF2_BOARD_ID "ESP32S3-StampS3-01"
Here is the correct board id, i didnt want to make another PR
to better fit with the actual board being used here.
thank you @hathach, I think this is all set now, please let me know if I missed anything. |
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.
2 of change requests are still not fixed
@jamesjnadeau In leu of making another PR I commented the fixes for the open changes |
@hathach this should now be fixed, thanks @panguin6010 for the help! |
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.
look good
Checklist
By completing this PR sufficiently, you help us to review this Pull Request quicker and also help improve the quality of Release Notes
UF2_BOARD_ID
in your board.h follow correct format from uf2 specsThis checklist items that are not applicable to your PR can be deleted.
Description of Change
Adding setup for m5 stack stamp s3