Re: [PATCH 1/2] Revert "drm/amdgpu: disable runpm if we are the primary adapter"

[PATCH 1/2] Revert "drm/amdgpu: disable runpm if we are the primary adapter" · Alex Deucher <alexander.deucher@amd.com> · 2022-05-04
[PATCH 2/2] Revert "fbdev: fbmem: add a helper to determine if an aperture is used by a fw fb" · Alex Deucher <alexander.deucher@amd.com> · 2022-05-04
Re: [PATCH 2/2] Revert "fbdev: fbmem: add a helper to determine if an aperture is used by a fw fb" · Javier Martinez Canillas <javierm@redhat.com> · 2022-05-04
Re: [PATCH 1/2] Revert "drm/amdgpu: disable runpm if we are the primary adapter" · Javier Martinez Canillas <javierm@redhat.com> · 2022-05-04
Re: [PATCH 1/2] Revert "drm/amdgpu: disable runpm if we are the primary adapter" · Alex Deucher <hidden> · 2022-05-04
Re: [PATCH 1/2] Revert "drm/amdgpu: disable runpm if we are the primary adapter" · Javier Martinez Canillas <javierm@redhat.com> · 2022-05-04
Re: [PATCH 1/2] Revert "drm/amdgpu: disable runpm if we are the primary adapter" · Daniel Vetter <hidden> · 2022-05-04

From: Javier Martinez Canillas <javierm@redhat.com>
Date: 2022-05-04 16:46:05
Also in: amd-gfx, dri-devel

Hello Alex,

On 5/4/22 15:48, Alex Deucher wrote:

This reverts commit b95dc06af3e683d6b7ddbbae178b2b2a21ee8b2b.

This workaround is no longer necessary.  We have a better workaround
in commit f95af4a9236695 ("drm/amdgpu: don't runtime suspend if there are displays attached (v3)").

I would write this line instead as:

in commit f95af4a9236695 ("drm/amdgpu: don't runtime suspend if there are
displays attached (v3)").

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---

The patch looks good to me.

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Linux Engineering
Red Hat

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help