Hi Andrew,
On Thu, May 08, 2025 at 10:15:13PM -0500, Andrew Ballance wrote:
> currently the rust `Io` type maps to the c read{b, w, l, q}/write{b, w, l, q}
> functions and have no support for port io.this is a problem for pci::Bar
> because the pointer returned by pci_iomap is expected to accessed with
> the ioread/iowrite api [0].
>
> this patch series splits the `Io` type into `Io`, `PortIo` and `MMIo`.and,
> updates pci::Bar, as suggested in the zulip[1], so that it is generic over
> Io and, a user can optionally give a compile time hint about the type of io.
>
> Link: https://docs.kernel.org/6.11/driver-api/pci/pci.html#c.pci_iomap [0]
> Link:
> https://rust-for-linux.zulipchat.com/#narrow/channel/288089-General/topic/.60IoRaw.60.20and.20.60usize.60/near/514788730
> [1]
>
> Andrew Ballance (6):
> rust: io: add new Io type
> rust: io: add from_raw_cookie functions
> rust: pci: make Bar generic over Io
> samples: rust: rust_driver_pci: update to use new bar and io api
> gpu: nova-core: update to use the new bar and io api
> rust: devres: fix doctest
>
> Fiona Behrens (5):
> rust: helpers: io: use macro to generate io accessor functions
> rust: io: Replace Io with MMIo using IoAccess trait
> rust: io: implement Debug for IoRaw and add some doctests
> rust: io: add PortIo
> io: move PIO_OFFSET to linux/io.h
Thanks for sending out the patch series.
I gave it a quick shot and the series breaks the build starting with patch 2. I
see that you have fixup commits later in the series, however in the kernel we
don't allow patches to intermediately introduce build failures, build warnings,
bugs, etc., see also [1]. You should still try to break things down logically as
good as possible.
>From the current structure of your patches it seems to me that structure-wise
you should be good by going through them one by one and fix them up; your later
patches should become "noops" then. But feel free to re-organize things if you
feel that's not the best approach.
Can you please fix this up and resend right away? This should make the
subsequent review much easier.
[1]
https://docs.kernel.org/process/submitting-patches.html#separate-your-changes
