Hi Cédric,
> Subject: Re: [PATCH v2 00/20] i3c: aspeed: Add I3C support
>
> Jamin,
>
>
> On 2/3/26 09:52, Jamin Lin via qemu development wrote:
> > v1:
> > The initial patch series was based on work by Joe Komlodi
> <komlodi@google.com>.
> >
> > This series adds I3C bus support to QEMU and adds more functionality to
> the
> > Aspeed I3C controller.
> >
> > This implementation is a basic implementation that introduces IBIs
> > (including hot-join), CCCs, and SDR data transfer. As-is, it doesn't support
> > multi-controller buses or HDR transfers.
> >
> > First we add the I3C bus and controller model. With that added we
> extend
> > the functionality of the Aspeed I3C controller so it can do transfers
> > and handle IBIs.
> >
> > Next, we add a mock I3C target. It's intended to be a very simple target
> > just to verify that I3C is working on the guest. Internally, we've used it
> > on Linux to verify that i3C devices can be probed and can send/receive
> data
> > and IBIs.
> > This target is sort of like an EEPROM, and it can also send IBIs upon
> > reception of a user-defined magic number.
> >
> > Lastly we add hotplugging support. The hotplugging doesn't do
> anything too
> > complicated, it just adds the device attempting to hotplug to the bus. It
> > is the device's responsibility to hot-join and go through the DAA process
> > to participate on the bus.
> >
> > v2:
> > Jamin Lin <jamin_lin@aspeedtech.com> has taken ownership of the I3C
> patch
> > series for upstream submission.
> >
> > Changes in this version include:
> >
> > 1. Added I3C functional tests.
> > 2. Updated patch 4 to refine register field definitions.
> > 3. Updated patch 7 to correct read-only register field masks.
> >
> > Jamin Lin (1):
> > tests/functional/arm/test_aspeed_ast2600_sdk: Add i3c functional
> > test
> >
> > Joe Komlodi (19):
> > hw/misc/aspeed_i3c: Move to i3c directory
> > hw/i3c: Add bus support
> > hw/i3c: Split DesignWare I3C out of Aspeed I3C
> > hw/i3c/dw-i3c: Add more register fields
> > hw/i3c/aspeed_i3c: Add more register fields
> > hw/i3c/dw-i3c: Add more reset values
> > hw/i3c/aspeed_i3c: Add register RO field masks
> > hw/i3c/dw-i3c: Add register RO field masks
> > hw/i3c/dw-i3c: Treat more registers as read-as-zero
> > hw/i3c/dw-i3c: Use 32 bits on MMIO writes
> > hw/i3c/dw-i3c: Add IRQ MMIO behavior
> > hw/i3c/dw-i3c: Add data TX and RX
> > hw/i3c/dw-i3c: Add IBI handling
> > hw/i3c/dw-i3c: Add ctrl MMIO handling
> > hw/i3c/dw-i3c: Add controller resets
> > hw/i3c/aspeed: Add I3C bus get function
> > hw/i3c: Add Mock target
> > hw/arm/aspeed: Build with I3C_DEVICES
> > hw/i3c: Add hotplug support
> >
> > meson.build | 1 +
> > hw/i3c/trace.h | 2 +
> > include/hw/arm/aspeed_soc.h | 2 +-
> > include/hw/{misc => i3c}/aspeed_i3c.h | 22 +-
> > include/hw/i3c/dw-i3c.h | 201 ++
> > include/hw/i3c/i3c.h | 277 +++
> > include/hw/i3c/mock-i3c-target.h | 52 +
> > hw/i3c/aspeed_i3c.c | 261 +++
> > hw/i3c/core.c | 669 ++++++
> > hw/i3c/dw-i3c.c | 1864
> +++++++++++++++++
> > hw/i3c/mock-i3c-target.c | 311 +++
> > hw/misc/aspeed_i3c.c | 383 ----
> > hw/Kconfig | 1 +
> > hw/arm/Kconfig | 3 +
> > hw/i3c/Kconfig | 15 +
> > hw/i3c/meson.build | 6 +
> > hw/i3c/trace-events | 47 +
> > hw/meson.build | 1 +
> > hw/misc/meson.build | 1 -
> > hw/misc/trace-events | 6 -
> > .../functional/arm/test_aspeed_ast2600_sdk.py | 12 +
> > 21 files changed, 3730 insertions(+), 407 deletions(-)
> > create mode 100644 hw/i3c/trace.h
> > rename include/hw/{misc => i3c}/aspeed_i3c.h (63%)
> > create mode 100644 include/hw/i3c/dw-i3c.h
> > create mode 100644 include/hw/i3c/i3c.h
> > create mode 100644 include/hw/i3c/mock-i3c-target.h
> > create mode 100644 hw/i3c/aspeed_i3c.c
> > create mode 100644 hw/i3c/core.c
> > create mode 100644 hw/i3c/dw-i3c.c
> > create mode 100644 hw/i3c/mock-i3c-target.c
> > delete mode 100644 hw/misc/aspeed_i3c.c
> > create mode 100644 hw/i3c/Kconfig
> > create mode 100644 hw/i3c/meson.build
> > create mode 100644 hw/i3c/trace-events
>
>
> I didn't receive directly this series. So I grabbed it as a mbox file from lore.
> Could you please check with Kane how he sends his patches ?
> All is fine with him.
>
>
> Please add your Signed-off-by to the series.
>
Will add in v3.
Thanks-Jamin
> Thanks,
>
> C.