Skip to contentSkip to conversation

drm/tegra: Changes for v3.13-rc1

GIT PULL Thierry Reding 3 participants 4 messages Oct 31, 2013 – Nov 4, 2013 3 messages not in the archive Tegra architecture support

lore
Share & tools
lore
#0 lore raw

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

_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel

A message that is not in the archive (sent off-list or not yet received): 20131031091727.GA16198-AwZRO8vwLAwmlAP/[email protected]

#2 lore raw
Quoted 71 lines · Thierry Reding

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

Quoted 8 lines

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

Quoted 113 lines

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 [email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Signature · 3 lines
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

A message that is not in the archive (sent off-list or not yet received): 20131104102253.GE4167-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org

#4 lore raw
Quoted 1 line · Daniel Vetter

On Thu, Oct 31, 2013 at 10:17:28AM +0100, Thierry Reding wrote:

[...]

Quoted 5 lines
Quoted 1 line

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

A message that is not in the archive (sent off-list or not yet received): 20131104121145.GB18722-AwZRO8vwLAwmlAP/[email protected]

#6 lore raw
Quoted 9 lines · Thierry Reding
Quoted 1 line · Daniel Vetter

On Thu, Oct 31, 2013 at 10:17:28AM +0100, Thierry Reding wrote:

[...]

Quoted 5 lines
Quoted 1 line

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 ...

Quoted 7 lines

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 ;-)

Quoted 9 lines

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 ;-)

Quoted 4 lines

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.

Quoted 4 lines

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

Signature · 3 lines
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch