public inbox for linux-i2c@vger.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hansg@kernel.org>
To: "Cepeda Lopez, Israel A" <israel.a.cepeda.lopez@intel.com>,
	Sakari Ailus <sakari.ailus@linux.intel.com>
Cc: Andi Shyti <andi.shyti@kernel.org>,
	Wolfram Sang <wsa+renesas@sang-engineering.com>,
	"linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>
Subject: Re: RE: [PATCH 1/2] i2c: usbio: Add ACPI device-id for MTL-CVF devices
Date: Sat, 4 Oct 2025 16:03:51 +0200	[thread overview]
Message-ID: <1e1b4323-2eb8-4c02-a47b-8e951606cffd@kernel.org> (raw)
In-Reply-To: <DS4PPF11A2D5672F6724542FB9C1E9AEF60FAE7A@DS4PPF11A2D5672.namprd11.prod.outlook.com>

Hi Israel,

On 2-Oct-25 8:05 PM, Cepeda Lopez, Israel A wrote:
> Thank you Hans,
> 
>> -----Original Message-----
>> From: Hans de Goede <hansg@kernel.org>
>> Sent: Thursday, October 2, 2025 10:49 AM
>> To: Cepeda Lopez, Israel A <israel.a.cepeda.lopez@intel.com>; Sakari Ailus
>> <sakari.ailus@linux.intel.com>
>> Cc: Andi Shyti <andi.shyti@kernel.org>; Wolfram Sang <wsa+renesas@sang-
>> engineering.com>; linux-i2c@vger.kernel.org
>> Subject: Re: [PATCH 1/2] i2c: usbio: Add ACPI device-id for MTL-CVF devices
>>
>> Hi Israel,
>>
>> On 2-Oct-25 5:52 PM, Cepeda Lopez, Israel A wrote:
>>> Hi Hans,
>>>
>>> LJCA and USBIO shouldn't be sharing HIDs.
>>
>> Yes in practice we have shipping hardware that is sharing HIDs
>> and since this is hardware out there in the field there is
>> nothing we can do about this.
> 
> There is. DELL has to provide a new FW/BIOS update to fix the HID
> of the system on fault. That is the correct solution to the issue.

Even if Dell is willing to do this (which I find unlikely)
it will still likely take at least 6 months for a BIOS update
to show up with these changes (BIOS QA takes a lot of time in
my experience).

And even then we cannot count on users to actually install BIOS
updates. So we will still need the INTC10D1, INTC10D2 and
INTC10D3 HIDs in both the ljca and usbio drivers to support
people who are still at the older BIOS.

And if I've understood Sakari correct then the INTC10D1, INTC10D2
HIDs are indeed intended for use with the USBIO driver and
the problem is that ideally they should not also be used by
the LJCA driver.

Since this patch adds the HIDs to the USBIO drivers, where they
belong the whole discussion of them not supposed to be in the LJCA
driver is sort of offtopic and this patch is correct regardless.

As such can you please give your Acked-by or Reviewed-by
for this patch and for the matching GPIO patch:

https://lore.kernel.org/linux-gpio/20251001212416.4871-1-hansg@kernel.org/

Regards,

Hans


  reply	other threads:[~2025-10-04 14:03 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-01 21:23 [PATCH 1/2] i2c: usbio: Add ACPI device-id for MTL-CVF devices Hans de Goede
2025-10-01 21:58 ` Sakari Ailus
2025-10-01 22:09   ` Sakari Ailus
2025-10-02  8:56   ` Hans de Goede
2025-10-02  9:18     ` Sakari Ailus
2025-10-02 12:03       ` Hans de Goede
2025-10-02 13:04       ` Hans de Goede
2025-10-02 15:52         ` Cepeda Lopez, Israel A
2025-10-02 17:49           ` Hans de Goede
2025-10-02 18:05             ` Cepeda Lopez, Israel A
2025-10-04 14:03               ` Hans de Goede [this message]
2025-10-09  1:50                 ` Cepeda Lopez, Israel A
2025-10-06  6:29         ` Sakari Ailus
2025-10-14 11:58 ` Wolfram Sang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1e1b4323-2eb8-4c02-a47b-8e951606cffd@kernel.org \
    --to=hansg@kernel.org \
    --cc=andi.shyti@kernel.org \
    --cc=israel.a.cepeda.lopez@intel.com \
    --cc=linux-i2c@vger.kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    --cc=wsa+renesas@sang-engineering.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox