From: Thierry Reding <thierry.reding@gmail.com>
To: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: DRI Development <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 07/19] drm/host1x: Call drm_put_dev directly instead of drm_platform_exit
Date: Mon, 4 Nov 2013 11:51:59 +0100 [thread overview]
Message-ID: <20131104105158.GM27445@ulmo.nvidia.com> (raw)
In-Reply-To: <CAKMK7uH1bQwPWPLXH2qojHWghnJ8pd-_HLBxfQUn4oPmvFKz6Q@mail.gmail.com>
[-- Attachment #1.1: Type: text/plain, Size: 1686 bytes --]
On Mon, Nov 04, 2013 at 11:14:39AM +0100, Daniel Vetter wrote:
> On Mon, Nov 4, 2013 at 10:10 AM, Thierry Reding
> <thierry.reding@gmail.com> 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.
> >
> > So if you really want to get rid of drm_bus altogether I'd appreciate a
> > heads-up so I know what's coming and can prepare.
>
> It's almost completely gone now, the one thing left to do is rip out
> bus->set_busid. But I have that planned already.
>
> http://cgit.freedesktop.org/~danvet/drm/log/?h=drm-init-cleanup
>
> I guess I get to rebase the entire thing again, meh.
>
> <rant>
> It'd be awesome of other drm drivers would submit their stuff a bit
> earlier, atm there's absolutely nothing in drm-next beside all of
> intel and a few things for armada... Maybe we need to have a drm
> integration tree to make this a bit less frustrating.
> </rant>
Yeah, I know. I had actually planned on sending out this pull request
much earlier, but then all the discussion around DRM panel and device
tree bindings started again...
Thierry
[-- Attachment #1.2: Type: application/pgp-signature, Size: 836 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2013-11-04 10:52 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-03 13:31 [PATCH 00/19] drm de-midlayer, part 1: device list and agp bus stuff Daniel Vetter
2013-11-03 13:31 ` [PATCH 01/19] drm/rcar: call drm_put_dev directly in the ->remove hook Daniel Vetter
2013-11-03 13:47 ` Laurent Pinchart
2013-11-03 13:31 ` [PATCH 02/19] drm/exynos: call drm_put_dev directly from ->remove Daniel Vetter
2013-11-03 13:31 ` [PATCH 03/19] drm/imx: directly call drm_put_dev in ->remove Daniel Vetter
2013-11-04 8:19 ` Sascha Hauer
2013-11-03 13:31 ` [PATCH 04/19] drm/tilcdc: call drm_put_dev directly from ->remove Daniel Vetter
2013-11-03 13:31 ` [PATCH 05/19] drm/omap: call drm_put_dev directly in ->remove Daniel Vetter
2013-11-03 13:31 ` [PATCH 06/19] drm/shmob: call drm_put_dev directly from ->remove hook Daniel Vetter
2013-11-03 13:31 ` [PATCH 07/19] drm/host1x: Call drm_put_dev directly instead of drm_platform_exit Daniel Vetter
2013-11-04 9:10 ` Thierry Reding
2013-11-04 9:13 ` Thierry Reding
2013-11-04 10:14 ` Daniel Vetter
2013-11-04 10:51 ` Thierry Reding [this message]
2013-11-03 13:31 ` [PATCH 08/19] drm/armada: directly call drm_put_dev in ->remove Daniel Vetter
2013-11-03 13:31 ` [PATCH 09/19] drm/msm: call drm_put_dev directly " Daniel Vetter
2013-11-03 13:31 ` [PATCH 10/19] drm: rip out drm_platform_exit Daniel Vetter
2013-11-03 13:31 ` [PATCH 11/19] drm: restrict the device list for shadow attached drivers Daniel Vetter
2013-11-03 14:05 ` David Herrmann
2013-11-03 14:14 ` Daniel Vetter
2013-11-03 18:51 ` [PATCH] " Daniel Vetter
2013-11-04 9:20 ` [PATCH 11/19] " Thierry Reding
2013-11-03 13:31 ` [PATCH 12/19] drm/bufs: remove handling of _DRM_GEM mappings Daniel Vetter
2013-11-03 14:06 ` David Herrmann
2013-11-03 13:31 ` [PATCH 13/19] drm: kill DRIVER_REQUIRE_AGP Daniel Vetter
2013-11-03 13:31 ` [PATCH 14/19] drm: ->agp_init can't fail Daniel Vetter
2013-11-03 14:12 ` David Herrmann
2013-11-03 13:31 ` [PATCH 15/19] drm: rip out drm_core_has_AGP Daniel Vetter
2013-11-03 13:31 ` [PATCH 16/19] drm: remove agp_init() bus callback Daniel Vetter
2013-11-03 14:14 ` David Herrmann
2013-11-03 14:20 ` Daniel Vetter
2013-11-03 14:42 ` [PATCH] " Daniel Vetter
2013-11-03 13:31 ` [PATCH 17/19] drm: inline drm_agp_destroy Daniel Vetter
2013-11-03 13:43 ` David Herrmann
2013-11-03 14:36 ` Daniel Vetter
2013-11-03 13:31 ` [PATCH 18/19] drm: kill the ->agp_destroy callback Daniel Vetter
2013-11-03 13:49 ` David Herrmann
2013-11-03 14:24 ` [PATCH] " Daniel Vetter
2013-11-03 14:34 ` David Herrmann
2013-11-03 17:32 ` Daniel Vetter
2013-11-03 13:31 ` [PATCH 19/19] drm: remove global_mutex locking around agp_init Daniel Vetter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131104105158.GM27445@ulmo.nvidia.com \
--to=thierry.reding@gmail.com \
--cc=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.