Merge tag 'drm-misc-next-2017-11-30' of git://anongit.freedesktop.org/drm/drm-misc into drm-next

Cross-subsystem Changes:

- device tree doc for the Mitsubishi AA070MC01 and Tianma TM070RVHG71
panels (Lukasz Majewski) and for a 2nd endpoint on stm32 (Philippe Cornu)

Core Changes:

The most important changes are:

- Add drm_driver .last_close and .output_poll_changed helpers to reduce
fbdev emulation footprint in drivers (Noralf)
- Fix plane clipping in core and for vmwgfx (Ville)

Then we have a bunch of of improvement for print and debug such as the
addition of a framebuffer debugfs file. ELD connector, HDMI and
improvements.  And a bunch of misc improvements, clean ups and style
changes and doc updates

[airlied: drop eld bits from amdgpu_dm]

Driver Changes:

- sii8620: filter unsupported modes and add DVI mode support (Maciej Purski)
- rockchip: analogix_dp: Remove unnecessary init code (Jeffy Chen)
- virtio, cirrus: add fb create_handle support to enable screenshots(Lepton Wu)
- virtio: replace reference/unreference with get/put (Aastha Gupta)
- vc4, gma500: Convert timers to use timer_setup() (Kees Cook)
- vc4: Reject HDMI modes with too high of clocks (Eric)
- vc4: Add support for more pixel formats (Dave Stevenson)
- stm: dsi: Rename driver name to "stm32-display-dsi" (Philippe Cornu)
- stm: ltdc: add a 2nd endpoint (Philippe Cornu)
- via: use monotonic time for VIA_WAIT_IRQ (Arnd Bergmann)

* tag 'drm-misc-next-2017-11-30' of git://anongit.freedesktop.org/drm/drm-misc: (96 commits)
  drm/bridge: tc358767: add copyright lines
  MAINTAINERS: change maintainer for Rockchip drm drivers
  drm/vblank: Fix vblank timestamp debugs
  drm/via: use monotonic time for VIA_WAIT_IRQ
  dma-buf: Fix ifnullfree.cocci warnings
  drm/printer: Add drm_vprintf()
  drm/edid: Allow HDMI infoframe without VIC or S3D
  video/hdmi: Allow "empty" HDMI infoframes
  dma-buf/fence: Fix lock inversion within dma-fence-array
  drm/sti: Handle return value of platform_get_irq_byname
  drm/vc4: Add support for NV21 and NV61.
  drm/vc4: Use .pixel_order instead of custom .flip_cbcr
  drm/vc4: Add support for DRM_FORMAT_RGB888 and DRM_FORMAT_BGR888
  drm: Move drm_plane_helper_check_state() into drm_atomic_helper.c
  drm: Check crtc_state->enable rather than crtc->enabled in drm_plane_helper_check_state()
  drm/vmwgfx: Try to fix plane clipping
  drm/vmwgfx: Use drm_plane_helper_check_state()
  drm/vmwgfx: Remove bogus crtc coords vs fb size check
  gpu: gma500: remove unneeded DRIVER_LICENSE #define
  drm: don't link DP aux i2c adapter to the hardware device node
  ...

1 files changed, 79 insertions, 8 deletions

diff --git a/drivers/gpu/drm/drm_framebuffer.c b/drivers/gpu/drm/drm_framebuffer.c
index 279c1035c12d..d63d4c2ac4c8 100644
--- a/drivers/gpu/drm/drm_framebuffer.c
+++ b/drivers/gpu/drm/drm_framebuffer.c
@@ -25,7 +25,9 @@
 #include <drm/drm_auth.h>
 #include <drm/drm_framebuffer.h>
 #include <drm/drm_atomic.h>
+#include <drm/drm_print.h>
 
+#include "drm_internal.h"
 #include "drm_crtc_internal.h"
 
 /**
@@ -78,11 +80,12 @@ int drm_framebuffer_check_src_coords(uint32_t src_x, uint32_t src_y,
             src_h > fb_height ||
             src_y > fb_height - src_h) {
                 DRM_DEBUG_KMS("Invalid source coordinates "
-                              "%u.%06ux%u.%06u+%u.%06u+%u.%06u\n",
+                              "%u.%06ux%u.%06u+%u.%06u+%u.%06u (fb %ux%u)\n",
                               src_w >> 16, ((src_w & 0xffff) * 15625) >> 10,
                               src_h >> 16, ((src_h & 0xffff) * 15625) >> 10,
                               src_x >> 16, ((src_x & 0xffff) * 15625) >> 10,
-                              src_y >> 16, ((src_y & 0xffff) * 15625) >> 10);
+                              src_y >> 16, ((src_y & 0xffff) * 15625) >> 10,
+                              fb->width, fb->height);
                 return -ENOSPC;
         }
 
@@ -766,14 +769,18 @@ static int atomic_remove_fb(struct drm_framebuffer *fb)
         struct drm_plane *plane;
         struct drm_connector *conn;
         struct drm_connector_state *conn_state;
-        int i, ret = 0;
+        int i, ret;
         unsigned plane_mask;
+        bool disable_crtcs = false;
 
-        state = drm_atomic_state_alloc(dev);
-        if (!state)
-                return -ENOMEM;
-
+retry_disable:
         drm_modeset_acquire_init(&ctx, 0);
+
+        state = drm_atomic_state_alloc(dev);
+        if (!state) {
+                ret = -ENOMEM;
+                goto out;
+        }
         state->acquire_ctx = &ctx;
 
 retry:
@@ -794,7 +801,7 @@ retry:
                         goto unlock;
                 }
 
-                if (plane_state->crtc->primary == plane) {
+                if (disable_crtcs && plane_state->crtc->primary == plane) {
                         struct drm_crtc_state *crtc_state;
 
                         crtc_state = drm_atomic_get_existing_crtc_state(state, plane_state->crtc);
@@ -819,6 +826,7 @@ retry:
                 plane->old_fb = plane->fb;
         }
 
+        /* This list is only filled when disable_crtcs is set. */
         for_each_new_connector_in_state(state, conn, conn_state, i) {
                 ret = drm_atomic_set_crtc_for_connector(conn_state, NULL);
 
@@ -841,9 +849,15 @@ unlock:
 
         drm_atomic_state_put(state);
 
+out:
         drm_modeset_drop_locks(&ctx);
         drm_modeset_acquire_fini(&ctx);
 
+        if (ret == -EINVAL && !disable_crtcs) {
+                disable_crtcs = true;
+                goto retry_disable;
+        }
+
         return ret;
 }
 
@@ -957,3 +971,60 @@ int drm_framebuffer_plane_height(int height,
         return fb_plane_height(height, fb->format, plane);
 }
 EXPORT_SYMBOL(drm_framebuffer_plane_height);
+
+void drm_framebuffer_print_info(struct drm_printer *p, unsigned int indent,
+                                const struct drm_framebuffer *fb)
+{
+        struct drm_format_name_buf format_name;
+        unsigned int i;
+
+        drm_printf_indent(p, indent, "refcount=%u\n",
+                          drm_framebuffer_read_refcount(fb));
+        drm_printf_indent(p, indent, "format=%s\n",
+                          drm_get_format_name(fb->format->format, &format_name));
+        drm_printf_indent(p, indent, "modifier=0x%llx\n", fb->modifier);
+        drm_printf_indent(p, indent, "size=%ux%u\n", fb->width, fb->height);
+        drm_printf_indent(p, indent, "layers:\n");
+
+        for (i = 0; i < fb->format->num_planes; i++) {
+                drm_printf_indent(p, indent + 1, "size[%u]=%dx%d\n", i,
+                                  drm_framebuffer_plane_width(fb->width, fb, i),
+                                  drm_framebuffer_plane_height(fb->height, fb, i));
+                drm_printf_indent(p, indent + 1, "pitch[%u]=%u\n", i, fb->pitches[i]);
+                drm_printf_indent(p, indent + 1, "offset[%u]=%u\n", i, fb->offsets[i]);
+                drm_printf_indent(p, indent + 1, "obj[%u]:%s\n", i,
+                                  fb->obj[i] ? "" : "(null)");
+                if (fb->obj[i])
+                        drm_gem_print_info(p, indent + 2, fb->obj[i]);
+        }
+}
+
+#ifdef CONFIG_DEBUG_FS
+static int drm_framebuffer_info(struct seq_file *m, void *data)
+{
+        struct drm_info_node *node = m->private;
+        struct drm_device *dev = node->minor->dev;
+        struct drm_printer p = drm_seq_file_printer(m);
+        struct drm_framebuffer *fb;
+
+        mutex_lock(&dev->mode_config.fb_lock);
+        drm_for_each_fb(fb, dev) {
+                drm_printf(&p, "framebuffer[%u]:\n", fb->base.id);
+                drm_framebuffer_print_info(&p, 1, fb);
+        }
+        mutex_unlock(&dev->mode_config.fb_lock);
+
+        return 0;
+}
+
+static const struct drm_info_list drm_framebuffer_debugfs_list[] = {
+        { "framebuffer", drm_framebuffer_info, 0 },
+};
+
+int drm_framebuffer_debugfs_init(struct drm_minor *minor)
+{
+        return drm_debugfs_create_files(drm_framebuffer_debugfs_list,
+                                ARRAY_SIZE(drm_framebuffer_debugfs_list),
+                                minor->debugfs_root, minor);
+}
+#endif