Skip to content

Macro to load LP core code #779

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 12 commits into from
Sep 14, 2023
Merged

Macro to load LP core code #779

merged 12 commits into from
Sep 14, 2023

Conversation

bjoernQ
Copy link
Contributor

@bjoernQ bjoernQ commented Sep 4, 2023

This adds a macro for ESP32-C6 LP core to take configured pins.
On the HP core side there is a macro to load that code into LP ram and execute it.

Most probably the error handling should get improved

@bjoernQ
Copy link
Contributor Author

bjoernQ commented Sep 4, 2023

No idea what the problem with the checkout action is about.

@bjoernQ bjoernQ marked this pull request as draft September 4, 2023 16:02
@bjoernQ bjoernQ marked this pull request as ready for review September 5, 2023 11:07
Copy link
Contributor

@JurajSadel JurajSadel left a comment

Choose a reason for hiding this comment

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

LGTM!

@jessebraham
Copy link
Member

jessebraham commented Sep 11, 2023

Would you mind explaining the changes to the CI workflow? I don't quite follow. If we are changing the MSRV back for whatever reason, could you please update the README to reflect this as well?

I'm also not super excited about checking binaries in, if we're just doing that for CI then I'd rather just build it in CI I think.

@MabezDev
Copy link
Member

Would you mind explaining the changes to the CI workflow? I don't quite follow. If we are changing the MSRV back for whatever reason, could you please update the README to reflect this as well?

I think the idea is that RUSTC_BOOTSTRAP allows unstable features to be used with a stable compiler, which seems to be a good fit for this msrv check where we use -Zbuild-std

@jessebraham
Copy link
Member

Ahh gotcha, I didn't really find much in an initial search for that variable. Thanks!

@bjoernQ
Copy link
Contributor Author

bjoernQ commented Sep 13, 2023

Thanks for the review! Any objections or does someone want more time to look into this? Otherwise, I guess I'll rebase and merge it end of today or tomorrow morning

Naming of esp32c6-lp-procmacros is probably debatable since we might be able to the same code for the ULPs and upcoming LP cores, too. But we can still rename it once that happens

@jessebraham
Copy link
Member

I really would prefer not to have binaries checked into git if at all possible, as I previously mentioned. Other than that I think it looks okay, haven't dug into it in too much detail.

@bjoernQ
Copy link
Contributor Author

bjoernQ commented Sep 13, 2023

I can make sure to build the LP examples in the same job before building the HP examples. Will try that

@bjoernQ bjoernQ mentioned this pull request Sep 14, 2023
@bjoernQ
Copy link
Contributor Author

bjoernQ commented Sep 14, 2023

Not having the binary checked in makes the CI builds a bit more complicated (but not too much) and needs the user to follow the instructions in the example. But not a big deal

@bjoernQ bjoernQ merged commit a0e5737 into esp-rs:main Sep 14, 2023
@bjoernQ bjoernQ deleted the esp32c6-lp-macros branch September 14, 2023 10:16
playfulFence pushed a commit to playfulFence/esp-hal that referenced this pull request Sep 14, 2023
* Macro to load LP core code

* Fix imports, add CHANGELOG.md entry

* Avoid code warning

* Omit path from function signature

* More error checking

* Clippy fix

* Include the ELF used by the lp_core_basic example

* Make object dependency optional

* Use 1.65 for RISCV MSRV check

* Use RUSTC_BOOTSTRAP for RISCV MSRV check

* Remove the pre-compiled LP core example

* Pin toml_edit in esp32c6-lp-hal-procmacro
SergioGasquez pushed a commit to SergioGasquez/esp-hal that referenced this pull request Sep 22, 2023
* Macro to load LP core code

* Fix imports, add CHANGELOG.md entry

* Avoid code warning

* Omit path from function signature

* More error checking

* Clippy fix

* Include the ELF used by the lp_core_basic example

* Make object dependency optional

* Use 1.65 for RISCV MSRV check

* Use RUSTC_BOOTSTRAP for RISCV MSRV check

* Remove the pre-compiled LP core example

* Pin toml_edit in esp32c6-lp-hal-procmacro
playfulFence pushed a commit to playfulFence/esp-hal that referenced this pull request Sep 26, 2023
* Macro to load LP core code

* Fix imports, add CHANGELOG.md entry

* Avoid code warning

* Omit path from function signature

* More error checking

* Clippy fix

* Include the ELF used by the lp_core_basic example

* Make object dependency optional

* Use 1.65 for RISCV MSRV check

* Use RUSTC_BOOTSTRAP for RISCV MSRV check

* Remove the pre-compiled LP core example

* Pin toml_edit in esp32c6-lp-hal-procmacro
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.

4 participants