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@gmail.com Signed-off-by: Daniel Vetter daniel.vetter@ffwll.ch
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