[patch] drm/amd/powerplay: fix a reversed condition

Alex Deucher alexdeucher at gmail.com
Mon Jan 4 13:17:58 PST 2016


On Mon, Jan 4, 2016 at 3:42 PM, Dan Carpenter <dan.carpenter at oracle.com> wrote:
> This test was reversed so it would end up leading to a NULL dereference.
>
> Fixes: 4630f0faae80 ('drm/amd/powerplay: add Carrizo smu support')
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>

Applied.  thanks!

Alex

>
> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c
> index e74023b..873a8d2 100644
> --- a/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c
> @@ -818,7 +818,7 @@ static int cz_smu_fini(struct pp_smumgr *smumgr)
>                 return -EINVAL;
>
>         cz_smu = (struct cz_smumgr *)smumgr->backend;
> -       if (!cz_smu) {
> +       if (cz_smu) {
>                 cgs_free_gpu_mem(smumgr->device,
>                                 cz_smu->toc_buffer.handle);
>                 cgs_free_gpu_mem(smumgr->device,
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel


More information about the dri-devel mailing list