On Mon, Nov 04, 2013 at 10:10:38AM +0100, Thierry Reding wrote: > On Sun, Nov 03, 2013 at 02:31:13PM +0100, Daniel Vetter wrote: > > I'm a bit confused about how this all works wrt host1x clients, but > > this patch looks like the right thing to me. > > > > Cc: Thierry Reding <thierry.reding@xxxxxxxxx> > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > > --- > > drivers/gpu/host1x/drm/drm.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > This has been largely rewritten in the code that I submitted in my 3.13 > pull request. In fact Tegra DRM no longer uses drm_platform, but a > custom bus type to deal with the quirkiness of the multi-driver > architecture. And for what it's worth, there's a drm_host1x_exit() that actually does only drm_put_dev(), which I guess could be simplified in the way that you've done here, but I sort of liked the symmetry between an *_init()/ *_exit() pair of functions. Thierry
Attachment:
pgp7xrocu1UEc.pgp
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel