[pull] amdgpu drm-fixes-5.2

[Daniel Vetter]

On Wed, Jul 3, 2019 at 3:10 PM Alex Deucher <alexdeucher at gmail.com> wrote:
>
> On Wed, Jul 3, 2019 at 5:54 AM Daniel Vetter <daniel at ffwll.ch> wrote:
> >
> > On Tue, Jul 02, 2019 at 08:57:05PM -0500, Alex Deucher wrote:
> > > Hi Dave, Daniel,
> > >
> > > 3 fixes all cc'ed to stable.  Note that dim complains about the Fixes tag
> > > in one of the patches.  The patch has:
> > > Fixes: 921935dc6404 ("drm/amd/powerplay: enforce display related settings only on needed")
> > > while dim recommends:
> >
> > This is the right format.
> >
> > > Fixes: commit 921935dc6404 ("drm/amd/powerplay: enforce display related settings only on needed")
> >
> > Adding a "commit" is wrong. And at least my dim here doesn't complain
> > about your pull. How did you test this?
>
> dim checkpatch 665d6d4e32313a7952bb3339647f74c3a6b0d78a

Hah, another one fell into the trap :-/

dim checkpatch is just a wrapper around scripts/checkpatch.pl, it's
not what's used when you push to a dim managed branch, or what we use
when processing a pull request.

> -:8: ERROR:GIT_COMMIT_ID: Please use git commit description style
> 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit
> 921935dc6404 ("drm/amd/powerplay: enforce display related settings
> only on needed")'
> #8:
> 921935dc6404 ("drm/amd/powerplay: enforce display related settings
> only on needed")

Because checkpatch is garbage. The real dim checks only get used for
dim push and dim apply-pull. There was a half-baked patch somewhere to
integrate that into dim checkpatch, but that still leaves the problem
that checkpatch.pl is useless. Other option is if you use dim push to
push to the drm-amd.git tree (it's still all set up from years back
when at least Harry seemed somewhat enthusiastic about
group-maintaining amd.git outside of the amd firewall instead of
inside).




> Alex
>
> > -Daniel
> >
> > > I feel like the former is the more common nomencleture (at least historically),
> > > but I'm happy to respin if you'd prefer.
> > >
> > > The following changes since commit 665d6d4e32313a7952bb3339647f74c3a6b0d78a:
> > >
> > >   Merge tag 'drm-misc-fixes-2019-06-26' of git://anongit.freedesktop.org/drm/drm-misc into drm-fixes (2019-06-27 11:34:52 +1000)
> > >
> > > are available in the Git repository at:
> > >
> > >   git://people.freedesktop.org/~agd5f/linux tags/drm-fixes-5.2-2019-07-02
> > >
> > > for you to fetch changes up to 25f09f858835b0e9a06213811031190a17d8ab78:
> > >
> > >   drm/amdgpu/gfx9: use reset default for PA_SC_FIFO_SIZE (2019-07-01 12:16:26 -0500)
> > >
> > > ----------------------------------------------------------------
> > > drm-fixes-5.2-2019-07-02:
> > >
> > > Fixes for stable
> > >
> > > amdgpu:
> > > - stability fix for gfx9
> > > - regression fix for HG on some polaris boards
> > > - crash fix for some new OEM boards
> > >
> > > ----------------------------------------------------------------
> > > Alex Deucher (1):
> > >       drm/amdgpu/gfx9: use reset default for PA_SC_FIFO_SIZE
> > >
> > > Evan Quan (1):
> > >       drm/amd/powerplay: use hardware fan control if no powerplay fan table
> > >
> > > Lyude Paul (1):
> > >       drm/amdgpu: Don't skip display settings in hwmgr_resume()
> > >
> > >  drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c                 | 19 -------------------
> > >  drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c           |  2 +-
> > >  .../drm/amd/powerplay/hwmgr/process_pptables_v1_0.c   |  4 +++-
> > >  drivers/gpu/drm/amd/powerplay/inc/hwmgr.h             |  1 +
> > >  .../gpu/drm/amd/powerplay/smumgr/polaris10_smumgr.c   |  4 ++++
> > >  5 files changed, 9 insertions(+), 21 deletions(-)
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch