From: Heikki Krogerus <heikki.krogerus@linux.intel.com>
To: Emilie Roberts <hadrosaur@google.com>
Cc: Prashant Malani <pmalani@chromium.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Benson Leung <bleung@chromium.org>,
Tzung-Bi Shih <tzungbi@kernel.org>,
Guenter Roeck <groeck@chromium.org>,
"Nyman, Mathias" <mathias.nyman@intel.com>,
"Regupathy, Rajaram" <rajaram.regupathy@intel.com>,
"Radjacoumar, Shyam Sundar" <ssradjacoumar@google.com>,
Samuel Jacob <samjaco@google.com>,
Uday Bhat <uday.m.bhat@intel.com>,
linux-usb@vger.kernel.org, chrome-platform@lists.linux.dev,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 2/2] platform/chrome: cros_ec_typec: Make sure the USB role switch has PLD
Date: Wed, 14 Feb 2024 13:59:06 +0200 [thread overview]
Message-ID: <ZcyrCosNDXxGZh9O@kuha.fi.intel.com> (raw)
In-Reply-To: <CAAuZZi9h=d2_CM4tU4-H9wJfhLZbw99X2dGSddiCeDFDdb+kjw@mail.gmail.com>
Hi Emilie,
On Wed, Feb 14, 2024 at 12:04:22PM +0100, Emilie Roberts wrote:
> My understanding is that this is related to the wiring spec and not
> ChromeOS specific. It seems possible that OEMs making non-ChromeOS devices
> may have this same issue. Or are we certain that only Chromebooks will ever
> see this?
Non-ChromeOS platforms do not have this issue.
The issue is with the ACPI tables - the USB role switch ACPI device
nodes don't have the _PLD object on these systems. Ideally this could
be fixed there by simply adding the _PLD to those ACPI device objects,
but I understood that that is not an option.
But maybe I misunderstood... Can the ACPI tables on these platforms
still be updated?
thanks,
--
heikki
next prev parent reply other threads:[~2024-02-14 11:59 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-13 13:00 [PATCH v2 0/2] platform/chrome: typec: xHCI DbC Heikki Krogerus
2024-02-13 13:00 ` [PATCH v2 1/2] usb: roles: Link the switch to its connector Heikki Krogerus
2024-02-14 9:25 ` Prashant Malani
2024-02-13 13:00 ` [PATCH v2 2/2] platform/chrome: cros_ec_typec: Make sure the USB role switch has PLD Heikki Krogerus
2024-02-13 16:55 ` Prashant Malani
2024-02-14 10:22 ` Heikki Krogerus
[not found] ` <CAAuZZi9h=d2_CM4tU4-H9wJfhLZbw99X2dGSddiCeDFDdb+kjw@mail.gmail.com>
2024-02-14 11:59 ` Heikki Krogerus [this message]
2024-02-14 18:11 ` Prashant Malani
2024-03-25 1:54 ` [PATCH v2 0/2] platform/chrome: typec: xHCI DbC patchwork-bot+chrome-platform
2024-03-25 2:13 ` patchwork-bot+chrome-platform
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=ZcyrCosNDXxGZh9O@kuha.fi.intel.com \
--to=heikki.krogerus@linux.intel.com \
--cc=bleung@chromium.org \
--cc=chrome-platform@lists.linux.dev \
--cc=gregkh@linuxfoundation.org \
--cc=groeck@chromium.org \
--cc=hadrosaur@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mathias.nyman@intel.com \
--cc=pmalani@chromium.org \
--cc=rajaram.regupathy@intel.com \
--cc=samjaco@google.com \
--cc=ssradjacoumar@google.com \
--cc=tzungbi@kernel.org \
--cc=uday.m.bhat@intel.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