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

PSD profile and PDP context #99

Merged
merged 7 commits into from
Feb 15, 2024

Conversation

tarfu
Copy link
Contributor

@tarfu tarfu commented Jan 14, 2024

No description provided.

@tarfu
Copy link
Contributor Author

tarfu commented Jan 26, 2024

Next would be Handling URC properly that actually helps you or/and the socket stuff. Maybe also some refactoring to clean up.

@tarfu
Copy link
Contributor Author

tarfu commented Jan 28, 2024

I only tested it for sara-r5 (I just got this module here) but it compiles for the others.

@tarfu tarfu marked this pull request as ready for review January 28, 2024 11:18
@MathiasKoch
Copy link
Member

Awesome!
I can give it a test spin on my LARA-R6 & TOBY-R2 modules here, during the week, if you want me to? :)

@tarfu
Copy link
Contributor Author

tarfu commented Jan 29, 2024

Would appreciate that :)

@tarfu
Copy link
Contributor Author

tarfu commented Feb 7, 2024

Did you already have any chance to test it with one of your modules?

@tarfu
Copy link
Contributor Author

tarfu commented Feb 14, 2024

@MathiasKoch builds again with newest atat changes.

@MathiasKoch
Copy link
Member

MathiasKoch commented Feb 15, 2024

@tarfu Awesome!
Just ran your example in a slightly modified version for RP2040 and LARA-R6, and it seems to work! 🎉

1.002702 INFO  set_desired_state(PowerState::Alive)
└─ embassy_rp2040_example::__main_task_task::{async_fn#0} @ src/main.rs:153 
1.002820 INFO  Desired state: Ok(DataEstablished)
└─ ublox_cellular::asynch::runner::{impl#0}::run::{async_fn#0} @ /home/mathias/git/blackbird/ublox-cellular-rs/src/fmt.rs:160 
1.002913 INFO  vint_pin = false
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
1.002940 INFO  power_pin
└─ embassy_rp2040_example::{impl#0}::power_pin @ src/main.rs:65  
4.153069 INFO  vint_pin = true
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
5.253370 INFO  vint_pin = true
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
6.353548 INFO  vint_pin = true
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
7.453697 INFO  vint_pin = true
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
8.553843 INFO  vint_pin = true
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
8.749926 INFO  vint_pin = true
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
8.974924 WARN  Received error response (18/18): InternalError::CmeError(SIM not inserted)
└─ atat::ingress::{impl#1}::advance::{async_fn#0} @ /home/mathias/.cargo/git/checkouts/atat-1e6ba3d4ab83bd35/2d40dde/atat/src/fmt.rs:173 
10.003127 INFO  CCID: 89883030000075454733
└─ ublox_cellular::asynch::runner::{impl#0}::init_at::{async_fn#0} @ /home/mathias/git/blackbird/ublox-cellular-rs/src/fmt.rs:160 
26.013681 INFO  Ok(OperatorSelection { mode: Automatic, oper: Some(Long("3 DK EMnify")), act: Some(Lte) })
└─ embassy_rp2040_example::__main_task_task::{async_fn#0} @ src/main.rs:162 
26.031037 INFO  Ok(SignalQuality { rxlev: 99, ber: 99, rscp: 255, ecn0: 255, rsrq: 20, rsrp: 65 })
└─ embassy_rp2040_example::__main_task_task::{async_fn#0} @ src/main.rs:164 
26.249071 INFO  dns: Ok(ResolveNameIpResponse { ip_domain_string: "209.85.203.104" })
└─ embassy_rp2040_example::__main_task_task::{async_fn#0} @ src/main.rs:187 
36.249275 INFO  set_desired_state(PowerState::PowerDown)
└─ embassy_rp2040_example::__main_task_task::{async_fn#0} @ src/main.rs:190 
36.249397 INFO  Desired state: Ok(PowerDown)
└─ ublox_cellular::asynch::runner::{impl#0}::run::{async_fn#0} @ /home/mathias/git/blackbird/ublox-cellular-rs/src/fmt.rs:160 
36.249471 INFO  vint_pin = true
└─ embassy_rp2040_example::{impl#0}::vint_pin @ src/main.rs:69  
36.249507 INFO  power_pin
└─ embassy_rp2040_example::{impl#0}::power_pin @ src/main.rs:65  

If you are ready, we can merge this PR as a new milestone?

@tarfu
Copy link
Contributor Author

tarfu commented Feb 15, 2024

Yes, please merge it.

@MathiasKoch MathiasKoch merged commit dc4f74a into FactbirdHQ:feature/async Feb 15, 2024
3 of 4 checks passed
@tarfu tarfu deleted the feature/async-pdp branch February 16, 2024 22:36
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