RE: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Sider, Graham
d-gfx list ; Kasiviswanathan, Harish ; Sakhnovitch, Elena (Elen) Subject: Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation [CAUTION: External Email] On Fri, May 21, 2021 at 5:47 PM Alex Deucher wrote: > > On Fri, May 21, 2021 at 5:32 PM Sider, Graham wrote: > > > > Would thi

Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Alex Deucher
change. If we > > bump the version, at least the tools will know how to handle it. > > > > Alex > > > > > > > Best, > > > Graham > > > > > > -Original Message- > > > From: Alex Deucher > > > Sent: Friday, May 21, 2021 10:27

Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Alex Deucher
> Best, > Graham > > -Original Message- > From: Alex Deucher > Sent: Friday, May 21, 2021 4:15 PM > To: Sider, Graham > Cc: amd-gfx list ; Kasiviswanathan, Harish > ; Sakhnovitch, Elena (Elen) > > Subject: Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler t

RE: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Sider, Graham
To: Sider, Graham Cc: amd-gfx list ; Kasiviswanathan, Harish ; Sakhnovitch, Elena (Elen) Subject: Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation [CAUTION: External Email] On Fri, May 21, 2021 at 1:39 PM Sider, Graham wrote: > > Hi Alex, > > Are you re

Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Alex Deucher
ex Deucher > Sent: Friday, May 21, 2021 10:27 AM > To: Sider, Graham > Cc: amd-gfx list ; Kasiviswanathan, Harish > ; Sakhnovitch, Elena (Elen) > > Subject: Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation > > [CAUTION: External Email] > > General comment

RE: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Sider, Graham
m Cc: amd-gfx list ; Kasiviswanathan, Harish ; Sakhnovitch, Elena (Elen) Subject: Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation [CAUTION: External Email] General comment on the patch series, do you want to bump the metrics table version since the meaning of the throttler statu

Re: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Alex Deucher
General comment on the patch series, do you want to bump the metrics table version since the meaning of the throttler status has changed? Alex On Thu, May 20, 2021 at 10:30 AM Graham Sider wrote: > > Perform dependent to independent throttle status translation for > arcturus. > --- > .../gpu/dr

RE: [PATCH 2/6] drm/amd/pm: Add arcturus throttler translation

2021-05-21 Thread Kasiviswanathan, Harish
[AMD Official Use Only] Hi Graham, This patch series looks good. Please add "Signed-off-by" to all the commit messages. One additional comment inline below. -Original Message- From: Sider, Graham Sent: Thursday, May 20, 2021 10:29 AM To: amd-gfx@lists.freedesktop.org Cc: Kasiviswanat