Am 06.11.2017 um 12:43 schrieb Dan Carpenter: > backlight_device_register() never returns NULL, it returns error > pointers on error so the check here is wrong. > > Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com> Acked-by: Christian König <christian.koenig at amd.com> > > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > index 33a15a1d818c..75f9383f5b9b 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > @@ -1296,7 +1296,7 @@ amdgpu_dm_register_backlight_device(struct amdgpu_display_manager *dm) > &amdgpu_dm_backlight_ops, > &props); > > - if (NULL == dm->backlight_dev) > + if (IS_ERR(dm->backlight_dev)) > DRM_ERROR("DM: Backlight registration failed!\n"); > else > DRM_DEBUG_DRIVER("DM: Registered Backlight device: %s\n", bl_name);