Skip to content

Bunch of refactorings / Added support for Arduino Nano RP2040 Connect #20

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

Merged
merged 9 commits into from
Apr 23, 2021

Conversation

cmaglie
Copy link
Member

@cmaglie cmaglie commented Apr 14, 2021

  • Made flasher.Close handling more consistent
  • Removed DumpAndFlash function (it was not really working)
  • Simplified Programmer interface
  • Added support for Arduino RP2040 Connect

I'm waiting for the final release of the firmware so I'm pushing as a draft in the meantime.

@cmaglie cmaglie requested a review from facchinm April 14, 2021 09:26
@umbynos umbynos marked this pull request as ready for review April 23, 2021 09:46
@umbynos umbynos self-requested a review April 23, 2021 09:46
Copy link
Contributor

@umbynos umbynos left a comment

Choose a reason for hiding this comment

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

As discussed with @cmaglie and @rsora we decided to merge this PR and include the firmware for rp2040 when it will be released in order to test everything

@umbynos umbynos merged commit 2fc1d74 into main Apr 23, 2021
umbynos added a commit that referenced this pull request Apr 23, 2021
…#20)

* Close flasher after completing operations

* Refactored and simplified Programmer

- removed DumpAndFlash (it doesn't work reliably)
- simplified ctx message passing

* Removed now unused DumpAndFlash methods

* Added Arduino Nano RP2040 Connect support

* Fix upload problems

* Fixed bossac init... ooops

* fix import removed by error

* bump arduino-cli -> include the latest updates in serialutils.go

Co-authored-by: Umberto Baldi <[email protected]>
Co-authored-by: Umberto Baldi <[email protected]>
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