drm/panfrost: Use GPU_MMU_FEATURES_VA_BITS/PA_BITS macros
authorAriel D'Alessandro <ariel.dalessandro@collabora.com>
Mon, 24 Mar 2025 18:57:57 +0000 (15:57 -0300)
committerSteven Price <steven.price@arm.com>
Mon, 31 Mar 2025 11:08:19 +0000 (12:08 +0100)
As done in panthor, define and use these GPU_MMU_FEATURES_* macros,
which makes code easier to read and reuse.

Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: Steven Price <steven.price@arm.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: Adrián Larumbe <adrian.larumbe@collabora.com>
Signed-off-by: Steven Price <steven.price@arm.com>
Link: https://lore.kernel.org/r/20250324185801.168664-3-ariel.dalessandro@collabora.com
drivers/gpu/drm/panfrost/panfrost_mmu.c
drivers/gpu/drm/panfrost/panfrost_regs.h

index d5c136bb64ec1ef07401b6f979c5d981c0dc09d6..fb2ae356802708cff3ae915303c4607c19abdb28 100644 (file)
@@ -615,6 +615,8 @@ static void panfrost_drm_mm_color_adjust(const struct drm_mm_node *node,
 
 struct panfrost_mmu *panfrost_mmu_ctx_create(struct panfrost_device *pfdev)
 {
+       u32 va_bits = GPU_MMU_FEATURES_VA_BITS(pfdev->features.mmu_features);
+       u32 pa_bits = GPU_MMU_FEATURES_PA_BITS(pfdev->features.mmu_features);
        struct panfrost_mmu *mmu;
 
        mmu = kzalloc(sizeof(*mmu), GFP_KERNEL);
@@ -633,8 +635,8 @@ struct panfrost_mmu *panfrost_mmu_ctx_create(struct panfrost_device *pfdev)
 
        mmu->pgtbl_cfg = (struct io_pgtable_cfg) {
                .pgsize_bitmap  = SZ_4K | SZ_2M,
-               .ias            = FIELD_GET(0xff, pfdev->features.mmu_features),
-               .oas            = FIELD_GET(0xff00, pfdev->features.mmu_features),
+               .ias            = va_bits,
+               .oas            = pa_bits,
                .coherent_walk  = pfdev->coherent,
                .tlb            = &mmu_tlb_ops,
                .iommu_dev      = pfdev->dev,
index c7bba476ab3f34d6ae110dc81733b41e35865be7..b5f279a19a0848f25fbac251f63b2fbc1e7cf265 100644 (file)
@@ -16,6 +16,8 @@
 #define   GROUPS_L2_COHERENT           BIT(0)  /* Cores groups are l2 coherent */
 
 #define GPU_MMU_FEATURES               0x014   /* (RO) MMU features */
+#define  GPU_MMU_FEATURES_VA_BITS(x)   ((x) & GENMASK(7, 0))
+#define  GPU_MMU_FEATURES_PA_BITS(x)   (((x) >> 8) & GENMASK(7, 0))
 #define GPU_AS_PRESENT                 0x018   /* (RO) Address space slots present */
 #define GPU_JS_PRESENT                 0x01C   /* (RO) Job slots present */