[PATCH 2/2] drm/amd/pp: Remove useless smu7 running state check

[Alex Deucher]

On Wed, Apr 4, 2018 at 6:44 AM, Rex Zhu <Rex.Zhu at amd.com> wrote:
> Only check smc running state before start smu.
>
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>

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

> ---
>  drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c | 8 --------
>  1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c
> index cbace93..10a1123 100644
> --- a/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c
> @@ -167,10 +167,6 @@ int smu7_send_msg_to_smc(struct pp_hwmgr *hwmgr, uint16_t msg)
>  {
>         int ret;
>
> -       if (!smu7_is_smc_ram_running(hwmgr))
> -               return -EINVAL;
> -
> -
>         PHM_WAIT_FIELD_UNEQUAL(hwmgr, SMC_RESP_0, SMC_RESP, 0);
>
>         ret = PHM_READ_FIELD(hwmgr->device, SMC_RESP_0, SMC_RESP);
> @@ -203,10 +199,6 @@ int smu7_send_msg_to_smc_without_waiting(struct pp_hwmgr *hwmgr, uint16_t msg)
>
>  int smu7_send_msg_to_smc_with_parameter(struct pp_hwmgr *hwmgr, uint16_t msg, uint32_t parameter)
>  {
> -       if (!smu7_is_smc_ram_running(hwmgr)) {
> -               return -EINVAL;
> -       }
> -
>         PHM_WAIT_FIELD_UNEQUAL(hwmgr, SMC_RESP_0, SMC_RESP, 0);
>
>         cgs_write_register(hwmgr->device, mmSMC_MSG_ARG_0, parameter);
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx