Skip to content

feat: wifi power-save mode - #17

Open
ForsakenHarmony wants to merge 1 commit into
esp-rs:masterfrom
ForsakenHarmony:wifi-power-save
Open

feat: wifi power-save mode#17
ForsakenHarmony wants to merge 1 commit into
esp-rs:masterfrom
ForsakenHarmony:wifi-power-save

Conversation

@ForsakenHarmony

Copy link
Copy Markdown
Contributor

No description provided.

@ForsakenHarmony
ForsakenHarmony marked this pull request as ready for review October 26, 2021 02:32
Comment thread src/wifi.rs
unsafe {
TAKEN.lock(|taken| {
self.clear_all().unwrap();
// self.clear_all().unwrap();

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Forgotten debugging code?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

most likely, I think I ran into a crash related to this, but I don't remember for sure

Comment thread src/wifi.rs
Ok(wifi)
}

pub fn set_power_save_mode(&mut self, mode: PowerSaveMode) -> Result<(), EspError> {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just for symmetry, perhaps implement also get_power_safe_mode?

@ivmarkov
ivmarkov force-pushed the master branch 3 times, most recently from 8a8adba to 5ccb542 Compare July 5, 2023 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants