Re: [PATCH 1/2] i2c: usbio: Add ACPI device-id for MTL-CVF devices

[Sakari Ailus <sakari.ailus@linux.intel.com>]

Hi Hans, Lixu,

On Thu, Oct 02, 2025 at 12:58:07AM +0300, Sakari Ailus wrote:
> Hi Hans,
> 
> On Wed, Oct 01, 2025 at 11:23:10PM +0200, Hans de Goede wrote:
> > Add "INTC10D2" ACPI device-id for MTL-CVF devices, like the Dell Latitude
> > 7450.
> > 
> > Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2368506
> > Signed-off-by: Hans de Goede <hansg@kernel.org>
> > ---
> > Note this applies on top of the addition of the new i2c-usbio driver which
> > is being merged through gregkh/usb.git usb-next
> > ---
> >  drivers/i2c/busses/i2c-usbio.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/i2c/busses/i2c-usbio.c b/drivers/i2c/busses/i2c-usbio.c
> > index d42f9ab6e9a5..e7799abf6787 100644
> > --- a/drivers/i2c/busses/i2c-usbio.c
> > +++ b/drivers/i2c/busses/i2c-usbio.c
> > @@ -27,6 +27,7 @@ static const struct acpi_device_id usbio_i2c_acpi_hids[] = {
> >  	{ "INTC1008" }, /* MTL */
> >  	{ "INTC10B3" }, /* ARL */
> >  	{ "INTC10B6" }, /* LNL */
> > +	{ "INTC10D2" }, /* MTL-CVF */
> 
> INTC10D2 is apparently already claimed by the LJCA I²C driver. Do you have
> an actual system that uses USBIO and this HID?

Actually it looks like i2c-ljca.c shouldn't list this HID. The same
probably goes for all 10CF--10D3.

Lixu: any idea?

I thought USBIO always came with CVS but that doesn't seem to be the case
here.

> 
> >  	{ "INTC10E3" }, /* PTL */
> >  	{ }
> >  };
> 

-- 
Regards,

Sakari Ailus