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
