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

1.fix compile error for esp32c3 #377

Closed
wants to merge 5 commits into from
Closed

1.fix compile error for esp32c3 #377

wants to merge 5 commits into from

Conversation

zrz4066
Copy link

@zrz4066 zrz4066 commented Dec 6, 2023

Fixed the following compile error

error[E0599]: no method named cpu_intr_from_cpu_3 found for reference &esp32c3::system::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\timer\riscv.rs:74:14
|
73 | / (&*SystemPeripheral::PTR)
74 | | .cpu_intr_from_cpu_3()
| | -^^^^^^^^^^^^^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

error[E0599]: no method named cpu_intr_from_cpu_3 found for reference &esp32c3::system::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\timer\riscv.rs:92:14
|
91 | / (&*SystemPeripheral::PTR)
92 | | .cpu_intr_from_cpu_3()
| | -^^^^^^^^^^^^^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

error[E0599]: no method named cpu_int_enable found for struct esp32c3::interrupt_core0::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\wifi\os_adapter_esp32c3.rs:6:14
|
5 | / (*peripherals::INTERRUPT_CORE0::PTR)
6 | | .cpu_int_enable()
| | -^^^^^^^^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

error[E0599]: no method named cpu_int_enable found for struct esp32c3::interrupt_core0::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\wifi\os_adapter_esp32c3.rs:14:14
|
13 | / (*peripherals::INTERRUPT_CORE0::PTR)
14 | | .cpu_int_enable()
| | -^^^^^^^^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

error[E0599]: no method named dig_pwc found for reference &esp32c3::rtc_cntl::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\common_adapter\common_adapter_esp32c3.rs:42:14
|
41 | / rtc_cntl
42 | | .dig_pwc()
| | -^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

error[E0599]: no method named wifi_rst_en found for reference &esp32c3::apb_ctrl::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\common_adapter\common_adapter_esp32c3.rs:46:14
|
45 | / syscon
46 | | .wifi_rst_en()
| | -^^^^^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

error[E0599]: no method named wifi_rst_en found for reference &esp32c3::apb_ctrl::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\common_adapter\common_adapter_esp32c3.rs:49:14
|
48 | / syscon
49 | | .wifi_rst_en()
| | -^^^^^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

error[E0599]: no method named dig_iso found for reference &esp32c3::rtc_cntl::RegisterBlock in the current scope
--> D:\Rust.cargo\git\checkouts\esp-wifi-836f3b2af57fa847\d46dfc4\esp-wifi\src\common_adapter\common_adapter_esp32c3.rs:53:14
|
52 | / rtc_cntl
53 | | .dig_iso()
| | -^^^^^^^-- help: remove the arguments
| | ||
| |_____________|field, not a method
|

@bjoernQ
Copy link
Contributor

bjoernQ commented Dec 6, 2023

Thank you!

The issue is that the current main is targeting the (almost) current main of esp-hal where some things changed in an incompatible way.

So, the idea is to use this as a git dependency and use a suitable commit of esp-hal or use a published version of esp-hal together with a suitable published version of esp-wifi from crates.io: https://crates.io/crates/esp-wifi

Before we released to crates.io our policy (more or less) was to always target the currently released esp-hal - that is changed now

Given that I think we should close this

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