Re: [PULL] drm/tegra: Changes for v3.13-rc1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 04, 2013 at 11:22:53AM +0100, Daniel Vetter wrote:
> On Thu, Oct 31, 2013 at 10:17:28AM +0100, Thierry Reding wrote:
[...]
> >       drm/tegra: Move subdevice infrastructure to host1x
> 
> I've just shot at this patch on the m-l, but I'd be rather unhappy if the
> new drm_bus madness this add gets into drm-next. Would be a definite step
> backwards imo for the drm core. Also more work for me to fix it all up ...

In all fairness, the patches were posted for review a while back (4
weeks ago) and I had no idea any such rework was in place, much less
that anybody else considered drm_bus to be a bad idea.

Perhaps we could maintain a list of TODO items somewhere with notes as
to what's currently being worked on. Maybe such a list already exists
and I'm just not aware of it?

It'd be unfortunate if this series can't be merged for 3.13. The patch
you object to is early in the list and everything after it depends on
it, so if that doesn't make it in, then none of the rest will make it
either.

I've also explained elsewhere that the only thing drm_bus related that
this adds is a new define for DRIVER_BUS_HOST1X and an implementation of
.set_busid. The former should be trivial to remove, while the latter is
the only one that you've kept in the cleanup tree you've posted.

Also I'd like to reassert my offer to help. While working on this I've
actually came across various oddities myself, like how the bus type was
completely unused, and had added them to my TODO list of things to look
into later.

I really appreciate the work you do, but I think we could use some more
coordination to avoid conflicts such as these and perhaps share the load
of cleanup work. Is there anything in particular that I could do to help
improve the situation?

Thierry

Attachment: pgpMqpJMJRURw.pgp
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux