Re: [PATCH v5 6/7] sample: rust: pci: use `register!` macro
From: Danilo Krummrich
Date: Fri Feb 06 2026 - 13:57:19 EST
On Thu Jan 29, 2026 at 2:32 PM CET, Alexandre Courbot wrote:
> Convert the direct IO accesses to properly defined registers.
>
> Tested-by: Zhi Wang <zhiw@xxxxxxxxxx>
> Signed-off-by: Alexandre Courbot <acourbot@xxxxxxxxxx>
I think you forgot the address my comment from [1].
Changing the config space is nice, but I think it would be even nicer to replace
the raw register writes and reads of the pci::Bar.
[1] https://lore.kernel.org/all/DG2DEH4ELYHK.2KUOIDQAILZHT@xxxxxxxxxx/