[PATCH] drm/amdgpu: Remove duplicated power source update

[Alex Deucher]

On Thu, Aug 23, 2018 at 2:40 AM Rex Zhu <Rex.Zhu at amd.com> wrote:
>
> when ac/dc switch, driver will be notified by acpi event.
> then the power source will be updated. so don't need to
> get power source when set power state.

Don't we need this to get the initial state?  Maybe we should move
this to one of the init functions if we don't already check there.

Alex

>
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 8 --------
>  1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> index daa55fb..3e51e9c 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> @@ -1933,14 +1933,6 @@ void amdgpu_pm_compute_clocks(struct amdgpu_device *adev)
>                         amdgpu_fence_wait_empty(ring);
>         }
>
> -       mutex_lock(&adev->pm.mutex);
> -       /* update battery/ac status */
> -       if (power_supply_is_system_supplied() > 0)
> -               adev->pm.ac_power = true;
> -       else
> -               adev->pm.ac_power = false;
> -       mutex_unlock(&adev->pm.mutex);
> -
>         if (adev->powerplay.pp_funcs->dispatch_tasks) {
>                 if (!amdgpu_device_has_dc_support(adev)) {
>                         mutex_lock(&adev->pm.mutex);
> --
> 1.9.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx