From: Lijo Lazar <lijo.la...@amd.com>

Keep a helper function to get description of compute partition mode.

Signed-off-by: Lijo Lazar <lijo.la...@amd.com>
Reviewed-by: Le Ma <le...@amd.com>
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 24 +-----------------------
 drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 21 +++++++++++++++++++++
 2 files changed, 22 insertions(+), 23 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
index 817e7b7d32b7..2ebf5c6f4ff7 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
@@ -1169,33 +1169,11 @@ static ssize_t 
amdgpu_gfx_get_current_compute_partition(struct device *dev,
        struct drm_device *ddev = dev_get_drvdata(dev);
        struct amdgpu_device *adev = drm_to_adev(ddev);
        int mode;
-       char *partition_mode;
 
        mode = amdgpu_xcp_query_partition_mode(adev->xcp_mgr,
                                               AMDGPU_XCP_FL_NONE);
 
-       switch (mode) {
-       case AMDGPU_SPX_PARTITION_MODE:
-               partition_mode = "SPX";
-               break;
-       case AMDGPU_DPX_PARTITION_MODE:
-               partition_mode = "DPX";
-               break;
-       case AMDGPU_TPX_PARTITION_MODE:
-               partition_mode = "TPX";
-               break;
-       case AMDGPU_QPX_PARTITION_MODE:
-               partition_mode = "QPX";
-               break;
-       case AMDGPU_CPX_PARTITION_MODE:
-               partition_mode = "CPX";
-               break;
-       default:
-               partition_mode = "UNKNOWN";
-               break;
-       }
-
-       return sysfs_emit(buf, "%s\n", partition_mode);
+       return sysfs_emit(buf, "%s\n", amdgpu_gfx_compute_mode_desc(mode));
 }
 
 static ssize_t amdgpu_gfx_set_compute_partition(struct device *dev,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
index e9c93f6e12b8..3d11b7a0bd75 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
@@ -494,4 +494,25 @@ int amdgpu_gfx_poison_consumption_handler(struct 
amdgpu_device *adev,
 bool amdgpu_gfx_is_master_xcc(struct amdgpu_device *adev, int xcc_id);
 int amdgpu_gfx_sysfs_init(struct amdgpu_device *adev);
 void amdgpu_gfx_sysfs_fini(struct amdgpu_device *adev);
+
+static inline const char *amdgpu_gfx_compute_mode_desc(int mode)
+{
+       switch (mode) {
+       case AMDGPU_SPX_PARTITION_MODE:
+               return "SPX";
+       case AMDGPU_DPX_PARTITION_MODE:
+               return "DPX";
+       case AMDGPU_TPX_PARTITION_MODE:
+               return "TPX";
+       case AMDGPU_QPX_PARTITION_MODE:
+               return "QPX";
+       case AMDGPU_CPX_PARTITION_MODE:
+               return "CPX";
+       default:
+               return "UNKNOWN";
+       }
+
+       return "UNKNOWN";
+}
+
 #endif
-- 
2.40.1

Reply via email to