mbox series

[v2,0/2] drm/panfrost: Tidy up the devfreq implementation

Message ID 20191025134143.14324-1-steven.price@arm.com (mailing list archive)
Headers show
Series drm/panfrost: Tidy up the devfreq implementation | expand

Message

Steven Price Oct. 25, 2019, 1:41 p.m. UTC
The devfreq implementation in panfrost is unnecessarily open coded. It
also tracks utilisation metrics per slot which isn't very useful. Let's
tidy it up!

Changes since v1:
 http://lkml.kernel.org/r/20190912112804.10104-1-steven.price%40arm.com
 * Rebased onto latest drm-misc-next, specifically after
   d18a96620411 ("drm/panfrost: Remove NULL checks for regulator")
 * Added tags

Steven Price (2):
  drm/panfrost: Use generic code for devfreq
  drm/panfrost: Simplify devfreq utilisation tracking

 drivers/gpu/drm/panfrost/panfrost_devfreq.c | 124 ++++++--------------
 drivers/gpu/drm/panfrost/panfrost_devfreq.h |   3 +-
 drivers/gpu/drm/panfrost/panfrost_device.h  |  14 +--
 drivers/gpu/drm/panfrost/panfrost_job.c     |  15 ++-
 4 files changed, 48 insertions(+), 108 deletions(-)

Comments

Rob Herring (Arm) Oct. 29, 2019, 6:26 p.m. UTC | #1
On Fri, Oct 25, 2019 at 8:41 AM Steven Price <steven.price@arm.com> wrote:
>
> The devfreq implementation in panfrost is unnecessarily open coded. It
> also tracks utilisation metrics per slot which isn't very useful. Let's
> tidy it up!
>
> Changes since v1:
>  http://lkml.kernel.org/r/20190912112804.10104-1-steven.price%40arm.com
>  * Rebased onto latest drm-misc-next, specifically after
>    d18a96620411 ("drm/panfrost: Remove NULL checks for regulator")
>  * Added tags
>
> Steven Price (2):
>   drm/panfrost: Use generic code for devfreq
>   drm/panfrost: Simplify devfreq utilisation tracking

Series applied.

Rob