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

Added WisCore RAK4631 #322

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Added WisCore RAK4631 #322

wants to merge 2 commits into from

Conversation

hallard
Copy link

@hallard hallard commented Feb 15, 2024

Checklist

By completing this PR sufficiently, you help us to review this Pull Request quicker and also help improve the quality of Release Notes

  • Please provide specific title of the PR describing the change
  • Please provide related links (eg. Issue which will be closed by this Pull Request)
  • If you are adding an new boards, please make sure
    • Provide link to your allocated VID/PID if applicable
    • Add your board to action ci in correct workflow and alphabet order for release binary
    • UF2_BOARD_ID in your board.h follow correct format from uf2 specs

This checklist items that are not applicable to your PR can be deleted.


Description of Change

Added RAK Wireless WisCore RAK4631 original change are coming from RAK
https://github.com/RAKWireless/WisBlock/tree/master/bootloader/RAK4630

@hallard hallard changed the title Rak4631 Added WisCore RAK4631 Feb 15, 2024
Copy link
Member

@hathach hathach left a comment

Choose a reason for hiding this comment

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

please update pr per review

//--------------------------------------------------------------------+
// USB
//--------------------------------------------------------------------+
#define USB_DESC_VID 0x239A
Copy link
Member

Choose a reason for hiding this comment

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

this VID and PID is already used for Adafruit product

//------------- UF2 -------------//
#define UF2_PRODUCT_NAME "WisBlock RAK4631 Board"
#define UF2_VOLUME_LABEL "RAK4631"
#define UF2_BOARD_ID "WisBlock-RAK4631-Board"
Copy link
Member

Choose a reason for hiding this comment

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

incorrect board_id format

@hathach hathach mentioned this pull request Feb 19, 2024
6 tasks
@hathach hathach marked this pull request as draft May 9, 2024 13:25
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