[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ZIl+pyKgFdQuYCKm@shikoro>
Date: Wed, 14 Jun 2023 10:47:35 +0200
From: Wolfram Sang <wsa@...nel.org>
To: Andi Shyti <andi.shyti@...nel.org>
Cc: Mario Limonciello <mario.limonciello@....com>,
heikki.krogerus@...ux.intel.com, ajayg@...dia.com,
andriy.shevchenko@...ux.intel.com, linux-i2c@...r.kernel.org,
linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-usb@...r.kernel.org, Evan.Quan@....com, Lijo.Lazar@....com,
Sanket.Goswami@....com
Subject: Re: [PATCH v2] usb: typec: ucsi: Mark dGPUs as DEVICE scope
On Wed, Jun 14, 2023 at 02:16:21AM +0200, Andi Shyti wrote:
> Hi Mario,
>
> On Thu, May 18, 2023 at 11:11:50AM -0500, Mario Limonciello wrote:
> > power_supply_is_system_supplied() checks whether any power
> > supplies are present that aren't batteries to decide whether
> > the system is running on DC or AC. Downstream drivers use
> > this to make performance decisions.
> >
> > Navi dGPUs include an UCSI function that has been exported
> > since commit 17631e8ca2d3 ("i2c: designware: Add driver
> > support for AMD NAVI GPU").
> >
> > This UCSI function registers a power supply since commit
> > 992a60ed0d5e ("usb: typec: ucsi: register with power_supply class")
> > but this is not a system power supply.
> >
> > As the power supply for a dGPU is only for powering devices connected
> > to dGPU, create a device property to indicate that the UCSI endpoint
> > is only for the scope of `POWER_SUPPLY_SCOPE_DEVICE`.
> >
> > Link: https://lore.kernel.org/lkml/20230516182541.5836-2-mario.limonciello@amd.com/
> > Reviewed-by: Evan Quan <evan.quan@....com>
> > Tested-by: Evan Quan <evan.quan@....com>
> > Signed-off-by: Mario Limonciello <mario.limonciello@....com>
>
> Acked-by: Andi Shyti <andi.shyti@...nel.org>
Applied to for-next, thanks!
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists