[PATCH 1/2] drm/amd/powerplay: refine code in cz_smumgr.c

Alex Deucher alexdeucher at gmail.com
Sat Dec 31 06:30:41 UTC 2016


On Fri, Dec 23, 2016 at 4:50 AM, Rex Zhu <Rex.Zhu at amd.com> wrote:
> cz_smu_init will be called in sw_init.
> so it should not touch other blocks's
> firmware as they were not ready.
>
> Change-Id: I3af8cce2059aaacfc260e24a41b1af46a0aeb18b
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
> Reviewed-by: Huang Rui <ray.huang at amd.com>

Series is:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c | 135 ++++++++++++-----------
>  1 file changed, 70 insertions(+), 65 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c
> index 5a44485..3ea437c 100644
> --- a/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c
> @@ -141,41 +141,6 @@ static int cz_send_msg_to_smc_with_parameter(struct pp_smumgr *smumgr,
>         return cz_send_msg_to_smc(smumgr, msg);
>  }
>
> -static int cz_request_smu_load_fw(struct pp_smumgr *smumgr)
> -{
> -       struct cz_smumgr *cz_smu = (struct cz_smumgr *)(smumgr->backend);
> -       uint32_t smc_address;
> -
> -       if (!smumgr->reload_fw) {
> -               printk(KERN_INFO "[ powerplay ] skip reloading...\n");
> -               return 0;
> -       }
> -
> -       smc_address = SMU8_FIRMWARE_HEADER_LOCATION +
> -               offsetof(struct SMU8_Firmware_Header, UcodeLoadStatus);
> -
> -       cz_write_smc_sram_dword(smumgr, smc_address, 0, smc_address+4);
> -
> -       cz_send_msg_to_smc_with_parameter(smumgr,
> -                                       PPSMC_MSG_DriverDramAddrHi,
> -                                       cz_smu->toc_buffer.mc_addr_high);
> -
> -       cz_send_msg_to_smc_with_parameter(smumgr,
> -                                       PPSMC_MSG_DriverDramAddrLo,
> -                                       cz_smu->toc_buffer.mc_addr_low);
> -
> -       cz_send_msg_to_smc(smumgr, PPSMC_MSG_InitJobs);
> -
> -       cz_send_msg_to_smc_with_parameter(smumgr,
> -                                       PPSMC_MSG_ExecuteJob,
> -                                       cz_smu->toc_entry_aram);
> -       cz_send_msg_to_smc_with_parameter(smumgr, PPSMC_MSG_ExecuteJob,
> -                               cz_smu->toc_entry_power_profiling_index);
> -
> -       return cz_send_msg_to_smc_with_parameter(smumgr,
> -                                       PPSMC_MSG_ExecuteJob,
> -                                       cz_smu->toc_entry_initialize_index);
> -}
>
>  static int cz_check_fw_load_finish(struct pp_smumgr *smumgr,
>                                    uint32_t firmware)
> @@ -250,34 +215,6 @@ static int cz_load_mec_firmware(struct pp_smumgr *smumgr)
>         return 0;
>  }
>
> -static int cz_start_smu(struct pp_smumgr *smumgr)
> -{
> -       int ret = 0;
> -       uint32_t fw_to_check = UCODE_ID_RLC_G_MASK |
> -                               UCODE_ID_SDMA0_MASK |
> -                               UCODE_ID_SDMA1_MASK |
> -                               UCODE_ID_CP_CE_MASK |
> -                               UCODE_ID_CP_ME_MASK |
> -                               UCODE_ID_CP_PFP_MASK |
> -                               UCODE_ID_CP_MEC_JT1_MASK |
> -                               UCODE_ID_CP_MEC_JT2_MASK;
> -
> -       if (smumgr->chip_id == CHIP_STONEY)
> -               fw_to_check &= ~(UCODE_ID_SDMA1_MASK | UCODE_ID_CP_MEC_JT2_MASK);
> -
> -       ret = cz_request_smu_load_fw(smumgr);
> -       if (ret)
> -               printk(KERN_ERR "[ powerplay] SMU firmware load failed\n");
> -
> -       cz_check_fw_load_finish(smumgr, fw_to_check);
> -
> -       ret = cz_load_mec_firmware(smumgr);
> -       if (ret)
> -               printk(KERN_ERR "[ powerplay ] Mec Firmware load failed\n");
> -
> -       return ret;
> -}
> -
>  static uint8_t cz_translate_firmware_enum_to_arg(struct pp_smumgr *smumgr,
>                         enum cz_scratch_entry firmware_enum)
>  {
> @@ -729,6 +666,76 @@ static int cz_upload_pptable_settings(struct pp_smumgr *smumgr)
>         return 0;
>  }
>
> +static int cz_request_smu_load_fw(struct pp_smumgr *smumgr)
> +{
> +       struct cz_smumgr *cz_smu = (struct cz_smumgr *)(smumgr->backend);
> +       uint32_t smc_address;
> +
> +       if (!smumgr->reload_fw) {
> +               printk(KERN_INFO "[ powerplay ] skip reloading...\n");
> +               return 0;
> +       }
> +
> +       cz_smu_populate_firmware_entries(smumgr);
> +
> +       cz_smu_construct_toc(smumgr);
> +
> +       smc_address = SMU8_FIRMWARE_HEADER_LOCATION +
> +               offsetof(struct SMU8_Firmware_Header, UcodeLoadStatus);
> +
> +       cz_write_smc_sram_dword(smumgr, smc_address, 0, smc_address+4);
> +
> +       cz_send_msg_to_smc_with_parameter(smumgr,
> +                                       PPSMC_MSG_DriverDramAddrHi,
> +                                       cz_smu->toc_buffer.mc_addr_high);
> +
> +       cz_send_msg_to_smc_with_parameter(smumgr,
> +                                       PPSMC_MSG_DriverDramAddrLo,
> +                                       cz_smu->toc_buffer.mc_addr_low);
> +
> +       cz_send_msg_to_smc(smumgr, PPSMC_MSG_InitJobs);
> +
> +       cz_send_msg_to_smc_with_parameter(smumgr,
> +                                       PPSMC_MSG_ExecuteJob,
> +                                       cz_smu->toc_entry_aram);
> +       cz_send_msg_to_smc_with_parameter(smumgr, PPSMC_MSG_ExecuteJob,
> +                               cz_smu->toc_entry_power_profiling_index);
> +
> +       return cz_send_msg_to_smc_with_parameter(smumgr,
> +                                       PPSMC_MSG_ExecuteJob,
> +                                       cz_smu->toc_entry_initialize_index);
> +}
> +
> +static int cz_start_smu(struct pp_smumgr *smumgr)
> +{
> +       int ret = 0;
> +       uint32_t fw_to_check = 0;
> +
> +       fw_to_check = UCODE_ID_RLC_G_MASK |
> +                       UCODE_ID_SDMA0_MASK |
> +                       UCODE_ID_SDMA1_MASK |
> +                       UCODE_ID_CP_CE_MASK |
> +                       UCODE_ID_CP_ME_MASK |
> +                       UCODE_ID_CP_PFP_MASK |
> +                       UCODE_ID_CP_MEC_JT1_MASK |
> +                       UCODE_ID_CP_MEC_JT2_MASK;
> +
> +       if (smumgr->chip_id == CHIP_STONEY)
> +               fw_to_check &= ~(UCODE_ID_SDMA1_MASK | UCODE_ID_CP_MEC_JT2_MASK);
> +
> +       ret = cz_request_smu_load_fw(smumgr);
> +       if (ret)
> +               printk(KERN_ERR "[ powerplay] SMU firmware load failed\n");
> +
> +       cz_check_fw_load_finish(smumgr, fw_to_check);
> +
> +       ret = cz_load_mec_firmware(smumgr);
> +       if (ret)
> +               printk(KERN_ERR "[ powerplay ] Mec Firmware load failed\n");
> +
> +       return ret;
> +}
> +
>  static int cz_smu_init(struct pp_smumgr *smumgr)
>  {
>         struct cz_smumgr *cz_smu = (struct cz_smumgr *)smumgr->backend;
> @@ -769,7 +776,6 @@ static int cz_smu_init(struct pp_smumgr *smumgr)
>         cz_smu->smu_buffer.mc_addr_high = smu_upper_32_bits(mc_addr);
>         cz_smu->smu_buffer.mc_addr_low = smu_lower_32_bits(mc_addr);
>
> -       cz_smu_populate_firmware_entries(smumgr);
>         if (0 != cz_smu_populate_single_scratch_entry(smumgr,
>                 CZ_SCRATCH_ENTRY_UCODE_ID_RLC_SCRATCH,
>                 UCODE_ID_RLC_SCRATCH_SIZE_BYTE,
> @@ -808,7 +814,6 @@ static int cz_smu_init(struct pp_smumgr *smumgr)
>                 printk(KERN_ERR "[ powerplay ] Error when Populate Firmware Entry.\n");
>                 return -1;
>         }
> -       cz_smu_construct_toc(smumgr);
>
>         return 0;
>  }
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list