Am 2022-11-16 um 11:40 schrieb Alex Deucher:
All of the IP specific versions are the same now, so
we can just use a common function.

Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>

The series is

Acked-by: Felix Kuehling <felix.kuehl...@amd.com>


---
  drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c  | 26 +++++++++++++++++++++++
  drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h  |  2 --
  drivers/gpu/drm/amd/amdgpu/psp_v10_0.c   | 27 ------------------------
  drivers/gpu/drm/amd/amdgpu/psp_v11_0.c   | 27 ------------------------
  drivers/gpu/drm/amd/amdgpu/psp_v11_0_8.c | 27 ------------------------
  drivers/gpu/drm/amd/amdgpu/psp_v12_0.c   | 27 ------------------------
  drivers/gpu/drm/amd/amdgpu/psp_v13_0.c   | 27 ------------------------
  drivers/gpu/drm/amd/amdgpu/psp_v13_0_4.c | 27 ------------------------
  drivers/gpu/drm/amd/amdgpu/psp_v3_1.c    | 27 ------------------------
  9 files changed, 26 insertions(+), 191 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
index d9cb4c4b8289..7bb2de1d11ff 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
@@ -52,6 +52,32 @@ static int psp_load_smu_fw(struct psp_context *psp);
  static int psp_rap_terminate(struct psp_context *psp);
  static int psp_securedisplay_terminate(struct psp_context *psp);
+static int psp_ring_init(struct psp_context *psp,
+                        enum psp_ring_type ring_type)
+{
+       int ret = 0;
+       struct psp_ring *ring;
+       struct amdgpu_device *adev = psp->adev;
+
+       ring = &psp->km_ring;
+
+       ring->ring_type = ring_type;
+
+       /* allocate 4k Page of Local Frame Buffer memory for ring */
+       ring->ring_size = 0x1000;
+       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
+                                     AMDGPU_GEM_DOMAIN_VRAM,
+                                     &adev->firmware.rbuf,
+                                     &ring->ring_mem_mc_addr,
+                                     (void **)&ring->ring_mem);
+       if (ret) {
+               ring->ring_size = 0;
+               return ret;
+       }
+
+       return 0;
+}
+
  /*
   * Due to DF Cstate management centralized to PMFW, the firmware
   * loading sequence will be updated as below:
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h
index cbd4194a2883..cf4f60c66122 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h
@@ -118,7 +118,6 @@ struct psp_funcs
        int (*bootloader_load_dbg_drv)(struct psp_context *psp);
        int (*bootloader_load_ras_drv)(struct psp_context *psp);
        int (*bootloader_load_sos)(struct psp_context *psp);
-       int (*ring_init)(struct psp_context *psp, enum psp_ring_type ring_type);
        int (*ring_create)(struct psp_context *psp,
                           enum psp_ring_type ring_type);
        int (*ring_stop)(struct psp_context *psp,
@@ -396,7 +395,6 @@ struct amdgpu_psp_funcs {
  };
-#define psp_ring_init(psp, type) (psp)->funcs->ring_init((psp), (type))
  #define psp_ring_create(psp, type) (psp)->funcs->ring_create((psp), (type))
  #define psp_ring_stop(psp, type) (psp)->funcs->ring_stop((psp), (type))
  #define psp_ring_destroy(psp, type) ((psp)->funcs->ring_destroy((psp), 
(type)))
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c
index ed2293686f0d..9de46fa8f46c 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c
@@ -126,32 +126,6 @@ static int psp_v10_0_init_microcode(struct psp_context 
*psp)
        return err;
  }
-static int psp_v10_0_ring_init(struct psp_context *psp,
-                              enum psp_ring_type ring_type)
-{
-       int ret = 0;
-       struct psp_ring *ring;
-       struct amdgpu_device *adev = psp->adev;
-
-       ring = &psp->km_ring;
-
-       ring->ring_type = ring_type;
-
-       /* allocate 4k Page of Local Frame Buffer memory for ring */
-       ring->ring_size = 0x1000;
-       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
-                                     AMDGPU_GEM_DOMAIN_VRAM,
-                                     &adev->firmware.rbuf,
-                                     &ring->ring_mem_mc_addr,
-                                     (void **)&ring->ring_mem);
-       if (ret) {
-               ring->ring_size = 0;
-               return ret;
-       }
-
-       return 0;
-}
-
  static int psp_v10_0_ring_create(struct psp_context *psp,
                                 enum psp_ring_type ring_type)
  {
@@ -245,7 +219,6 @@ static void psp_v10_0_ring_set_wptr(struct psp_context 
*psp, uint32_t value)
static const struct psp_funcs psp_v10_0_funcs = {
        .init_microcode = psp_v10_0_init_microcode,
-       .ring_init = psp_v10_0_ring_init,
        .ring_create = psp_v10_0_ring_create,
        .ring_stop = psp_v10_0_ring_stop,
        .ring_destroy = psp_v10_0_ring_destroy,
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c
index 9518b4394a6e..bd3e3e23a939 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c
@@ -360,32 +360,6 @@ static int psp_v11_0_bootloader_load_sos(struct 
psp_context *psp)
        return ret;
  }
-static int psp_v11_0_ring_init(struct psp_context *psp,
-                             enum psp_ring_type ring_type)
-{
-       int ret = 0;
-       struct psp_ring *ring;
-       struct amdgpu_device *adev = psp->adev;
-
-       ring = &psp->km_ring;
-
-       ring->ring_type = ring_type;
-
-       /* allocate 4k Page of Local Frame Buffer memory for ring */
-       ring->ring_size = 0x1000;
-       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
-                                     AMDGPU_GEM_DOMAIN_VRAM,
-                                     &adev->firmware.rbuf,
-                                     &ring->ring_mem_mc_addr,
-                                     (void **)&ring->ring_mem);
-       if (ret) {
-               ring->ring_size = 0;
-               return ret;
-       }
-
-       return 0;
-}
-
  static int psp_v11_0_ring_stop(struct psp_context *psp,
                              enum psp_ring_type ring_type)
  {
@@ -779,7 +753,6 @@ static const struct psp_funcs psp_v11_0_funcs = {
        .bootloader_load_spl = psp_v11_0_bootloader_load_spl,
        .bootloader_load_sysdrv = psp_v11_0_bootloader_load_sysdrv,
        .bootloader_load_sos = psp_v11_0_bootloader_load_sos,
-       .ring_init = psp_v11_0_ring_init,
        .ring_create = psp_v11_0_ring_create,
        .ring_stop = psp_v11_0_ring_stop,
        .ring_destroy = psp_v11_0_ring_destroy,
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v11_0_8.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v11_0_8.c
index ff13e1beb49b..5697760a819b 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v11_0_8.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v11_0_8.c
@@ -28,32 +28,6 @@
#include "mp/mp_11_0_8_offset.h" -static int psp_v11_0_8_ring_init(struct psp_context *psp,
-                             enum psp_ring_type ring_type)
-{
-       int ret = 0;
-       struct psp_ring *ring;
-       struct amdgpu_device *adev = psp->adev;
-
-       ring = &psp->km_ring;
-
-       ring->ring_type = ring_type;
-
-       /* allocate 4k Page of Local Frame Buffer memory for ring */
-       ring->ring_size = 0x1000;
-       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
-                                     AMDGPU_GEM_DOMAIN_VRAM,
-                                     &adev->firmware.rbuf,
-                                     &ring->ring_mem_mc_addr,
-                                     (void **)&ring->ring_mem);
-       if (ret) {
-               ring->ring_size = 0;
-               return ret;
-       }
-
-       return 0;
-}
-
  static int psp_v11_0_8_ring_stop(struct psp_context *psp,
                               enum psp_ring_type ring_type)
  {
@@ -194,7 +168,6 @@ static void psp_v11_0_8_ring_set_wptr(struct psp_context 
*psp, uint32_t value)
  }
