On Wed, May 4, 2022 at 12:46 PM Javier Martinez Canillas <[email protected]> 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 <[email protected]> > > --- > > The patch looks good to me. > > Reviewed-by: Javier Martinez Canillas <[email protected]> > > -- > Best regards, > > Javier Martinez Canillas > Linux Engineering > Red Hat >
