Generally, keep using the same name for your generics. In this case that would be S.

In the impl you're not using a generic, but you're naming a trait.

It should be like this:
```rust
impl<S: Smbus<Error = std::io::Error>> Dashboard for LCDDashboard<S> {
    // ...
}
```