GPUs with support for the security features need some additional
setup to get the frontend started.

Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
---
 drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 37 +++++++++++++++++++++++++++++++++--
 1 file changed, 35 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c 
b/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
index 08ee716f7b3c..ccd5d5ff7ffd 100644
--- a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
+++ b/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
@@ -596,6 +596,12 @@ void etnaviv_gpu_start_fe(struct etnaviv_gpu *gpu, u32 
address, u16 prefetch)
        gpu_write(gpu, VIVS_FE_COMMAND_CONTROL,
                  VIVS_FE_COMMAND_CONTROL_ENABLE |
                  VIVS_FE_COMMAND_CONTROL_PREFETCH(prefetch));
+
+       if (gpu->sec_mode == ETNA_SEC_KERNEL) {
+               gpu_write(gpu, VIVS_MMUv2_SEC_COMMAND_CONTROL,
+                         VIVS_MMUv2_SEC_COMMAND_CONTROL_ENABLE |
+                         VIVS_MMUv2_SEC_COMMAND_CONTROL_PREFETCH(prefetch));
+       }
 }
 
 static void etnaviv_gpu_setup_pulse_eater(struct etnaviv_gpu *gpu)
@@ -669,6 +675,12 @@ static void etnaviv_gpu_hw_init(struct etnaviv_gpu *gpu)
                gpu_write(gpu, VIVS_MC_BUS_CONFIG, bus_config);
        }
 
+       if (gpu->sec_mode == ETNA_SEC_KERNEL) {
+               u32 val = gpu_read(gpu, VIVS_MMUv2_AHB_CONTROL);
+               val |= VIVS_MMUv2_AHB_CONTROL_NONSEC_ACCESS;
+               gpu_write(gpu, VIVS_MMUv2_AHB_CONTROL, val);
+       }
+
        /* setup the pulse eater */
        etnaviv_gpu_setup_pulse_eater(gpu);
 
@@ -731,6 +743,14 @@ int etnaviv_gpu_init(struct etnaviv_gpu *gpu)
                gpu->identity.features &= ~chipFeatures_FAST_CLEAR;
        }
 
+       /*
+        * On cores with security features supported, we claim control over the
+        * security states.
+        */
+       if ((gpu->identity.minor_features7 & chipMinorFeatures7_BIT_SECURITY) &&
+           (gpu->identity.minor_features10 & chipMinorFeatures10_SECURITY_AHB))
+               gpu->sec_mode = ETNA_SEC_KERNEL;
+
        ret = etnaviv_hw_reset(gpu);
        if (ret) {
                dev_err(gpu->dev, "GPU reset failed\n");
@@ -1499,17 +1519,30 @@ static void sync_point_worker(struct work_struct *work)
 
 static void dump_mmu_fault(struct etnaviv_gpu *gpu)
 {
-       u32 status = gpu_read(gpu, VIVS_MMUv2_STATUS);
+       u32 status_reg, status;
        int i;
 
+       if (gpu->sec_mode == ETNA_SEC_NONE)
+               status_reg = VIVS_MMUv2_STATUS;
+       else
+               status_reg = VIVS_MMUv2_SEC_STATUS;
+
+       status = gpu_read(gpu, status_reg);
        dev_err_ratelimited(gpu->dev, "MMU fault status 0x%08x\n", status);
 
        for (i = 0; i < 4; i++) {
+               u32 address_reg;
+
                if (!(status & (VIVS_MMUv2_STATUS_EXCEPTION0__MASK << (i * 4))))
                        continue;
 
+               if (gpu->sec_mode == ETNA_SEC_NONE)
+                       address_reg = VIVS_MMUv2_EXCEPTION_ADDR(i);
+               else
+                       address_reg = VIVS_MMUv2_SEC_EXCEPTION_ADDR;
+
                dev_err_ratelimited(gpu->dev, "MMU %d fault addr 0x%08x\n", i,
-                               gpu_read(gpu, VIVS_MMUv2_EXCEPTION_ADDR(i)));
+                                   gpu_read(gpu, address_reg));
        }
 }
 
-- 
2.11.0

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to