Re: [PATCH v4 1/2] drm/etnaviv: Check for platform_device_register_simple() failure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Mittwoch, den 27.06.2018, 10:07 -0300 schrieb Fabio Estevam:
> > From: Fabio Estevam <fabio.estevam@xxxxxxx>
> 
> platform_device_register_simple() may fail, so we should better
> check its return value and propagate it in the case of error.
> 
> > Cc: <stable@xxxxxxxxxxxxxxx>
> Fixes: 246774d17fc0 ("drm/etnaviv: remove the need for a gpu-subsystem DT node")
> Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxx>

Thanks Fabio, both applied to etnaviv/fixes.

Regards,
Lucas

> ---
> Changes since v3:
> - Only set etnaviv_drm when platform_device_register_simple()
> succeeds (Phillip)
> 
>  drivers/gpu/drm/etnaviv/etnaviv_drv.c | 21 ++++++++++++++++++---
>  1 file changed, 18 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_drv.c b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> index e5013a9..f8d264a 100644
> --- a/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> +++ b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> @@ -631,8 +631,11 @@ static struct platform_driver etnaviv_platform_driver = {
> >  	},
>  };
>  
> +static struct platform_device *etnaviv_drm;
> +
>  static int __init etnaviv_init(void)
>  {
> > +	struct platform_device *pdev;
> >  	int ret;
> >  	struct device_node *np;
>  
> @@ -644,7 +647,7 @@ static int __init etnaviv_init(void)
>  
> >  	ret = platform_driver_register(&etnaviv_platform_driver);
> >  	if (ret != 0)
> > -		platform_driver_unregister(&etnaviv_gpu_driver);
> > +		goto unregister_gpu_driver;
>  
> >  	/*
> >  	 * If the DT contains at least one available GPU device, instantiate
> @@ -653,12 +656,24 @@ static int __init etnaviv_init(void)
> >  	for_each_compatible_node(np, NULL, "vivante,gc") {
> >  		if (!of_device_is_available(np))
> >  			continue;
> -
> > -		platform_device_register_simple("etnaviv", -1, NULL, 0);
> > +		pdev = platform_device_register_simple("etnaviv", -1,
> > +						       NULL, 0);
> > +		if (IS_ERR(pdev)) {
> > +			ret = PTR_ERR(pdev);
> > +			of_node_put(np);
> > +			goto unregister_platform_driver;
> > +		}
> > +		etnaviv_drm = pdev;
> >  		of_node_put(np);
> >  		break;
> >  	}
>  
> > +	return 0;
> +
> +unregister_platform_driver:
> > +	platform_driver_unregister(&etnaviv_platform_driver);
> +unregister_gpu_driver:
> > +	platform_driver_unregister(&etnaviv_gpu_driver);
> >  	return ret;
>  }
>  module_init(etnaviv_init);



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux