Re: [PATCH v8 2/2] platform: Add initial synology microp driver
From: Miguel Ojeda
Date: Tue Apr 21 2026 - 11:09:20 EST
On Tue, Apr 21, 2026 at 4:17 PM Markus Probst <markus.probst@xxxxxxxxx> wrote:
>
> How about `const BLINK_DELAY: Msecs` ? The unit would be implied
> through the already existing type alias `kernel::time::Msecs` for u32.
Ideally we would use strong types for things like this, i.e. Rust
newtypes, rather than using type aliases.
e.g. see our `Delta` type.
Cheers,
Miguel