static const struct psp_funcs psp_v11_0_8_funcs = {
-       .ring_init = psp_v11_0_8_ring_init,
        .ring_create = psp_v11_0_8_ring_create,
        .ring_stop = psp_v11_0_8_ring_stop,
        .ring_destroy = psp_v11_0_8_ring_destroy,
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c
index d7dd84090c09..8ed2281b6557 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c
@@ -236,32 +236,6 @@ static void psp_v12_0_reroute_ih(struct psp_context *psp)
                     0x80000000, 0x8000FFFF, false);
  }
-static int psp_v12_0_ring_init(struct psp_context *psp,
-                             enum psp_ring_type ring_type)
-{
-       int ret = 0;
-       struct psp_ring *ring;
-       struct amdgpu_device *adev = psp->adev;
-
-       ring = &psp->km_ring;
-
-       ring->ring_type = ring_type;
-
-       /* allocate 4k Page of Local Frame Buffer memory for ring */
-       ring->ring_size = 0x1000;
-       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
-                                     AMDGPU_GEM_DOMAIN_VRAM,
-                                     &adev->firmware.rbuf,
-                                     &ring->ring_mem_mc_addr,
-                                     (void **)&ring->ring_mem);
-       if (ret) {
-               ring->ring_size = 0;
-               return ret;
-       }
-
-       return 0;
-}
-
  static int psp_v12_0_ring_create(struct psp_context *psp,
                                enum psp_ring_type ring_type)
  {
@@ -425,7 +399,6 @@ static const struct psp_funcs psp_v12_0_funcs = {
        .init_microcode = psp_v12_0_init_microcode,
        .bootloader_load_sysdrv = psp_v12_0_bootloader_load_sysdrv,
        .bootloader_load_sos = psp_v12_0_bootloader_load_sos,
-       .ring_init = psp_v12_0_ring_init,
        .ring_create = psp_v12_0_ring_create,
        .ring_stop = psp_v12_0_ring_stop,
        .ring_destroy = psp_v12_0_ring_destroy,
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
index 88f9b327183a..ee27bfaba6fd 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
@@ -268,32 +268,6 @@ static int psp_v13_0_bootloader_load_sos(struct 
psp_context *psp)
        return ret;
  }
-static int psp_v13_0_ring_init(struct psp_context *psp,
-                             enum psp_ring_type ring_type)
-{
-       int ret = 0;
-       struct psp_ring *ring;
-       struct amdgpu_device *adev = psp->adev;
-
-       ring = &psp->km_ring;
-
-       ring->ring_type = ring_type;
-
-       /* allocate 4k Page of Local Frame Buffer memory for ring */
-       ring->ring_size = 0x1000;
-       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
-                                     AMDGPU_GEM_DOMAIN_VRAM,
-                                     &adev->firmware.rbuf,
-                                     &ring->ring_mem_mc_addr,
-                                     (void **)&ring->ring_mem);
-       if (ret) {
-               ring->ring_size = 0;
-               return ret;
-       }
-
-       return 0;
-}
-
  static int psp_v13_0_ring_stop(struct psp_context *psp,
                               enum psp_ring_type ring_type)
  {
@@ -729,7 +703,6 @@ static const struct psp_funcs psp_v13_0_funcs = {
        .bootloader_load_dbg_drv = psp_v13_0_bootloader_load_dbg_drv,
        .bootloader_load_ras_drv = psp_v13_0_bootloader_load_ras_drv,
        .bootloader_load_sos = psp_v13_0_bootloader_load_sos,
-       .ring_init = psp_v13_0_ring_init,
        .ring_create = psp_v13_0_ring_create,
        .ring_stop = psp_v13_0_ring_stop,
        .ring_destroy = psp_v13_0_ring_destroy,
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v13_0_4.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v13_0_4.c
index 321089dfa7db..9d4e24e518e8 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v13_0_4.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v13_0_4.c
@@ -199,32 +199,6 @@ static int psp_v13_0_4_bootloader_load_sos(struct 
psp_context *psp)
        return ret;
  }
-static int psp_v13_0_4_ring_init(struct psp_context *psp,
-                             enum psp_ring_type ring_type)
-{
-       int ret = 0;
-       struct psp_ring *ring;
-       struct amdgpu_device *adev = psp->adev;
-
-       ring = &psp->km_ring;
-
-       ring->ring_type = ring_type;
-
-       /* allocate 4k Page of Local Frame Buffer memory for ring */
-       ring->ring_size = 0x1000;
-       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
-                                     AMDGPU_GEM_DOMAIN_VRAM,
-                                     &adev->firmware.rbuf,
-                                     &ring->ring_mem_mc_addr,
-                                     (void **)&ring->ring_mem);
-       if (ret) {
-               ring->ring_size = 0;
-               return ret;
-       }
-
-       return 0;
-}
-
  static int psp_v13_0_4_ring_stop(struct psp_context *psp,
                               enum psp_ring_type ring_type)
  {
@@ -373,7 +347,6 @@ static const struct psp_funcs psp_v13_0_4_funcs = {
        .bootloader_load_intf_drv = psp_v13_0_4_bootloader_load_intf_drv,
        .bootloader_load_dbg_drv = psp_v13_0_4_bootloader_load_dbg_drv,
        .bootloader_load_sos = psp_v13_0_4_bootloader_load_sos,
-       .ring_init = psp_v13_0_4_ring_init,
        .ring_create = psp_v13_0_4_ring_create,
        .ring_stop = psp_v13_0_4_ring_stop,
        .ring_destroy = psp_v13_0_4_ring_destroy,
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
index 01f3bcc62a6c..157147c6c94e 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
@@ -160,32 +160,6 @@ static int psp_v3_1_bootloader_load_sos(struct psp_context 
*psp)
        return ret;
  }
-static int psp_v3_1_ring_init(struct psp_context *psp,
-                             enum psp_ring_type ring_type)
-{
-       int ret = 0;
-       struct psp_ring *ring;
-       struct amdgpu_device *adev = psp->adev;
-
-       ring = &psp->km_ring;
-
-       ring->ring_type = ring_type;
-
-       /* allocate 4k Page of Local Frame Buffer memory for ring */
-       ring->ring_size = 0x1000;
-       ret = amdgpu_bo_create_kernel(adev, ring->ring_size, PAGE_SIZE,
-                                     AMDGPU_GEM_DOMAIN_VRAM,
-                                     &adev->firmware.rbuf,
-                                     &ring->ring_mem_mc_addr,
-                                     (void **)&ring->ring_mem);
-       if (ret) {
-               ring->ring_size = 0;
-               return ret;
-       }
-
-       return 0;
-}
-
  static void psp_v3_1_reroute_ih(struct psp_context *psp)
  {
        struct amdgpu_device *adev = psp->adev;
@@ -401,7 +375,6 @@ static const struct psp_funcs psp_v3_1_funcs = {
        .init_microcode = psp_v3_1_init_microcode,
        .bootloader_load_sysdrv = psp_v3_1_bootloader_load_sysdrv,
        .bootloader_load_sos = psp_v3_1_bootloader_load_sos,
-       .ring_init = psp_v3_1_ring_init,
        .ring_create = psp_v3_1_ring_create,
        .ring_stop = psp_v3_1_ring_stop,
        .ring_destroy = psp_v3_1_ring_destroy,

Reply via email to