error: useless conversion to the same type: `std::vec::Vec<hal::serial::Handler>`
  --> src/hal/serial.rs:59:23
   |
59 |             handlers: handlers.into(),
   |                       ^^^^^^^^^^^^^^^ help: consider removing `.into()`: `handlers`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
   = note: `-D clippy::useless-conversion` implied by `-D warnings`
   = help: to override `-D warnings` add `#[allow(clippy::useless_conversion)]`

error: methods `get_state` and `open_channel` are never used
   --> src/hal/serial.rs:185:12
    |
164 | impl Handler {
    | ------------ methods in this implementation
...
185 |     pub fn get_state(&self) -> HandlerState {
    |            ^^^^^^^^^
...
193 |     pub fn open_channel(&self) -> Result<SerialChannel, SerialError> {
    |            ^^^^^^^^^^^^

error: enum `HandlerState` is never used
   --> src/hal/serial.rs:146:6
    |
146 | enum HandlerState {
    |      ^^^^^^^^^^^^

error: field `inner` is never read
   --> src/hal/serial.rs:109:5
    |
108 | pub struct SerialChannel {
    |            ------------- field in this struct
109 |     inner: (broadcast::Receiver<BytesMut>, mpsc::Sender<BytesMut>),
    |     ^^^^^

error: methods `open_channel` and `get_state_handlers` are never used
   --> src/hal/serial.rs:75:18
    |
39  | impl SerialConnections {
    | ---------------------- methods in this implementation
...
75  |     pub async fn open_channel(
    |                  ^^^^^^^^^^^^
...
102 |     pub fn get_state_handlers(&self) -> Vec<HandlerState> {
    |            ^^^^^^^^^^^^^^^^^^
    |
    = note: `-D dead-code` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(dead_code)]`

error: type `hal::serial::HandlerState` is more private than the item `hal::serial::SerialConnections::get_state_handlers`
   --> src/hal/serial.rs:102:5
    |
102 |     pub fn get_state_handlers(&self) -> Vec<HandlerState> {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ method `hal::serial::SerialConnections::get_state_handlers` is reachable at visibility `pub(crate)`
    |
note: but type `hal::serial::HandlerState` is only usable at visibility `pub(self)`
   --> src/hal/serial.rs:146:1
    |
146 | enum HandlerState {
    | ^^^^^^^^^^^^^^^^^
    = note: `-D private-interfaces` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(private_interfaces)]`

error: unused variable: `cx`
   --> src/hal/serial.rs:140:9
    |
140 |         cx: &mut std::task::Context<'_>,
    |         ^^ help: if this is intentional, prefix it with an underscore: `_cx`

error: unused variable: `cx`
   --> src/hal/serial.rs:133:9
    |
133 |         cx: &mut std::task::Context<'_>,
    |         ^^ help: if this is intentional, prefix it with an underscore: `_cx`

error: unused variable: `buf`
   --> src/hal/serial.rs:126:9
    |
126 |         buf: &[u8],
    |         ^^^ help: if this is intentional, prefix it with an underscore: `_buf`

error: unused variable: `cx`
   --> src/hal/serial.rs:125:9
    |
125 |         cx: &mut std::task::Context<'_>,
    |         ^^ help: if this is intentional, prefix it with an underscore: `_cx`

error: unused variable: `buf`
   --> src/hal/serial.rs:116:9
    |
116 |         buf: &mut tokio::io::ReadBuf<'_>,
    |         ^^^ help: if this is intentional, prefix it with an underscore: `_buf`

error: unused variable: `cx`
   --> src/hal/serial.rs:115:9
    |
115 |         cx: &mut std::task::Context<'_>,
    |         ^^ help: if this is intentional, prefix it with an underscore: `_cx`
    |
    = note: `-D unused-variables` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(unused_variables)]`

error: variant `initialized` should have an upper camel case name
   --> src/hal/serial.rs:147:5
    |
147 |     initialized,
    |     ^^^^^^^^^^^ help: convert the identifier to upper camel case (notice the capitalization): `Initialized`
    |
    = note: `-D non-camel-case-types` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(non_camel_case_types)]`