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