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

Compile failure esp32s3 esp-now e.g.: invalid register name #270

Closed
Ben-PH opened this issue Sep 22, 2023 · 2 comments
Closed

Compile failure esp32s3 esp-now e.g.: invalid register name #270

Ben-PH opened this issue Sep 22, 2023 · 2 comments

Comments

@Ben-PH
Copy link
Contributor

Ben-PH commented Sep 22, 2023

git clone https://github.com/esp-rs/esp-wifi.git
cd esp-wifi/examples-esp32s3
cargo build --example esp_now --release --features "esp-now"

error:

❯ cargo build --example esp_now --release --features "esp-now"
   Compiling esp-wifi v0.1.0 (/home/ben/rust/esp-stuff/esp-wifi/esp-wifi)
   Compiling examples-esp32s3 v0.1.0 (/home/ben/rust/esp-stuff/esp-wifi/examples-esp32s3)
error: invalid register name
   --> esp-wifi/src/timer_esp32s3.rs:196:27
    |
196 |         core::arch::asm!("wsr.intset  {0}", in(reg) intr, options(nostack));
    |                           ^
    |
note: instantiated into assembly here
   --> <inline asm>:1:2
    |
1   |     wsr.intset  a8
    |     ^

error: could not compile `esp-wifi` (lib) due to previous error

then:

git checkout 87687c82db75bb7378957c48d990602f423c7d36

build works. checkout the next commit that edits the file:

git checkout 5d0c416c0d6937acf8813306a419fb270c830715
...and the error re-appears.

seems to be this diff:

5d0c416#diff-526af10fb97077f069ba729e126156e3b4a38b7650f98c5c92835d2e063bb456L196-R196

@MabezDev
Copy link
Member

You need to update your esp compiler version, inset was added in 1.70 IIRC.

@github-project-automation github-project-automation bot moved this from Todo to Done in esp-rs Sep 22, 2023
@Ben-PH
Copy link
Contributor Author

Ben-PH commented Sep 22, 2023

You need to update your esp compiler version, inset was added in 1.70 IIRC.

...I should have checked out the PR comments: #248 (comment)

I'm on v1.70, it's v1.72 I need

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants