warning: unused `core::result::Result` that must be used
   --> src/asynch/runner.rs:511:21
    |
511 |                     self.handle_urc(event).await;
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
    = note: `#[warn(unused_must_use)]` on by default
help: use `let _ = ...` to ignore the resulting value
    |
511 |                     let _ = self.handle_urc(event).await;
    |                     +++++++

warning: redundant closure
   --> src/asynch/state.rs:335:22
    |
335 |             .map_err(|x| Error::SubscriberOverflow(x))?;
    |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `Error::SubscriberOverflow`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure

warning: redundant closure
   --> src/asynch/state.rs:222:22
    |
222 |             .map_err(|x| Error::SubscriberOverflow(x))?;
    |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `Error::SubscriberOverflow`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure

warning: redundant closure
   --> src/asynch/state.rs:209:22
    |
209 |             .map_err(|x| Error::SubscriberOverflow(x))?;
    |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `Error::SubscriberOverflow`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure

warning: unneeded `return` statement
   --> src/asynch/runner.rs:910:9
    |
910 |         return Err(Error::ContextActivationTimeout);
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
help: remove `return`
    |
910 -         return Err(Error::ContextActivationTimeout);
910 +         Err(Error::ContextActivationTimeout)
    |

warning: unneeded `return` statement
   --> src/asynch/runner.rs:689:9
    |
689 |         return Ok(false);
    |         ^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
    = note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
    |
689 -         return Ok(false);
689 +         Ok(false)
    |

warning: redundant guard
   --> src/asynch/runner.rs:358:43
    |
358 |                 Ok(PinStatus { code }) if code == PinStatusCode::Ready => {
    |                                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_guards
    = note: `#[warn(clippy::redundant_guards)]` on by default
help: try
    |
358 -                 Ok(PinStatus { code }) if code == PinStatusCode::Ready => {
358 +                 Ok(PinStatus { code: PinStatusCode::Ready }) => {
    |

warning: this let-binding has unit value
   --> src/asynch/runner.rs:102:9
    |
102 | /         let alive = match self.at.send(&AT).await {
103 | |             Ok(_) => {
104 | |                 return Ok(true);
105 | |             }
...   |
108 | |             }
109 | |         };
    | |__________^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
    = note: `#[warn(clippy::let_unit_value)]` on by default
help: omit the `let` binding
    |
102 ~         match self.at.send(&AT).await {
103 +             Ok(_) => {
104 +                 return Ok(true);
105 +             }
106 +             Err(err) => {
107 +                 return Err(Error::Atat(err));
108 +             }
109 +         };
    |

warning: redundant closure
  --> src/asynch/control.rs:59:22
   |
59 |             .map_err(|e| Error::Atat(e))
   |                      ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `Error::Atat`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure

warning: redundant closure
  --> src/asynch/control.rs:50:22
   |
50 |             .map_err(|e| Error::Atat(e))
   |                      ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `Error::Atat`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
   = note: `#[warn(clippy::redundant_closure)]` on by default

warning: method `init` is never used
  --> src/asynch/control.rs:22:25
   |
17 | impl<'a, AT: AtatClient> Control<'a, AT> {
   | ---------------------------------------- method in this implementation
...
22 |     pub(crate) async fn init(&mut self) -> Result<(), Error> {
   |                         ^^^^

warning: function `kill_time` is never used
  --> src/module_timing.rs:56:8
   |
56 | pub fn kill_time() -> Option<Duration> {
   |        ^^^^^^^^^
   |
   = note: `#[warn(dead_code)]` on by default

warning: unused variable: `profile_id`
   --> src/asynch/runner.rs:821:9
    |
821 |         profile_id: ProfileId,
    |         ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_profile_id`

warning: unused variable: `model_id`
   --> src/asynch/runner.rs:190:13
    |
190 |         let model_id = self.at.send(&GetModelId).await?;
    |             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_model_id`

warning: unused variable: `alive`
   --> src/asynch/runner.rs:102:13
    |
102 |         let alive = match self.at.send(&AT).await {
    |             ^^^^^ help: if this is intentional, prefix it with an underscore: `_alive`
    |
    = note: `#[warn(unused_variables)]` on by default

warning: variable does not need to be mutable
  --> src/asynch/mod.rs:64:9
   |
64 |     let mut control = Control::new(state_ch, AtHandle(&state.at_handle));
   |         ----^^^^^^^
   |         |
   |         help: remove this `mut`

warning: variable does not need to be mutable
  --> src/asynch/mod.rs:54:9
   |
54 |     let mut runner = Runner::new(
   |         ----^^^^^^
   |         |
   |         help: remove this `mut`
   |
   = note: `#[warn(unused_mut)]` on by default

warning: unused import: `crate::asynch::state::OperationState::DataEstablished`
 --> src/asynch/state.rs:7:5
  |
7 | use crate::asynch::state::OperationState::DataEstablished;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `crate::error::Error::Network`
  --> src/asynch/runner.rs:49:5
   |
49 | use crate::error::Error::Network;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused imports: `Ipv4Addr`, `Ipv6Addr`
  --> src/asynch/runner.rs:45:18
   |
45 | use no_std_net::{Ipv4Addr, Ipv6Addr};
   |                  ^^^^^^^^  ^^^^^^^^

warning: unused import: `heapless::String`
  --> src/asynch/runner.rs:44:5
   |
44 | use heapless::String;
   |     ^^^^^^^^^^^^^^^^

warning: unused import: `crate::error::GenericError::Timeout`
  --> src/asynch/runner.rs:38:5
   |
38 | use crate::error::GenericError::Timeout;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `crate::command::psn::SetPacketSwitchedAction`
  --> src/asynch/runner.rs:33:5
   |
33 | use crate::command::psn::SetPacketSwitchedAction;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `crate::command::psn::types::PacketSwitchedNetworkDataParam`
  --> src/asynch/runner.rs:32:5
   |
32 | use crate::command::psn::types::PacketSwitchedNetworkDataParam;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `crate::command::psn::types::PacketSwitchedAction`
  --> src/asynch/runner.rs:31:5
   |
31 | use crate::command::psn::types::PacketSwitchedAction;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^