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