On Wed, May 4, 2022 at 12:46 PM Javier Martinez Canillas javierm@redhat.com wrote:
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)").
When you do it that way checkpatch and some maintainers complain about splitting up a commit line across multiple lines.
Alex
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