Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-21 Thread Alex Deucher
On Thu, Apr 21, 2022 at 9:06 AM Thorsten Leemhuis wrote: > > On 21.04.22 05:16, Alex Deucher wrote: > > We normally runtime suspend when there are displays attached if they > > are in the DPMS off state, however, if something wakes the GPU > > we send a hotplug event on resume (in case any display

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-21 Thread Thorsten Leemhuis
On 21.04.22 05:16, Alex Deucher wrote: > We normally runtime suspend when there are displays attached if they > are in the DPMS off state, however, if something wakes the GPU > we send a hotplug event on resume (in case any displays were connected > while the GPU was in suspend) which can cause use

RE: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-20 Thread Quan, Evan
rm/amdgpu: don't runtime suspend if there are displays > attached (v2) > > We normally runtime suspend when there are displays attached if they > are in the DPMS off state, however, if something wakes the GPU > we send a hotplug event on resume (in case any displays were connect

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-20 Thread Alex Deucher
Should be v3, fixed locally. Alex On Wed, Apr 20, 2022 at 11:16 PM Alex Deucher wrote: > > We normally runtime suspend when there are displays attached if they > are in the DPMS off state, however, if something wakes the GPU > we send a hotplug event on resume (in case any displays were connecte

[PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-20 Thread Alex Deucher
We normally runtime suspend when there are displays attached if they are in the DPMS off state, however, if something wakes the GPU we send a hotplug event on resume (in case any displays were connected while the GPU was in suspend) which can cause userspace to light up the displays again soon afte

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-20 Thread Alex Deucher
> > Cc: Deucher, Alexander ; Michele Ballabio > > > > Subject: [PATCH] drm/amdgpu: don't runtime suspend if there are displays > > attached (v2) > > > > We normally runtime suspend when there are displays attached if they > > are in the DPMS off state, how

RE: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-20 Thread Quan, Evan
[AMD Official Use Only] > -Original Message- > From: amd-gfx On Behalf Of Alex > Deucher > Sent: Thursday, April 14, 2022 4:15 AM > To: amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander ; Michele Ballabio > > Subject: [PATCH] drm/amdgpu: don't

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-20 Thread Christian König
I could ack it, but I'm absolutely not an expert on that stuff. Our DC team maybe? Or anybody working more on the PM code? Christian. Am 20.04.22 um 15:57 schrieb Alex Deucher: Ping? Anyone care to review this? Alex On Tue, Apr 19, 2022 at 9:47 AM Alex Deucher wrote: Ping? On Wed, Apr 13

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-20 Thread Alex Deucher
Ping? Anyone care to review this? Alex On Tue, Apr 19, 2022 at 9:47 AM Alex Deucher wrote: > > Ping? > > On Wed, Apr 13, 2022 at 4:15 PM Alex Deucher > wrote: > > > > We normally runtime suspend when there are displays attached if they > > are in the DPMS off state, however, if something wake

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-19 Thread Alex Deucher
On Tue, Apr 19, 2022 at 10:04 AM Paul Menzel wrote: > > Dear Alex, > > > Thank you for the patch. > > Am 13.04.22 um 22:15 schrieb Alex Deucher: > > We normally runtime suspend when there are displays attached if they > > are in the DPMS off state, however, if something wakes the GPU > > we send a

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-19 Thread Paul Menzel
Dear Alex, Thank you for the patch. Am 13.04.22 um 22:15 schrieb Alex Deucher: We normally runtime suspend when there are displays attached if they are in the DPMS off state, however, if something wakes the GPU we send a hotplug event on resume (in case any displays were connected while the GP

Re: [PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-19 Thread Alex Deucher
Ping? On Wed, Apr 13, 2022 at 4:15 PM Alex Deucher wrote: > > We normally runtime suspend when there are displays attached if they > are in the DPMS off state, however, if something wakes the GPU > we send a hotplug event on resume (in case any displays were connected > while the GPU was in suspe

[PATCH] drm/amdgpu: don't runtime suspend if there are displays attached (v2)

2022-04-13 Thread Alex Deucher
We normally runtime suspend when there are displays attached if they are in the DPMS off state, however, if something wakes the GPU we send a hotplug event on resume (in case any displays were connected while the GPU was in suspend) which can cause userspace to light up the displays again soon afte