This patch adds firmware binary and GPU model naming support for
Mali-Gx20 and Mali-Gx25 GPUs.

The GPU_COHERENCY_FEATURES macros are slightly reworked as the
assumption that FEATURE = BIT(PROTOCOL) no longer holds with the
introduction of the SHAREABLE_CACHE_SUPPORT, which is BIT(5) on the
GPU_COHERENCY_PROTOCOL register. As such, the feature bits are now
individually defined. Further changes were also made to enable
SHAREABLE_CACHE_SUPPORT if coherency is enabled and the feature is
supported.

This patch also fixes a minor bug that incorrectly writes ACE instead of
ACE_LITE to GPU_COHERENCY_PROTOCOL if coherency is enabled.

Reviewed-by: Chia-I Wu <olva...@gmail.com>
Reviewed-by: Liviu Dudau <liviu.du...@arm.com>
Signed-off-by: Karunika Choo <karunika.c...@arm.com>
---
 drivers/gpu/drm/panthor/panthor_device.c |  3 +--
 drivers/gpu/drm/panthor/panthor_fw.c     |  2 ++
 drivers/gpu/drm/panthor/panthor_gpu.c    | 14 ++++++++++++--
 drivers/gpu/drm/panthor/panthor_hw.c     | 18 ++++++++++++++++++
 drivers/gpu/drm/panthor/panthor_regs.h   |  5 ++++-
 5 files changed, 37 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/panthor/panthor_device.c 
b/drivers/gpu/drm/panthor/panthor_device.c
index 81df49880bd8..b85a744d99f8 100644
--- a/drivers/gpu/drm/panthor/panthor_device.c
+++ b/drivers/gpu/drm/panthor/panthor_device.c
@@ -33,8 +33,7 @@ static int panthor_gpu_coherency_init(struct panthor_device 
*ptdev)
        /* Check if the ACE-Lite coherency protocol is actually supported by 
the GPU.
         * ACE protocol has never been supported for command stream frontend 
GPUs.
         */
-       if ((gpu_read(ptdev, GPU_COHERENCY_FEATURES) &
-                     GPU_COHERENCY_PROT_BIT(ACE_LITE)))
+       if (ptdev->gpu_info.coherency_features & GPU_COHERENCY_FEATURE_ACE_LITE)
                return 0;
 
        drm_err(&ptdev->base, "Coherency not supported by the device");
diff --git a/drivers/gpu/drm/panthor/panthor_fw.c 
b/drivers/gpu/drm/panthor/panthor_fw.c
index fa6e0b48a0b2..9bf06e55eaee 100644
--- a/drivers/gpu/drm/panthor/panthor_fw.c
+++ b/drivers/gpu/drm/panthor/panthor_fw.c
@@ -1405,3 +1405,5 @@ MODULE_FIRMWARE("arm/mali/arch10.8/mali_csffw.bin");
 MODULE_FIRMWARE("arm/mali/arch10.10/mali_csffw.bin");
 MODULE_FIRMWARE("arm/mali/arch10.12/mali_csffw.bin");
 MODULE_FIRMWARE("arm/mali/arch11.8/mali_csffw.bin");
+MODULE_FIRMWARE("arm/mali/arch12.8/mali_csffw.bin");
+MODULE_FIRMWARE("arm/mali/arch13.8/mali_csffw.bin");
diff --git a/drivers/gpu/drm/panthor/panthor_gpu.c 
b/drivers/gpu/drm/panthor/panthor_gpu.c
index 5e2c3173ae27..e8d8dbeefac7 100644
--- a/drivers/gpu/drm/panthor/panthor_gpu.c
+++ b/drivers/gpu/drm/panthor/panthor_gpu.c
@@ -45,8 +45,18 @@ struct panthor_gpu {
 
 static void panthor_gpu_coherency_set(struct panthor_device *ptdev)
 {
-       gpu_write(ptdev, GPU_COHERENCY_PROTOCOL,
-               ptdev->coherent ? GPU_COHERENCY_PROT_BIT(ACE_LITE) : 
GPU_COHERENCY_NONE);
+       u32 coherency_protocol = GPU_COHERENCY_NONE;
+
+       if (ptdev->coherent) {
+               coherency_protocol = GPU_COHERENCY_ACE_LITE;
+
+               if (ptdev->gpu_info.coherency_features &
+                   GPU_COHERENCY_FEATURE_SHAREABLE_CACHE_SUPPORT)
+                       coherency_protocol |=
+                               GPU_COHERENCY_SHAREABLE_CACHE_SUPPORT;
+       }
+
+       gpu_write(ptdev, GPU_COHERENCY_PROTOCOL, coherency_protocol);
 }
 
 static void panthor_gpu_irq_handler(struct panthor_device *ptdev, u32 status)
diff --git a/drivers/gpu/drm/panthor/panthor_hw.c 
b/drivers/gpu/drm/panthor/panthor_hw.c
index a7583342d797..3fcb69a6f959 100644
--- a/drivers/gpu/drm/panthor/panthor_hw.c
+++ b/drivers/gpu/drm/panthor/panthor_hw.c
@@ -35,6 +35,24 @@ static char *get_gpu_model_name(struct panthor_device *ptdev)
                fallthrough;
        case GPU_PROD_ID_MAKE(11, 3):
                return "Mali-G615";
+       case GPU_PROD_ID_MAKE(12, 0):
+               if (shader_core_count >= 10 && ray_intersection)
+                       return "Mali-G720-Immortalis";
+               else if (shader_core_count >= 6)
+                       return "Mali-G720";
+
+               fallthrough;
+       case GPU_PROD_ID_MAKE(12, 1):
+               return "Mali-G620";
+       case GPU_PROD_ID_MAKE(13, 0):
+               if (shader_core_count >= 10 && ray_intersection)
+                       return "Mali-G925-Immortalis";
+               else if (shader_core_count >= 6)
+                       return "Mali-G725";
+
+               fallthrough;
+       case GPU_PROD_ID_MAKE(13, 1):
+               return "Mali-G625";
        }
 
        return "(Unknown Mali GPU)";
diff --git a/drivers/gpu/drm/panthor/panthor_regs.h 
b/drivers/gpu/drm/panthor/panthor_regs.h
index 8bee76d01bf8..1beb365c0fec 100644
--- a/drivers/gpu/drm/panthor/panthor_regs.h
+++ b/drivers/gpu/drm/panthor/panthor_regs.h
@@ -111,12 +111,15 @@
 #define GPU_REVID                                      0x280
 
 #define GPU_COHERENCY_FEATURES                         0x300
-#define GPU_COHERENCY_PROT_BIT(name)                   BIT(GPU_COHERENCY_  ## 
name)
+#define   GPU_COHERENCY_FEATURE_ACE_LITE               BIT(0)
+#define   GPU_COHERENCY_FEATURE_ACE                    BIT(1)
+#define   GPU_COHERENCY_FEATURE_SHAREABLE_CACHE_SUPPORT        BIT(5)
 
 #define GPU_COHERENCY_PROTOCOL                         0x304
 #define   GPU_COHERENCY_ACE_LITE                       0
 #define   GPU_COHERENCY_ACE                            1
 #define   GPU_COHERENCY_NONE                           31
+#define   GPU_COHERENCY_SHAREABLE_CACHE_SUPPORT                BIT(5)
 
 #define MCU_CONTROL                                    0x700
 #define MCU_CONTROL_ENABLE                             1
-- 
2.49.0

Reply via email to