etnaviv: perfmon: fix total and idle HI cyleces readout
authorChristian Gmeiner <christian.gmeiner@gmail.com>
Wed, 31 Jul 2019 21:30:34 +0000 (23:30 +0200)
committerLucas Stach <l.stach@pengutronix.de>
Fri, 9 Aug 2019 12:08:29 +0000 (14:08 +0200)
As seen at CodeAurora's linux-imx git repo in imx_4.19.35_1.0.0 branch.

Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
drivers/gpu/drm/etnaviv/etnaviv_perfmon.c

index ef24a08..8adbf28 100644 (file)
@@ -4,6 +4,7 @@
  * Copyright (C) 2017 Zodiac Inflight Innovations
  */
 
+#include "common.xml.h"
 #include "etnaviv_gpu.h"
 #include "etnaviv_perfmon.h"
 #include "state_hi.xml.h"
@@ -35,13 +36,6 @@ struct etnaviv_pm_domain_meta {
        u32 nr_domains;
 };
 
-static u32 simple_reg_read(struct etnaviv_gpu *gpu,
-       const struct etnaviv_pm_domain *domain,
-       const struct etnaviv_pm_signal *signal)
-{
-       return gpu_read(gpu, signal->data);
-}
-
 static u32 perf_reg_read(struct etnaviv_gpu *gpu,
        const struct etnaviv_pm_domain *domain,
        const struct etnaviv_pm_signal *signal)
@@ -75,6 +69,34 @@ static u32 pipe_reg_read(struct etnaviv_gpu *gpu,
        return value;
 }
 
+static u32 hi_total_cycle_read(struct etnaviv_gpu *gpu,
+       const struct etnaviv_pm_domain *domain,
+       const struct etnaviv_pm_signal *signal)
+{
+       u32 reg = VIVS_HI_PROFILE_TOTAL_CYCLES;
+
+       if (gpu->identity.model == chipModel_GC880 ||
+               gpu->identity.model == chipModel_GC2000 ||
+               gpu->identity.model == chipModel_GC2100)
+               reg = VIVS_MC_PROFILE_CYCLE_COUNTER;
+
+       return gpu_read(gpu, reg);
+}
+
+static u32 hi_total_idle_cycle_read(struct etnaviv_gpu *gpu,
+       const struct etnaviv_pm_domain *domain,
+       const struct etnaviv_pm_signal *signal)
+{
+       u32 reg = VIVS_HI_PROFILE_IDLE_CYCLES;
+
+       if (gpu->identity.model == chipModel_GC880 ||
+               gpu->identity.model == chipModel_GC2000 ||
+               gpu->identity.model == chipModel_GC2100)
+               reg = VIVS_HI_PROFILE_TOTAL_CYCLES;
+
+       return gpu_read(gpu, reg);
+}
+
 static const struct etnaviv_pm_domain doms_3d[] = {
        {
                .name = "HI",
@@ -84,13 +106,13 @@ static const struct etnaviv_pm_domain doms_3d[] = {
                .signal = (const struct etnaviv_pm_signal[]) {
                        {
                                "TOTAL_CYCLES",
-                               VIVS_HI_PROFILE_TOTAL_CYCLES,
-                               &simple_reg_read
+                               0,
+                               &hi_total_cycle_read
                        },
                        {
                                "IDLE_CYCLES",
-                               VIVS_HI_PROFILE_IDLE_CYCLES,
-                               &simple_reg_read
+                               0,
+                               &hi_total_idle_cycle_read
                        },
                        {
                                "AXI_CYCLES_READ_REQUEST_STALLED",