Re: [PATCH 01/15] drm/arc: Switch to devm_drm_dev_alloc

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

 



On Tue, Jan 12, 2021 at 12:42:16PM +0100, Thomas Zimmermann wrote:
> Hi
> 
> Am 12.01.21 um 09:43 schrieb Daniel Vetter:
> > - Need to embedded the drm_device, but for now we keep the usual
> >    pointer chasing.
> > - No more devm_kzalloc, which fixes a lifetime issues on driver
> >    remove.
> > - No more drm_dev_put, that's done by devm_ now.
> > 
> > Acked-by: Sam Ravnborg <sam@xxxxxxxxxxxx>
> > Cc: Eugeniy Paltsev <Eugeniy.Paltsev@xxxxxxxxxxxx>
> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> > Cc: Alexey Brodkin <abrodkin@xxxxxxxxxxxx>
> 
> For the whole patchset:
> 
> Acked-by: Thomas Zimmermann <tzimmermann@xxxxxxx>
> 
> There's a comment on patch 7.

Queued up the entire pile for 5.13, thanks for taking a look.
-Daniel

> 
> Best regards
> Thomas
> 
> > ---
> >   drivers/gpu/drm/arc/arcpgu.h     |  1 +
> >   drivers/gpu/drm/arc/arcpgu_drv.c | 33 +++++++++++++-------------------
> >   2 files changed, 14 insertions(+), 20 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/arc/arcpgu.h b/drivers/gpu/drm/arc/arcpgu.h
> > index 6aac44b953ad..cd9e932f501e 100644
> > --- a/drivers/gpu/drm/arc/arcpgu.h
> > +++ b/drivers/gpu/drm/arc/arcpgu.h
> > @@ -9,6 +9,7 @@
> >   #define _ARCPGU_H_
> >   struct arcpgu_drm_private {
> > +	struct drm_device	drm;
> >   	void __iomem		*regs;
> >   	struct clk		*clk;
> >   	struct drm_framebuffer	*fb;
> > diff --git a/drivers/gpu/drm/arc/arcpgu_drv.c b/drivers/gpu/drm/arc/arcpgu_drv.c
> > index f164818ec477..68eb4a31c54b 100644
> > --- a/drivers/gpu/drm/arc/arcpgu_drv.c
> > +++ b/drivers/gpu/drm/arc/arcpgu_drv.c
> > @@ -42,18 +42,14 @@ static void arcpgu_setup_mode_config(struct drm_device *drm)
> >   DEFINE_DRM_GEM_CMA_FOPS(arcpgu_drm_ops);
> > -static int arcpgu_load(struct drm_device *drm)
> > +static int arcpgu_load(struct arcpgu_drm_private *arcpgu)
> >   {
> > -	struct platform_device *pdev = to_platform_device(drm->dev);
> > -	struct arcpgu_drm_private *arcpgu;
> > +	struct platform_device *pdev = to_platform_device(arcpgu->drm.dev);
> >   	struct device_node *encoder_node = NULL, *endpoint_node = NULL;
> > +	struct drm_device *drm = &arcpgu->drm;
> >   	struct resource *res;
> >   	int ret;
> > -	arcpgu = devm_kzalloc(&pdev->dev, sizeof(*arcpgu), GFP_KERNEL);
> > -	if (arcpgu == NULL)
> > -		return -ENOMEM;
> > -
> >   	drm->dev_private = arcpgu;
> >   	arcpgu->clk = devm_clk_get(drm->dev, "pxlclk");
> > @@ -162,30 +158,28 @@ static struct drm_driver arcpgu_drm_driver = {
> >   static int arcpgu_probe(struct platform_device *pdev)
> >   {
> > -	struct drm_device *drm;
> > +	struct arcpgu_drm_private *arcpgu;
> >   	int ret;
> > -	drm = drm_dev_alloc(&arcpgu_drm_driver, &pdev->dev);
> > -	if (IS_ERR(drm))
> > -		return PTR_ERR(drm);
> > +	arcpgu = devm_drm_dev_alloc(&pdev->dev, &arcpgu_drm_driver,
> > +				    struct arcpgu_drm_private, drm);
> > +	if (IS_ERR(arcpgu))
> > +		return PTR_ERR(arcpgu);
> > -	ret = arcpgu_load(drm);
> > +	ret = arcpgu_load(arcpgu);
> >   	if (ret)
> > -		goto err_unref;
> > +		return ret;
> > -	ret = drm_dev_register(drm, 0);
> > +	ret = drm_dev_register(&arcpgu->drm, 0);
> >   	if (ret)
> >   		goto err_unload;
> > -	drm_fbdev_generic_setup(drm, 16);
> > +	drm_fbdev_generic_setup(&arcpgu->drm, 16);
> >   	return 0;
> >   err_unload:
> > -	arcpgu_unload(drm);
> > -
> > -err_unref:
> > -	drm_dev_put(drm);
> > +	arcpgu_unload(&arcpgu->drm);
> >   	return ret;
> >   }
> > @@ -196,7 +190,6 @@ static int arcpgu_remove(struct platform_device *pdev)
> >   	drm_dev_unregister(drm);
> >   	arcpgu_unload(drm);
> > -	drm_dev_put(drm);
> >   	return 0;
> >   }
> > 
> 
> -- 
> Thomas Zimmermann
> Graphics Driver Developer
> SUSE Software Solutions Germany GmbH
> Maxfeldstr. 5, 90409 Nürnberg, Germany
> (HRB 36809, AG Nürnberg)
> Geschäftsführer: Felix Imendörffer
> 




-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux