-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
boards: arm: Add BeaglePlay CC1352 #58877
Conversation
See TI application note SWRA640G section 6.4. "Tuning the HF Oscillator". Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
See TI application note SWRA640G section 6.4. "Tuning the HF Oscillator". Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
Add support for BeaglePlay, a TI AM6254 SoC based development board with a CC1352P7 wireless microcontroller for supporting software defined 2.4GHz and SubGHz wireless protocols. Support for running Zephyr on the quad-A53 SoC or the programmable M4 on the SoC would be provided separately. See https://beagleplay.org for details. Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
28cb071
to
23ed8a7
Compare
Add support for BeaglePlay, a TI AM6254 SoC based development board with a CC1352P7 wireless microcontroller for supporting software defined 2.4GHz and SubGHz wireless protocols. Support for running Zephyr on the quad-A53 SoC or the programmable M4 on the SoC would be provided separately. See https://beagleplay.org for details. Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
23ed8a7
to
554a9ad
Compare
@jadonk, @vaishnavachath - subtle change with Documentation issue is unrelated and should be solved by a rebase (when there is a fix in). |
Feature freeze right now in the release cycle, so 10 days before this can be merged. |
static int board_antenna_init(const struct device *dev) | ||
{ | ||
ARG_UNUSED(dev); | ||
|
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.
@jadonk can you please drop const struct device *dev
?
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 guess this was just changed to void
somewhat recently
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.
@Ayush1325 Ping, since you are actively working on BeagleConnect freedom and Beagleplay, can you fixup the minor changes required in this PR?
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.
@vaishnavachath This PR totally slipped by radar. I already do have a PR for beagleplay on-board cc1352: #64718. Other than caparray delta (which I am planning to add after #65048, my PR also contains everything rebased on upstream zephyr
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.
No problem @Ayush1325 , can you add the Caparray delta to #65048 (new commit) rather than creating a new PR for that alone? and please confirm if this can be closed as duplicate.
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.
@vaishnavachath Yes, I will add caparray to #65048
…d/zephyr into add-beagleplay-cc1352
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
This is under arm, not arm64, because it is for the M4 on the CC1352P7, not the A53s on AM6254.
For information about BeaglePlay, see https://beagleplay.org.