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

Re-export qingke-rt::entry #13

Closed
wants to merge 1 commit into from
Closed

Re-export qingke-rt::entry #13

wants to merge 1 commit into from

Conversation

barafael
Copy link

Without this, I believe it is not possible for users to use qingke-rt themselves, because the qingke-rt crate defines global symbols which the linker does not like.

Without this, I believe it is not possible for users to use qingke-rt themselves,
because the qingke-rt crate defines global symbols which the linker does not like.
@andelf
Copy link
Contributor

andelf commented May 12, 2024

Could you provide your Cargo.toml dependency and fn main config?
qingke-rt is designed to be used with ch32-hal, so a re-export is not necessary.

@barafael
Copy link
Author

@andelf
Copy link
Contributor

andelf commented May 13, 2024

With the following modification, everything compiles normally:

diff --git a/Cargo.toml b/Cargo.toml
index b45c2d3..9a329af 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -4,11 +4,13 @@ version = "0.1.0"
 edition = "2021"

 [dependencies]
-ch32-hal = { git = "https://github.com/barafael/ch32-hal", features = [
+ch32-hal = { git = "https://github.com/ch32-rs/ch32-hal", features = [
     "ch32v003f4u6",
     "embassy",
     "time-driver-tim2",
+    "rt"
 ] }
+qingke-rt = "0.2.0"

 embassy-executor = { version = "0.5.0", features = [
     "nightly",
diff --git a/src/main.rs b/src/main.rs
index 734ca01..df7057b 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -33,7 +33,7 @@ bind_interrupts!(
     }
 );

-#[embassy_executor::main(entry = "ch32_hal::entry")]
+#[embassy_executor::main(entry = "qingke_rt::entry")]
 async fn main(_spawner: Spawner) -> ! {

@barafael
Copy link
Author

great, it's like you said! Thanks :) closing

@barafael barafael closed this May 14, 2024
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