linux-kernel - Re: [PATCH v5 0/5] i3c: Add basic HDR mode support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aO1VHfHYNxboS3ag@lizhi-Precision-Tower-5810>
Date: Mon, 13 Oct 2025 15:38:05 -0400
From: Frank Li <Frank.li@....com>
To: Jonathan Cameron <jic23@...nel.org>
Cc: Alexandre Belloni <alexandre.belloni@...tlin.com>,
	Miquel Raynal <miquel.raynal@...tlin.com>,
	David Lechner <dlechner@...libre.com>,
	Nuno Sá <nuno.sa@...log.com>,
	Andy Shevchenko <andy@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>, linux-i3c@...ts.infradead.org,
	linux-kernel@...r.kernel.org, imx@...ts.linux.dev,
	linux-iio@...r.kernel.org, joshua.yeong@...rfivetech.com,
	devicetree@...r.kernel.org, Carlos Song <carlos.song@....com>,
	Adrian Fluturel <fluturel.adrian@...il.com>
Subject: Re: [PATCH v5 0/5] i3c: Add basic HDR mode support

On Sun, Oct 12, 2025 at 06:03:27PM +0100, Jonathan Cameron wrote:
> On Tue, 07 Oct 2025 16:06:12 -0400
> Frank Li <Frank.Li@....com> wrote:
>
> > Add basic HDR mode support, only support private transfer, not support
> > CCC command.
> >
> > Update i3c framework API to allow pass down mode and extend driver callback
> > function.
> >
> > Implement HDR transfer in svc i3c master driver.
> >
> > Simplifed HDR flow is (ref i3c spec line 5514) Figure 129
> >
> > <--              SDR            ---> | <--- HDR
> > START 0x7E RnW(0) ACK CCC(ENTHDR0) T   HDR-CMD(00-7f write, 80--ff read)
> >
> >                                     ----> |
> > HDR-DATA HDR-CRC HDR-RESTART .... HDR-EXIT
> >
> > Note: HDR-CMD is 16bit data, which included 7bit slave address and 8bit
> > read/write command.
> >
> > svc hardware can auto issue SDR part.
> >
> > Signed-off-by: Frank Li <Frank.Li@....com>
>
> Whilst there will probably have to be a v6 for the ACPI ID issue in patch 5,
> I'd like to ask the question of how are we planning to merge this?
>
> Maybe an immutable branch either in IIO or I3C trees that the other one picks up?
>
> It's a new driver so could gamble on taking the IIO driver the I3C tree but even
> then I'd like a topic / immutable branch in case any IIO wide refactors or similar
> hit this cycle.

It is up to Alexandre Belloni.

Frank
>
> Thanks,
>
> Jonathan
>
> > ---
> > Changes in v5:
> > - Just realized missed CC mail list devicetree@...r.kernel.org and resend
> > - Link to v4: https://lore.kernel.org/r/20251007-i3c_ddr-v4-0-3afea5105775@nxp.com
> >
> > Changes in v4:
> > - use master's hdr_cap to check HDR cap.
> > - add mmc5603 support.
> > - Link to v3: https://lore.kernel.org/r/20250930-i3c_ddr-v3-0-b627dc2ef172@nxp.com
> >
> > Changes in v3:
> > - Add new patch for change rnw to union for svc.
> > - Detial changes see each patch's change log.
> > - Link to v2: https://lore.kernel.org/r/20250924-i3c_ddr-v2-0-682a0eb32572@nxp.com
> >
> > Changes in v2:
> > - Add sensor driver, which use HDR mode read/write data.
> > - change priv_xfer to i3c_xfer.
> > - Link to v1: https://lore.kernel.org/r/20250129-i3c_ddr-v1-0-028a7a5d4324@nxp.com
> >
> > ---
> > Frank Li (5):
> >       i3c: Add HDR API support
> >       i3c: master: svc: Replace bool rnw with union for HDR support
> >       i3c: master: svc: Add basic HDR mode support
> >       dt-bindings: trivial-devices: add MEMSIC 3-axis magnetometer
> >       iio: magnetometer: Add mmc5633 sensor
> >
> >  .../devicetree/bindings/trivial-devices.yaml       |   4 +
> >  drivers/i3c/device.c                               |  27 +-
> >  drivers/i3c/internals.h                            |   6 +-
> >  drivers/i3c/master.c                               |  19 +-
> >  drivers/i3c/master/svc-i3c-master.c                |  96 +++-
> >  drivers/iio/magnetometer/Kconfig                   |  12 +
> >  drivers/iio/magnetometer/Makefile                  |   1 +
> >  drivers/iio/magnetometer/mmc5633.c                 | 579 +++++++++++++++++++++
> >  include/linux/i3c/device.h                         |  34 +-
> >  include/linux/i3c/master.h                         |   4 +
> >  10 files changed, 745 insertions(+), 37 deletions(-)
> > ---
> > base-commit: 5248ab97a6dd1dc6856422f6b1595baaf4fa85ce
> > change-id: 20250129-i3c_ddr-b15488901eb8
> >
> > Best regards,
> > --
> > Frank Li <Frank.Li@....com>
> >
>