Re: [PATCH v6 5/5] iio: magnetometer: Add mmc5633 sensor

Jonathan Cameron
2025-10-18T16:54:27Z

On Sat, 18 Oct 2025 17:52:32 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> On Tue, 14 Oct 2025 12:40:04 -0400
> Frank Li <Frank.Li@nxp.com> wrote:
>
> > Add mmc5633 sensor basic support.
> > - Support read 20 bits X/Y/Z magnetic.
> > - Support I3C HDR mode to send start measurememt command.
> > - Support I3C HDR mode to read all sensors data by one command.
> >
> > Co-developed-by: Carlos Song <carlos.song@nxp.com>
> > Signed-off-by: Carlos Song <carlos.song@nxp.com>
> > Co-developed-by: Adrian Fluturel <fluturel.adrian@gmail.com>
> > Signed-off-by: Adrian Fluturel <fluturel.adrian@gmail.com>
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> LGTM.
>
> So my ideal case (assuming i3c parts are good to go) is an immutable
> branch in an appropriate tree that gets merged into both i3c and IIO trees
> and has first 4 patches. I'll then put the driver on top in the IIO tree.
I can't count. First 3 patches. Binding and driver would go on top in the IIO tree.

>
> I can spin one if that is useful.
>
> Thanks,
>
> Jonathan