Hi Dave,
The following changes since commit 14c8d110e083d3a09ccf8cfe18ad22fe1450c2e9:
drm/i915: abstract the conversion of device->minor out to a macro (2013-10-15 18:06:06 +1000)
are available in the git repository at:
git://anongit.freedesktop.org/tegra/linux tags/drm/for-3.13-rc1
for you to fetch changes up to 977386a04bae2a5a5092c965c92c7c4d36eed23f:
drm/tegra: Reserve syncpoint base for gr3d (2013-10-31 09:55:49 +0100)
I decided to leave out the more controversial parts, such as DSI and panel support because of all the recent discussions and new rules for device tree bindings. My hope would be that we can get those issues worked out during the 3.13 release cycle so that we can finally have panel support for 3.14.
Thanks, Thierry
---------------------------------------------------------------- drm/tegra: Changes for v3.13-rc1
The biggest part of the changes is the decoupling of the host1x and DRM drivers followed by the move of Tegra DRM back to drivers/gpu/drm/tegra from whence it came. There is a lot of cleanup as well, and the drivers can now be properly unloaded and reloaded.
HDMI support for the Tegra114 SoC was contributed by Mikko Perttunen.
gr2d support was extended to Tegra114 and the gr3d driver that has been in the works for quite some time finally made it in. All pieces to run an OpenGL driver on top of an upstream kernel are now available.
Support for syncpoint bases was added by Arto Merilainen. This is useful for synchronizing between command streams from different engines such as gr2d and gr3d.
Erik Faye-Lund and Wei Yongjun contributed various small fixes. Thanks!
---------------------------------------------------------------- Arto Merilainen (4): gpu: host1x: Add 'flags' field to syncpt request gpu: host1x: Add syncpoint base support drm/tegra: Deliver syncpoint base to user space drm/tegra: Reserve base for gr2d
Erik Faye-Lund (1): gpu: host1x: check relocs after all gathers are consumed
Mikko Perttunen (3): drm/tegra: Add Tegra114 HDMI support drm/tegra: hdmi: Detect DVI-only displays drm/tegra: hdmi: Enable VDD earlier for hotplug/DDC
Thierry Reding (35): drm: Track the proper DPMS mode of connectors drm: Fix typo in debug message gpu: host1x: Remove unused Makefile drm/tegra: Remove unused fields drm/tegra: Cleanup tegra_dc structure drm/tegra: Rename host1x_drm structure to tegra_drm drm/tegra: Rename host1x_drm_file to tegra_drm_file drm/tegra: Rename host1x_drm_context to tegra_drm_context gpu: host1x: Cleanup includes gpu: host1x: Do not discard .remove() gpu: host1x: Fix alignment of function arguments gpu: host1x: firewall: Rename cmdbuf_id -> cmdbuf gpu: host1x: firewall: Refactor register check drm/tegra: gr2d: Miscellaneous cleanups drm/tegra: Rename gr2d to tegra-gr2d drm/tegra: gem: Miscellaneous cleanups gpu: host1x: Make host1x header file public drm/tegra: Introduce tegra_drm_client structure gpu: host1x: Expose syncpt and channel functionality drm/tegra: Move subdevice infrastructure to host1x gpu: host1x: Use relative include paths drm/tegra: Move driver to DRM tree drm/tegra: Properly cleanup and zero out resources drm/tegra: Allocate resources at probe time gpu: host1x: Add support for Tegra114 drm/tegra: hdmi: Rename tegra{2,3} to tegra{20,30} drm/tegra: hdmi: Parameterize based on compatible property drm/tegra: hdmi: Fix build warnings drm/tegra: Start connectors with correct DPMS mode drm/tegra: Use symbolic names for gr2d registers drm/tegra: Introduce tegra_drm_submit() drm/tegra: Add 3D support drm/tegra: Add support for tiled buffer objects drm/tegra: Support bottom-up buffer objects drm/tegra: Reserve syncpoint base for gr3d
Wei Yongjun (2): gpu: host1x: Disable clock on probe failure drm/tegra: Disable clock on probe failure
MAINTAINERS | 2 + drivers/gpu/drm/Kconfig | 2 + drivers/gpu/drm/Makefile | 1 + drivers/gpu/drm/drm_crtc_helper.c | 8 ++++ drivers/gpu/drm/drm_drv.c | 2 +- drivers/gpu/{host1x/drm => drm/tegra}/Kconfig | 12 +++--- drivers/gpu/drm/tegra/Makefile | 15 +++++++ drivers/gpu/drm/tegra/bus.c | 76 ++++++++++++++++++++++++++++++++++ drivers/gpu/{host1x/drm => drm/tegra}/dc.c | 108 ++++++++++++++++++++++++++++++++++++++---------- drivers/gpu/{host1x/drm => drm/tegra}/dc.h | 5 +++ drivers/gpu/{host1x/drm => drm/tegra}/drm.c | 593 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------------------------------------------------------------------------- drivers/gpu/{host1x/drm => drm/tegra}/drm.h | 101 +++++++++++++++++++++++---------------------- drivers/gpu/{host1x/drm => drm/tegra}/fb.c | 38 ++++++++++++----- drivers/gpu/{host1x/drm => drm/tegra}/gem.c | 44 ++++++++++---------- drivers/gpu/{host1x/drm => drm/tegra}/gem.h | 16 +++++--- drivers/gpu/drm/tegra/gr2d.c | 227 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/tegra/gr2d.h | 28 +++++++++++++ drivers/gpu/drm/tegra/gr3d.c | 338 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/tegra/gr3d.h | 27 ++++++++++++ drivers/gpu/{host1x/drm => drm/tegra}/hdmi.c | 257 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------- drivers/gpu/{host1x/drm => drm/tegra}/hdmi.h | 152 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++- drivers/gpu/{host1x/drm => drm/tegra}/output.c | 64 +++++++++++++++++------------ drivers/gpu/{host1x/drm => drm/tegra}/rgb.c | 19 +++++++-- drivers/gpu/host1x/Kconfig | 2 - drivers/gpu/host1x/Makefile | 13 ++---- drivers/gpu/host1x/bus.c | 550 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/{host1x_client.h => bus.h} | 24 ++++------- drivers/gpu/host1x/cdma.c | 2 +- drivers/gpu/host1x/channel.h | 6 --- drivers/gpu/host1x/dev.c | 82 ++++++++++++++++--------------------- drivers/gpu/host1x/dev.h | 11 ++--- drivers/gpu/host1x/drm/gr2d.c | 343 --------------------------------------------------------------------------------------------------------------------------------------------------------- drivers/gpu/host1x/host1x.h | 30 -------------- drivers/gpu/host1x/host1x_bo.h | 87 --------------------------------------- drivers/gpu/host1x/hw/Makefile | 6 --- drivers/gpu/host1x/hw/cdma_hw.c | 8 ++-- drivers/gpu/host1x/hw/channel_hw.c | 32 ++++++++++++--- drivers/gpu/host1x/hw/debug_hw.c | 16 ++------ drivers/gpu/host1x/hw/host1x01.c | 16 ++++---- drivers/gpu/host1x/hw/host1x02.c | 42 +++++++++++++++++++ drivers/gpu/host1x/hw/host1x02.h | 26 ++++++++++++ drivers/gpu/host1x/hw/hw_host1x01_uclass.h | 6 +++ drivers/gpu/host1x/hw/hw_host1x02_channel.h | 121 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/hw/hw_host1x02_sync.h | 243 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/hw/hw_host1x02_uclass.h | 175 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/hw/intr_hw.c | 4 +- drivers/gpu/host1x/hw/syncpt_hw.c | 4 +- drivers/gpu/host1x/job.c | 73 +++++++++++++++++---------------- drivers/gpu/host1x/job.h | 108 ------------------------------------------------ drivers/gpu/host1x/syncpt.c | 92 +++++++++++++++++++++++++++++++++++++---- drivers/gpu/host1x/syncpt.h | 46 ++++----------------- drivers/video/Kconfig | 4 +- include/drm/drmP.h | 1 + include/linux/host1x.h | 284 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ include/uapi/drm/tegra_drm.h | 29 +++++++++---- 55 files changed, 3362 insertions(+), 1259 deletions(-) rename drivers/gpu/{host1x/drm => drm/tegra}/Kconfig (90%) create mode 100644 drivers/gpu/drm/tegra/Makefile create mode 100644 drivers/gpu/drm/tegra/bus.c rename drivers/gpu/{host1x/drm => drm/tegra}/dc.c (93%) rename drivers/gpu/{host1x/drm => drm/tegra}/dc.h (98%) rename drivers/gpu/{host1x/drm => drm/tegra}/drm.c (50%) rename drivers/gpu/{host1x/drm => drm/tegra}/drm.h (72%) rename drivers/gpu/{host1x/drm => drm/tegra}/fb.c (92%) rename drivers/gpu/{host1x/drm => drm/tegra}/gem.c (86%) rename drivers/gpu/{host1x/drm => drm/tegra}/gem.h (84%) create mode 100644 drivers/gpu/drm/tegra/gr2d.c create mode 100644 drivers/gpu/drm/tegra/gr2d.h create mode 100644 drivers/gpu/drm/tegra/gr3d.c create mode 100644 drivers/gpu/drm/tegra/gr3d.h rename drivers/gpu/{host1x/drm => drm/tegra}/hdmi.c (83%) rename drivers/gpu/{host1x/drm => drm/tegra}/hdmi.h (72%) rename drivers/gpu/{host1x/drm => drm/tegra}/output.c (91%) rename drivers/gpu/{host1x/drm => drm/tegra}/rgb.c (96%) create mode 100644 drivers/gpu/host1x/bus.c rename drivers/gpu/host1x/{host1x_client.h => bus.h} (60%) delete mode 100644 drivers/gpu/host1x/drm/gr2d.c delete mode 100644 drivers/gpu/host1x/host1x.h delete mode 100644 drivers/gpu/host1x/host1x_bo.h delete mode 100644 drivers/gpu/host1x/hw/Makefile create mode 100644 drivers/gpu/host1x/hw/host1x02.c create mode 100644 drivers/gpu/host1x/hw/host1x02.h create mode 100644 drivers/gpu/host1x/hw/hw_host1x02_channel.h create mode 100644 drivers/gpu/host1x/hw/hw_host1x02_sync.h create mode 100644 drivers/gpu/host1x/hw/hw_host1x02_uclass.h create mode 100644 include/linux/host1x.h
On Thu, Oct 31, 2013 at 10:17:28AM +0100, Thierry Reding wrote:
Hi Dave,
The following changes since commit 14c8d110e083d3a09ccf8cfe18ad22fe1450c2e9:
drm/i915: abstract the conversion of device->minor out to a macro (2013-10-15 18:06:06 +1000)
are available in the git repository at:
git://anongit.freedesktop.org/tegra/linux tags/drm/for-3.13-rc1
for you to fetch changes up to 977386a04bae2a5a5092c965c92c7c4d36eed23f:
drm/tegra: Reserve syncpoint base for gr3d (2013-10-31 09:55:49 +0100)
I decided to leave out the more controversial parts, such as DSI and panel support because of all the recent discussions and new rules for device tree bindings. My hope would be that we can get those issues worked out during the 3.13 release cycle so that we can finally have panel support for 3.14.
Thanks, Thierry
drm/tegra: Changes for v3.13-rc1
The biggest part of the changes is the decoupling of the host1x and DRM drivers followed by the move of Tegra DRM back to drivers/gpu/drm/tegra from whence it came. There is a lot of cleanup as well, and the drivers can now be properly unloaded and reloaded.
HDMI support for the Tegra114 SoC was contributed by Mikko Perttunen.
gr2d support was extended to Tegra114 and the gr3d driver that has been in the works for quite some time finally made it in. All pieces to run an OpenGL driver on top of an upstream kernel are now available.
Support for syncpoint bases was added by Arto Merilainen. This is useful for synchronizing between command streams from different engines such as gr2d and gr3d.
Erik Faye-Lund and Wei Yongjun contributed various small fixes. Thanks!
Arto Merilainen (4): gpu: host1x: Add 'flags' field to syncpt request gpu: host1x: Add syncpoint base support drm/tegra: Deliver syncpoint base to user space drm/tegra: Reserve base for gr2d
Erik Faye-Lund (1): gpu: host1x: check relocs after all gathers are consumed
Mikko Perttunen (3): drm/tegra: Add Tegra114 HDMI support drm/tegra: hdmi: Detect DVI-only displays drm/tegra: hdmi: Enable VDD earlier for hotplug/DDC
Thierry Reding (35): drm: Track the proper DPMS mode of connectors drm: Fix typo in debug message gpu: host1x: Remove unused Makefile drm/tegra: Remove unused fields drm/tegra: Cleanup tegra_dc structure drm/tegra: Rename host1x_drm structure to tegra_drm drm/tegra: Rename host1x_drm_file to tegra_drm_file drm/tegra: Rename host1x_drm_context to tegra_drm_context gpu: host1x: Cleanup includes gpu: host1x: Do not discard .remove() gpu: host1x: Fix alignment of function arguments gpu: host1x: firewall: Rename cmdbuf_id -> cmdbuf gpu: host1x: firewall: Refactor register check drm/tegra: gr2d: Miscellaneous cleanups drm/tegra: Rename gr2d to tegra-gr2d drm/tegra: gem: Miscellaneous cleanups gpu: host1x: Make host1x header file public drm/tegra: Introduce tegra_drm_client structure gpu: host1x: Expose syncpt and channel functionality 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 ... -Daniel
gpu: host1x: Use relative include paths drm/tegra: Move driver to DRM tree drm/tegra: Properly cleanup and zero out resources drm/tegra: Allocate resources at probe time gpu: host1x: Add support for Tegra114 drm/tegra: hdmi: Rename tegra{2,3} to tegra{20,30} drm/tegra: hdmi: Parameterize based on compatible property drm/tegra: hdmi: Fix build warnings drm/tegra: Start connectors with correct DPMS mode drm/tegra: Use symbolic names for gr2d registers drm/tegra: Introduce tegra_drm_submit() drm/tegra: Add 3D support drm/tegra: Add support for tiled buffer objects drm/tegra: Support bottom-up buffer objects drm/tegra: Reserve syncpoint base for gr3d
Wei Yongjun (2): gpu: host1x: Disable clock on probe failure drm/tegra: Disable clock on probe failure
MAINTAINERS | 2 + drivers/gpu/drm/Kconfig | 2 + drivers/gpu/drm/Makefile | 1 + drivers/gpu/drm/drm_crtc_helper.c | 8 ++++ drivers/gpu/drm/drm_drv.c | 2 +- drivers/gpu/{host1x/drm => drm/tegra}/Kconfig | 12 +++--- drivers/gpu/drm/tegra/Makefile | 15 +++++++ drivers/gpu/drm/tegra/bus.c | 76 ++++++++++++++++++++++++++++++++++ drivers/gpu/{host1x/drm => drm/tegra}/dc.c | 108 ++++++++++++++++++++++++++++++++++++++---------- drivers/gpu/{host1x/drm => drm/tegra}/dc.h | 5 +++ drivers/gpu/{host1x/drm => drm/tegra}/drm.c | 593 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------------------------------------------------------------------------- drivers/gpu/{host1x/drm => drm/tegra}/drm.h | 101 +++++++++++++++++++++++---------------------- drivers/gpu/{host1x/drm => drm/tegra}/fb.c | 38 ++++++++++++----- drivers/gpu/{host1x/drm => drm/tegra}/gem.c | 44 ++++++++++---------- drivers/gpu/{host1x/drm => drm/tegra}/gem.h | 16 +++++--- drivers/gpu/drm/tegra/gr2d.c | 227 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/tegra/gr2d.h | 28 +++++++++++++ drivers/gpu/drm/tegra/gr3d.c | 338 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/tegra/gr3d.h | 27 ++++++++++++ drivers/gpu/{host1x/drm => drm/tegra}/hdmi.c | 257 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------- drivers/gpu/{host1x/drm => drm/tegra}/hdmi.h | 152 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++- drivers/gpu/{host1x/drm => drm/tegra}/output.c | 64 +++++++++++++++++------------ drivers/gpu/{host1x/drm => drm/tegra}/rgb.c | 19 +++++++-- drivers/gpu/host1x/Kconfig | 2 - drivers/gpu/host1x/Makefile | 13 ++---- drivers/gpu/host1x/bus.c | 550 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/{host1x_client.h => bus.h} | 24 ++++------- drivers/gpu/host1x/cdma.c | 2 +- drivers/gpu/host1x/channel.h | 6 --- drivers/gpu/host1x/dev.c | 82 ++++++++++++++++--------------------- drivers/gpu/host1x/dev.h | 11 ++--- drivers/gpu/host1x/drm/gr2d.c | 343 --------------------------------------------------------------------------------------------------------------------------------------------------------- drivers/gpu/host1x/host1x.h | 30 -------------- drivers/gpu/host1x/host1x_bo.h | 87 --------------------------------------- drivers/gpu/host1x/hw/Makefile | 6 --- drivers/gpu/host1x/hw/cdma_hw.c | 8 ++-- drivers/gpu/host1x/hw/channel_hw.c | 32 ++++++++++++--- drivers/gpu/host1x/hw/debug_hw.c | 16 ++------ drivers/gpu/host1x/hw/host1x01.c | 16 ++++---- drivers/gpu/host1x/hw/host1x02.c | 42 +++++++++++++++++++ drivers/gpu/host1x/hw/host1x02.h | 26 ++++++++++++ drivers/gpu/host1x/hw/hw_host1x01_uclass.h | 6 +++ drivers/gpu/host1x/hw/hw_host1x02_channel.h | 121 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/hw/hw_host1x02_sync.h | 243 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/hw/hw_host1x02_uclass.h | 175 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/host1x/hw/intr_hw.c | 4 +- drivers/gpu/host1x/hw/syncpt_hw.c | 4 +- drivers/gpu/host1x/job.c | 73 +++++++++++++++++---------------- drivers/gpu/host1x/job.h | 108 ------------------------------------------------ drivers/gpu/host1x/syncpt.c | 92 +++++++++++++++++++++++++++++++++++++---- drivers/gpu/host1x/syncpt.h | 46 ++++----------------- drivers/video/Kconfig | 4 +- include/drm/drmP.h | 1 + include/linux/host1x.h | 284 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ include/uapi/drm/tegra_drm.h | 29 +++++++++---- 55 files changed, 3362 insertions(+), 1259 deletions(-) rename drivers/gpu/{host1x/drm => drm/tegra}/Kconfig (90%) create mode 100644 drivers/gpu/drm/tegra/Makefile create mode 100644 drivers/gpu/drm/tegra/bus.c rename drivers/gpu/{host1x/drm => drm/tegra}/dc.c (93%) rename drivers/gpu/{host1x/drm => drm/tegra}/dc.h (98%) rename drivers/gpu/{host1x/drm => drm/tegra}/drm.c (50%) rename drivers/gpu/{host1x/drm => drm/tegra}/drm.h (72%) rename drivers/gpu/{host1x/drm => drm/tegra}/fb.c (92%) rename drivers/gpu/{host1x/drm => drm/tegra}/gem.c (86%) rename drivers/gpu/{host1x/drm => drm/tegra}/gem.h (84%) create mode 100644 drivers/gpu/drm/tegra/gr2d.c create mode 100644 drivers/gpu/drm/tegra/gr2d.h create mode 100644 drivers/gpu/drm/tegra/gr3d.c create mode 100644 drivers/gpu/drm/tegra/gr3d.h rename drivers/gpu/{host1x/drm => drm/tegra}/hdmi.c (83%) rename drivers/gpu/{host1x/drm => drm/tegra}/hdmi.h (72%) rename drivers/gpu/{host1x/drm => drm/tegra}/output.c (91%) rename drivers/gpu/{host1x/drm => drm/tegra}/rgb.c (96%) create mode 100644 drivers/gpu/host1x/bus.c rename drivers/gpu/host1x/{host1x_client.h => bus.h} (60%) delete mode 100644 drivers/gpu/host1x/drm/gr2d.c delete mode 100644 drivers/gpu/host1x/host1x.h delete mode 100644 drivers/gpu/host1x/host1x_bo.h delete mode 100644 drivers/gpu/host1x/hw/Makefile create mode 100644 drivers/gpu/host1x/hw/host1x02.c create mode 100644 drivers/gpu/host1x/hw/host1x02.h create mode 100644 drivers/gpu/host1x/hw/hw_host1x02_channel.h create mode 100644 drivers/gpu/host1x/hw/hw_host1x02_sync.h create mode 100644 drivers/gpu/host1x/hw/hw_host1x02_uclass.h create mode 100644 include/linux/host1x.h
dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
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
On Mon, Nov 04, 2013 at 01:11:46PM +0100, Thierry Reding wrote:
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?
drm_bus goes back to the shadow attach horror stories of legacy drm drivers with ums. It was done for platform drivers due to some out-of-tree ARM drivers that luckily have never been merged. It's just part of the lore ;-)
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.
It'll die soon, together with drm_bus. I've just grown a bit frustrated thinking that I'm too late at preventing old cruft from spreading ;-)
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.
After a bit of cooling of I've looked at the situation. See my reply to the patch itself, it should be easily fixed by just throwing a pretty tiny fixup patch on top. Then the host1x drm_bus is just a copy of the drm_bus, and so really easy to deal with.
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?
For better coordination I wonder whether we should have a drm-integration tree with all the driver wip stuff (lesser requirements than linux-next, so not just stuff ready for the next merge window) plus big cleanup branches like this and big features like the atomic modeset stuff rob is working on. But atm I'm still rather reluctant to try it out for fear of getting stuck with it ;-)
Cheers, Daniel
dri-devel@lists.freedesktop.org