[PATCH] drm/simpledrm: Fix power domain device link validity check

Thomas Zimmermann tzimmermann at suse.de
Thu Oct 12 08:36:17 UTC 2023


Hi

Am 11.10.23 um 16:32 schrieb Thierry Reding:
> From: Thierry Reding <treding at nvidia.com>
> 
> We need to check if a link is non-NULL before trying to delete it.
> 
> Signed-off-by: Thierry Reding <treding at nvidia.com>

Reviewed-by: Thomas Zimmermann <tzimmermann at suse.de>

I'm going to merge the patch into drm-misc-next with the additional 
Fixes tag

Fixes: 61df9ca23107 ("drm/simpledrm: Add support for multiple 
"power-domains"")

Best regards
Thomas


> ---
>   drivers/gpu/drm/tiny/simpledrm.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/tiny/simpledrm.c b/drivers/gpu/drm/tiny/simpledrm.c
> index 9c597461d1e2..8bdaf66044fc 100644
> --- a/drivers/gpu/drm/tiny/simpledrm.c
> +++ b/drivers/gpu/drm/tiny/simpledrm.c
> @@ -506,7 +506,7 @@ static void simpledrm_device_detach_genpd(void *res)
>   		return;
>   
>   	for (i = sdev->pwr_dom_count - 1; i >= 0; i--) {
> -		if (!sdev->pwr_dom_links[i])
> +		if (sdev->pwr_dom_links[i])
>   			device_link_del(sdev->pwr_dom_links[i]);
>   		if (!IS_ERR_OR_NULL(sdev->pwr_dom_devs[i]))
>   			dev_pm_domain_detach(sdev->pwr_dom_devs[i], true);

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20231012/4858b099/attachment.sig>


More information about the dri-devel mailing list