[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Mon, 11 Nov 2019 15:35:58 -0500
From: Alex Deucher <alexdeucher@...il.com>
To: Kyle Mahlkuch <kmahlkuc@...ux.vnet.ibm.com>
Cc: Chunming Zhou <David1.Zhou@....com>,
LKML <linux-kernel@...r.kernel.org>,
amd-gfx list <amd-gfx@...ts.freedesktop.org>,
"Deucher, Alexander" <alexander.deucher@....com>,
Christian Koenig <christian.koenig@....com>
Subject: Re: [PATCH] drm/radeon: Clean up code in radeon_pci_shutdown()
On Mon, Nov 11, 2019 at 3:29 PM Kyle Mahlkuch
<kmahlkuc@...ux.vnet.ibm.com> wrote:
>
> From: KyleMahlkuch <kmahlkuc@...ux.vnet.ibm.com>
>
> This fixes the formatting on one comment and consolidates the
> pci_get_drvdata() into the radeon_suspend_kms().
>
> Signed-off-by: Kyle Mahlkuch <kmahlkuc@...ux.vnet.ibm.com>
Applied. Thanks!
Alex
> ---
> drivers/gpu/drm/radeon/radeon_drv.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c
> index 4528f4d..357d29a 100644
> --- a/drivers/gpu/drm/radeon/radeon_drv.c
> +++ b/drivers/gpu/drm/radeon/radeon_drv.c
> @@ -379,10 +379,6 @@ static int radeon_pci_probe(struct pci_dev *pdev,
> static void
> radeon_pci_shutdown(struct pci_dev *pdev)
> {
> -#ifdef CONFIG_PPC64
> - struct drm_device *ddev = pci_get_drvdata(pdev);
> -#endif
> -
> /* if we are running in a VM, make sure the device
> * torn down properly on reboot/shutdown
> */
> @@ -390,13 +386,14 @@ static int radeon_pci_probe(struct pci_dev *pdev,
> radeon_pci_remove(pdev);
>
> #ifdef CONFIG_PPC64
> - /* Some adapters need to be suspended before a
> + /*
> + * Some adapters need to be suspended before a
> * shutdown occurs in order to prevent an error
> * during kexec.
> * Make this power specific becauase it breaks
> * some non-power boards.
> */
> - radeon_suspend_kms(ddev, true, true, false);
> + radeon_suspend_kms(pci_get_drvdata(pdev), true, true, false);
> #endif
> }
>
> --
> 1.8.3.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@...ts.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Powered by blists - more mailing lists