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

[REQUEST] Backport improvements from "CC1352 Flasher" fork by the BeagleConnect project #154

Open
Hedda opened this issue Jan 9, 2024 · 11 comments

Comments

@Hedda
Copy link
Contributor

Hedda commented Jan 9, 2024

BeagleConnect project has a fork of cc2538-bsl called "cc1352-flasher" written by @jadonk and they added a few improvements:

https://openbeagle.org/beagleconnect/cc1352-flasher

https://git.beagleboard.org/beagleconnect/cc1352-flasher

https://pypi.org/project/cc1352-flasher/

@JelmerT suggest you consider backporting their improvements if they do not themself submit pull requests to upstream repo.

Note! While the cc1352-flasher code is also available on GitHub it looks to be lagging behind the code in openbeagle.org repo:

https://github.com/beagleboard/cc1352-flasher

@Hedda
Copy link
Contributor Author

Hedda commented Jan 9, 2024

FYI, there are also these additional downstream forks of that "cc1352-flasher" code which have added further improvements, ex:

https://github.com/mercenaruss/cc1352-flasher by @mercenaruss

as well as in turn forks of it, like:

https://github.com/tube0013/tubeszb_addons/tree/main/tzb-cc2652-flasher by @tube0013

@jadonk
Copy link

jadonk commented Jan 9, 2024

I'm happy to put together properly formatted pull requests for specific features. I'd rebased several times in hopes of getting things cleaned up to submit here, but I got the impression there wasn't interest in continuing to maintain it. I am only interested in cc1352 (p1/p7) flashing on Beagle platforms and making it easier to use for Beagle users, specifically with Zephyr.

@Ayush1325
Copy link
Contributor

I will also be happy to help with the effort and improve the flasher.

I have created a argparse PR for a start.

@JelmerT
Copy link
Owner

JelmerT commented Mar 14, 2024

@Hedda @jadonk @Ayush1325 I'm more than happy to backport all these changes back into this script! feel free to submit a PR. Could take a little time for me to check them, but will get merged eventually!

@jadonk
Copy link

jadonk commented Mar 14, 2024

Nice. What order should we stage updates? We recently updated to put it on pypi.

@jadonk
Copy link

jadonk commented Mar 14, 2024

I've been thinking I want to set some defaults based on the name or symlink of the executable script.

@Hedda
Copy link
Contributor Author

Hedda commented Aug 5, 2024

@jadonk @Ayush1325 @git-developer Did you see Texas Instruments forked cc2538-bsl and are including it in SimpleLink SDK?

Originally posted by @Hedda in #160 (comment)

The SIMPLELINK-CC13XX-CC26XX-SDK 7.41.00.17 contains a modified version of cc2538-bsl with support for newer TI chips

Do you have link to the source code?

Here is the link for 7.40.00.77 update (from 5 months ago) but I could not find links to new 7.41.00.17 update on GitHub:

https://github.com/TexasInstruments/simplelink-lowpower-f2-sdk/tree/main/source/third_party/cc2538-bsl

Maybe the 7.41.00.17 update does not contain any updates to the third-party cc2538-bsl code?

https://github.com/TexasInstruments/simplelink-lowpower-f2-sdk/blob/main/source/third_party/cc2538-bsl/ti-python-sbl.py

@JelmerT Any chance you could have a look and create a patch to this upstream repo if those changes are of interest?

PS: Cool that Texas Instruments is using cc2538-bsl, even if it is a modified fork instead of submitting patches to upstream :)

@jadonk
Copy link

jadonk commented Aug 5, 2024

Didn't see that. Thanks for the news. This is also a ping for me to work on the PRs.

@Ayush1325
Copy link
Contributor

So, just for reference there are now 2 more independent flashing implementations now (mostly for cc1352p7 but can be easily extended):

  1. Linux kernel: https://lore.kernel.org/r/[email protected]
  2. bb-imager-rs: https://openbeagle.org/ayush1325/bb-imager-rs

@Hedda
Copy link
Contributor Author

Hedda commented Aug 14, 2024

FYI, @tl-sl has worked on a cc2538-bsl based web flasher with support for CC2674P P10 (CC2674P10) for @smlight-tech now:

That webflasher depends on their "smlight_cc_flasher" project which in turn is another fork of JelmerT's cc2538-bsl project:

@darkxst wrote smlight-cc-flasher is based on a heavily modified + improved fork of JelmerT's cc2538-bsl project, tTested with the new experimental SMLIGHT SLZB-07p10 Zigbee Coordinator USB adapter based in CC2674P P10 (CC2674P10) they released:

PS: Please also see related issue tracker requesting support for CC2674 (CC2674P/CC2674R) and CC1354 (CC1354P) compatibility:

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

No branches or pull requests

4 participants