On Tue, Jan 15, 2013 at 01:44:02PM +0200, Terje Bergstrom wrote: [...] > +void host1x_set_drm_data(struct platform_device *pdev, void *data) > +{ > + struct host1x *host1x = platform_get_drvdata(pdev); > + host1x->drm_data = data; > +} > + > +void *host1x_get_drm_data(struct platform_device *pdev) > +{ > + struct host1x *host1x = platform_get_drvdata(pdev); > + return host1x->drm_data; > +} Passing around struct device * should be enough and avoids the need for the explicit cast to struct platform_device. It is a bit unfortunate that we have now have two structures called host1x, but I think we can live with it for now. We can clean that up once the code has been merged. Thierry
Attachment:
pgpmxOGT91DaX.pgp
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel