All of lore.kernel.org
 help / color / mirror / Atom feed
* [PULL] drm/tegra: Changes for v3.13-rc1
@ 2013-10-31  9:17 Thierry Reding
       [not found] ` <20131031091727.GA16198-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: Thierry Reding @ 2013-10-31  9:17 UTC (permalink / raw)
  To: Dave Airlie; +Cc: linux-tegra, dri-devel


[-- Attachment #1.1: Type: text/plain, Size: 11020 bytes --]

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

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PULL] drm/tegra: Changes for v3.13-rc1
       [not found] ` <20131031091727.GA16198-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
@ 2013-11-04 10:22   ` Daniel Vetter
       [not found]     ` <20131104102253.GE4167-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: Daniel Vetter @ 2013-11-04 10:22 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Dave Airlie, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

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-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel


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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PULL] drm/tegra: Changes for v3.13-rc1
       [not found]     ` <20131104102253.GE4167-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
@ 2013-11-04 12:11       ` Thierry Reding
       [not found]         ` <20131104121145.GB18722-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: Thierry Reding @ 2013-11-04 12:11 UTC (permalink / raw)
  To: Daniel Vetter
  Cc: Dave Airlie, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

[-- Attachment #1: Type: text/plain, Size: 1812 bytes --]

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 #2: Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PULL] drm/tegra: Changes for v3.13-rc1
       [not found]         ` <20131104121145.GB18722-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
@ 2013-11-04 16:21           ` Daniel Vetter
  0 siblings, 0 replies; 4+ messages in thread
From: Daniel Vetter @ 2013-11-04 16:21 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Daniel Vetter, Dave Airlie, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

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
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-11-04 16:21 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-10-31  9:17 [PULL] drm/tegra: Changes for v3.13-rc1 Thierry Reding
     [not found] ` <20131031091727.GA16198-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2013-11-04 10:22   ` Daniel Vetter
     [not found]     ` <20131104102253.GE4167-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2013-11-04 12:11       ` Thierry Reding
     [not found]         ` <20131104121145.GB18722-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2013-11-04 16:21           ` Daniel Vetter

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.