Merge tag 'drm-misc-next-2019-01-16' of git://anongit.freedesktop.org/drm/drm-misc into drm-next

drm-misc-next for 5.1:

UAPI Changes:
 - New fourcc identifier for ARM Framebuffer Compression v1.3

Cross-subsystem Changes:

Core Changes:
 - Reorganisation of drm_device and drm_framebuffer headers
 - Cleanup of the drmP inclusion
 - Fix leaks in the fb-helpers
 - Allow for depth different from bpp in fb-helper fbdev emulation
 - Remove drm_mode_object from drm_display_mode

Driver Changes:
 - Add reflection properties to rockchip
 - a bunch of fixes for virtio
 - a bunch of fixes for dp_mst and drivers using it, and introduction of a
   new refcounting scheme
 - Convertion of bochs to atomic and generic fbdev emulation
 - Allow meson to remove the firmware framebuffers

[airlied: patch rcar-du to add drm_modes.h]
Signed-off-by: Dave Airlie <airlied@redhat.com>

From: Maxime Ripard <maxime.ripard@bootlin.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190116200428.u2n4jbk4mzza7n6e@flea

10 files changed, 105 insertions, 145 deletions

diff --git a/drivers/gpu/drm/virtio/Makefile b/drivers/gpu/drm/virtio/Makefile
index f29deec83d1f..4e90cc8fa651 100644
--- a/drivers/gpu/drm/virtio/Makefile
+++ b/drivers/gpu/drm/virtio/Makefile
@@ -3,7 +3,7 @@
 # Makefile for the drm device driver.  This driver provides support for the
 # Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher.
 
-virtio-gpu-y := virtgpu_drv.o virtgpu_kms.o virtgpu_drm_bus.o virtgpu_gem.o \
+virtio-gpu-y := virtgpu_drv.o virtgpu_kms.o virtgpu_gem.o \
         virtgpu_fb.o virtgpu_display.o virtgpu_vq.o virtgpu_ttm.o \
         virtgpu_fence.o virtgpu_object.o virtgpu_debugfs.o virtgpu_plane.o \
         virtgpu_ioctl.o virtgpu_prime.o
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index e1c223e18d86..87d7c49cf057 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -243,12 +243,8 @@ static enum drm_connector_status virtio_gpu_conn_detect(
 
 static void virtio_gpu_conn_destroy(struct drm_connector *connector)
 {
-        struct virtio_gpu_output *virtio_gpu_output =
-                drm_connector_to_virtio_gpu_output(connector);
-
         drm_connector_unregister(connector);
         drm_connector_cleanup(connector);
-        kfree(virtio_gpu_output);
 }
 
 static const struct drm_connector_funcs virtio_gpu_connector_funcs = {
@@ -362,7 +358,7 @@ static const struct drm_mode_config_funcs virtio_gpu_mode_funcs = {
         .atomic_commit = drm_atomic_helper_commit,
 };
 
-int virtio_gpu_modeset_init(struct virtio_gpu_device *vgdev)
+void virtio_gpu_modeset_init(struct virtio_gpu_device *vgdev)
 {
         int i;
 
@@ -381,7 +377,6 @@ int virtio_gpu_modeset_init(struct virtio_gpu_device *vgdev)
                 vgdev_output_init(vgdev, i);
 
         drm_mode_config_reset(vgdev->ddev);
-        return 0;
 }
 
 void virtio_gpu_modeset_fini(struct virtio_gpu_device *vgdev)
diff --git a/drivers/gpu/drm/virtio/virtgpu_drm_bus.c b/drivers/gpu/drm/virtio/virtgpu_drm_bus.c
deleted file mode 100644
index 0887e0b64b9c..000000000000
--- a/drivers/gpu/drm/virtio/virtgpu_drm_bus.c
+++ /dev/null
@@ -1,103 +0,0 @@
-/*
- * Copyright (C) 2015 Red Hat, Inc.
- * All Rights Reserved.
- *
- * Permission is hereby granted, free of charge, to any person obtaining
- * a copy of this software and associated documentation files (the
- * "Software"), to deal in the Software without restriction, including
- * without limitation the rights to use, copy, modify, merge, publish,
- * distribute, sublicense, and/or sell copies of the Software, and to
- * permit persons to whom the Software is furnished to do so, subject to
- * the following conditions:
- *
- * The above copyright notice and this permission notice (including the
- * next paragraph) shall be included in all copies or substantial
- * portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
- * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
- * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
- * IN NO EVENT SHALL THE COPYRIGHT OWNER(S) AND/OR ITS SUPPLIERS BE
- * LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
- * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
- * WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
- */
-
-#include <linux/pci.h>
-#include <drm/drm_fb_helper.h>
-
-#include "virtgpu_drv.h"
-
-int drm_virtio_init(struct drm_driver *driver, struct virtio_device *vdev)
-{
-        struct drm_device *dev;
-        int ret;
-
-        dev = drm_dev_alloc(driver, &vdev->dev);
-        if (IS_ERR(dev))
-                return PTR_ERR(dev);
-        vdev->priv = dev;
-
-        if (strcmp(vdev->dev.parent->bus->name, "pci") == 0) {
-                struct pci_dev *pdev = to_pci_dev(vdev->dev.parent);
-                const char *pname = dev_name(&pdev->dev);
-                bool vga = (pdev->class >> 8) == PCI_CLASS_DISPLAY_VGA;
-                char unique[20];
-
-                DRM_INFO("pci: %s detected at %s\n",
-                         vga ? "virtio-vga" : "virtio-gpu-pci",
-                         pname);
-                dev->pdev = pdev;
-                if (vga)
-                        drm_fb_helper_remove_conflicting_pci_framebuffers(pdev,
-                                                                          0,
-                                                                          "virtiodrmfb");
-
-                /*
-                 * Normally the drm_dev_set_unique() call is done by core DRM.
-                 * The following comment covers, why virtio cannot rely on it.
-                 *
-                 * Unlike the other virtual GPU drivers, virtio abstracts the
-                 * underlying bus type by using struct virtio_device.
-                 *
-                 * Hence the dev_is_pci() check, used in core DRM, will fail
-                 * and the unique returned will be the virtio_device "virtio0",
-                 * while a "pci:..." one is required.
-                 *
-                 * A few other ideas were considered:
-                 * - Extend the dev_is_pci() check [in drm_set_busid] to
-                 *   consider virtio.
-                 *   Seems like a bigger hack than what we have already.
-                 *
-                 * - Point drm_device::dev to the parent of the virtio_device
-                 *   Semantic changes:
-                 *   * Using the wrong device for i2c, framebuffer_alloc and
-                 *     prime import.
-                 *   Visual changes:
-                 *   * Helpers such as DRM_DEV_ERROR, dev_info, drm_printer,
-                 *     will print the wrong information.
-                 *
-                 * We could address the latter issues, by introducing
-                 * drm_device::bus_dev, ... which would be used solely for this.
-                 *
-                 * So for the moment keep things as-is, with a bulky comment
-                 * for the next person who feels like removing this
-                 * drm_dev_set_unique() quirk.
-                 */
-                snprintf(unique, sizeof(unique), "pci:%s", pname);
-                ret = drm_dev_set_unique(dev, unique);
-                if (ret)
-                        goto err_free;
-
-        }
-
-        ret = drm_dev_register(dev, 0);
-        if (ret)
-                goto err_free;
-
-        return 0;
-
-err_free:
-        drm_dev_put(dev);
-        return ret;
-}
diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.c b/drivers/gpu/drm/virtio/virtgpu_drv.c
index 7df50920c1e0..af92964b6889 100644
--- a/drivers/gpu/drm/virtio/virtgpu_drv.c
+++ b/drivers/gpu/drm/virtio/virtgpu_drv.c
@@ -40,8 +40,60 @@ static int virtio_gpu_modeset = -1;
 MODULE_PARM_DESC(modeset, "Disable/Enable modesetting");
 module_param_named(modeset, virtio_gpu_modeset, int, 0400);
 
+static int virtio_gpu_pci_quirk(struct drm_device *dev, struct virtio_device *vdev)
+{
+        struct pci_dev *pdev = to_pci_dev(vdev->dev.parent);
+        const char *pname = dev_name(&pdev->dev);
+        bool vga = (pdev->class >> 8) == PCI_CLASS_DISPLAY_VGA;
+        char unique[20];
+
+        DRM_INFO("pci: %s detected at %s\n",
+                 vga ? "virtio-vga" : "virtio-gpu-pci",
+                 pname);
+        dev->pdev = pdev;
+        if (vga)
+                drm_fb_helper_remove_conflicting_pci_framebuffers(pdev,
+                                                                  0,
+                                                                  "virtiodrmfb");
+
+        /*
+         * Normally the drm_dev_set_unique() call is done by core DRM.
+         * The following comment covers, why virtio cannot rely on it.
+         *
+         * Unlike the other virtual GPU drivers, virtio abstracts the
+         * underlying bus type by using struct virtio_device.
+         *
+         * Hence the dev_is_pci() check, used in core DRM, will fail
+         * and the unique returned will be the virtio_device "virtio0",
+         * while a "pci:..." one is required.
+         *
+         * A few other ideas were considered:
+         * - Extend the dev_is_pci() check [in drm_set_busid] to
+         *   consider virtio.
+         *   Seems like a bigger hack than what we have already.
+         *
+         * - Point drm_device::dev to the parent of the virtio_device
+         *   Semantic changes:
+         *   * Using the wrong device for i2c, framebuffer_alloc and
+         *     prime import.
+         *   Visual changes:
+         *   * Helpers such as DRM_DEV_ERROR, dev_info, drm_printer,
+         *     will print the wrong information.
+         *
+         * We could address the latter issues, by introducing
+         * drm_device::bus_dev, ... which would be used solely for this.
+         *
+         * So for the moment keep things as-is, with a bulky comment
+         * for the next person who feels like removing this
+         * drm_dev_set_unique() quirk.
+         */
+        snprintf(unique, sizeof(unique), "pci:%s", pname);
+        return drm_dev_set_unique(dev, unique);
+}
+
 static int virtio_gpu_probe(struct virtio_device *vdev)
 {
+        struct drm_device *dev;
         int ret;
 
         if (vgacon_text_force() && virtio_gpu_modeset == -1)
@@ -50,18 +102,39 @@ static int virtio_gpu_probe(struct virtio_device *vdev)
         if (virtio_gpu_modeset == 0)
                 return -EINVAL;
 
-        ret = drm_virtio_init(&driver, vdev);
+        dev = drm_dev_alloc(&driver, &vdev->dev);
+        if (IS_ERR(dev))
+                return PTR_ERR(dev);
+        vdev->priv = dev;
+
+        if (!strcmp(vdev->dev.parent->bus->name, "pci")) {
+                ret = virtio_gpu_pci_quirk(dev, vdev);
+                if (ret)
+                        goto err_free;
+        }
+
+        ret = virtio_gpu_init(dev);
         if (ret)
-                return ret;
+                goto err_free;
+
+        ret = drm_dev_register(dev, 0);
+        if (ret)
+                goto err_free;
 
         drm_fbdev_generic_setup(vdev->priv, 32);
         return 0;
+
+err_free:
+        drm_dev_put(dev);
+        return ret;
 }
 
 static void virtio_gpu_remove(struct virtio_device *vdev)
 {
         struct drm_device *dev = vdev->priv;
 
+        drm_dev_unregister(dev);
+        virtio_gpu_deinit(dev);
         drm_put_dev(dev);
 }
 
@@ -123,8 +196,6 @@ static const struct file_operations virtio_gpu_driver_fops = {
 
 static struct drm_driver driver = {
         .driver_features = DRIVER_MODESET | DRIVER_GEM | DRIVER_PRIME | DRIVER_RENDER | DRIVER_ATOMIC,
-        .load = virtio_gpu_driver_load,
-        .unload = virtio_gpu_driver_unload,
         .open = virtio_gpu_driver_open,
         .postclose = virtio_gpu_driver_postclose,
 
diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.h b/drivers/gpu/drm/virtio/virtgpu_drv.h
index 63704915f8ce..cf896d879382 100644
--- a/drivers/gpu/drm/virtio/virtgpu_drv.h
+++ b/drivers/gpu/drm/virtio/virtgpu_drv.h
@@ -50,9 +50,6 @@
 #define DRIVER_MINOR 1
 #define DRIVER_PATCHLEVEL 0
 
-/* virtgpu_drm_bus.c */
-int drm_virtio_init(struct drm_driver *driver, struct virtio_device *vdev);
-
 struct virtio_gpu_object {
         struct drm_gem_object gem_base;
         uint32_t hw_res_handle;
@@ -209,8 +206,8 @@ struct virtio_gpu_fpriv {
 extern struct drm_ioctl_desc virtio_gpu_ioctls[DRM_VIRTIO_NUM_IOCTLS];
 
 /* virtio_kms.c */
-int virtio_gpu_driver_load(struct drm_device *dev, unsigned long flags);
-void virtio_gpu_driver_unload(struct drm_device *dev);
+int virtio_gpu_init(struct drm_device *dev);
+void virtio_gpu_deinit(struct drm_device *dev);
 int virtio_gpu_driver_open(struct drm_device *dev, struct drm_file *file);
 void virtio_gpu_driver_postclose(struct drm_device *dev, struct drm_file *file);
 
@@ -320,7 +317,7 @@ int virtio_gpu_framebuffer_init(struct drm_device *dev,
                                 struct virtio_gpu_framebuffer *vgfb,
                                 const struct drm_mode_fb_cmd2 *mode_cmd,
                                 struct drm_gem_object *obj);
-int virtio_gpu_modeset_init(struct virtio_gpu_device *vgdev);
+void virtio_gpu_modeset_init(struct virtio_gpu_device *vgdev);
 void virtio_gpu_modeset_fini(struct virtio_gpu_device *vgdev);
 
 /* virtio_gpu_plane.c */
@@ -337,7 +334,6 @@ int virtio_gpu_mmap(struct file *filp, struct vm_area_struct *vma);
 /* virtio_gpu_fence.c */
 struct virtio_gpu_fence *virtio_gpu_fence_alloc(
         struct virtio_gpu_device *vgdev);
-void virtio_gpu_fence_cleanup(struct virtio_gpu_fence *fence);
 int virtio_gpu_fence_emit(struct virtio_gpu_device *vgdev,
                           struct virtio_gpu_ctrl_hdr *cmd_hdr,
                           struct virtio_gpu_fence *fence);
diff --git a/drivers/gpu/drm/virtio/virtgpu_fence.c b/drivers/gpu/drm/virtio/virtgpu_fence.c
index 4d6826b27814..21bd4c4a32d1 100644
--- a/drivers/gpu/drm/virtio/virtgpu_fence.c
+++ b/drivers/gpu/drm/virtio/virtgpu_fence.c
@@ -81,14 +81,6 @@ struct virtio_gpu_fence *virtio_gpu_fence_alloc(struct virtio_gpu_device *vgdev)
         return fence;
 }
 
-void virtio_gpu_fence_cleanup(struct virtio_gpu_fence *fence)
-{
-        if (!fence)
-                return;
-
-        dma_fence_put(&fence->f);
-}
-
 int virtio_gpu_fence_emit(struct virtio_gpu_device *vgdev,
                           struct virtio_gpu_ctrl_hdr *cmd_hdr,
                           struct virtio_gpu_fence *fence)
diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
index 161b80fee492..14ce8188c052 100644
--- a/drivers/gpu/drm/virtio/virtgpu_ioctl.c
+++ b/drivers/gpu/drm/virtio/virtgpu_ioctl.c
@@ -351,7 +351,7 @@ static int virtio_gpu_resource_create_ioctl(struct drm_device *dev, void *data,
                 virtio_gpu_cmd_resource_create_3d(vgdev, qobj, &rc_3d);
                 ret = virtio_gpu_object_attach(vgdev, qobj, fence);
                 if (ret) {
-                        virtio_gpu_fence_cleanup(fence);
+                        dma_fence_put(&fence->f);
                         goto fail_backoff;
                 }
                 ttm_eu_fence_buffer_objects(&ticket, &validate_list, &fence->f);
diff --git a/drivers/gpu/drm/virtio/virtgpu_kms.c b/drivers/gpu/drm/virtio/virtgpu_kms.c
index 1072064a0db2..84b6a6bf00c6 100644
--- a/drivers/gpu/drm/virtio/virtgpu_kms.c
+++ b/drivers/gpu/drm/virtio/virtgpu_kms.c
@@ -106,7 +106,7 @@ static void virtio_gpu_get_capsets(struct virtio_gpu_device *vgdev,
         vgdev->num_capsets = num_capsets;
 }
 
-int virtio_gpu_driver_load(struct drm_device *dev, unsigned long flags)
+int virtio_gpu_init(struct drm_device *dev)
 {
         static vq_callback_t *callbacks[] = {
                 virtio_gpu_ctrl_ack, virtio_gpu_cursor_ack
@@ -193,9 +193,7 @@ int virtio_gpu_driver_load(struct drm_device *dev, unsigned long flags)
                      num_capsets, &num_capsets);
         DRM_INFO("number of cap sets: %d\n", num_capsets);
 
-        ret = virtio_gpu_modeset_init(vgdev);
-        if (ret)
-                goto err_modeset;
+        virtio_gpu_modeset_init(vgdev);
 
         virtio_device_ready(vgdev->vdev);
         vgdev->vqs_ready = true;
@@ -209,7 +207,6 @@ int virtio_gpu_driver_load(struct drm_device *dev, unsigned long flags)
                            5 * HZ);
         return 0;
 
-err_modeset:
 err_scanouts:
         virtio_gpu_ttm_fini(vgdev);
 err_ttm:
@@ -231,7 +228,7 @@ static void virtio_gpu_cleanup_cap_cache(struct virtio_gpu_device *vgdev)
         }
 }
 
-void virtio_gpu_driver_unload(struct drm_device *dev)
+void virtio_gpu_deinit(struct drm_device *dev)
 {
         struct virtio_gpu_device *vgdev = dev->dev_private;
 
@@ -239,6 +236,7 @@ void virtio_gpu_driver_unload(struct drm_device *dev)
         flush_work(&vgdev->ctrlq.dequeue_work);
         flush_work(&vgdev->cursorq.dequeue_work);
         flush_work(&vgdev->config_changed_work);
+        vgdev->vdev->config->reset(vgdev->vdev);
         vgdev->vdev->config->del_vqs(vgdev->vdev);
 
         virtio_gpu_modeset_fini(vgdev);
diff --git a/drivers/gpu/drm/virtio/virtgpu_plane.c b/drivers/gpu/drm/virtio/virtgpu_plane.c
index ead5c53d4e21..024c2aa0c929 100644
--- a/drivers/gpu/drm/virtio/virtgpu_plane.c
+++ b/drivers/gpu/drm/virtio/virtgpu_plane.c
@@ -130,11 +130,12 @@ static void virtio_gpu_primary_plane_update(struct drm_plane *plane,
                                    plane->state->src_h >> 16,
                                    plane->state->src_x >> 16,
                                    plane->state->src_y >> 16);
-        virtio_gpu_cmd_resource_flush(vgdev, handle,
-                                      plane->state->src_x >> 16,
-                                      plane->state->src_y >> 16,
-                                      plane->state->src_w >> 16,
-                                      plane->state->src_h >> 16);
+        if (handle)
+                virtio_gpu_cmd_resource_flush(vgdev, handle,
+                                              plane->state->src_x >> 16,
+                                              plane->state->src_y >> 16,
+                                              plane->state->src_w >> 16,
+                                              plane->state->src_h >> 16);
 }
 
 static int virtio_gpu_cursor_prepare_fb(struct drm_plane *plane,
@@ -168,8 +169,10 @@ static void virtio_gpu_cursor_cleanup_fb(struct drm_plane *plane,
                 return;
 
         vgfb = to_virtio_gpu_framebuffer(plane->state->fb);
-        if (vgfb->fence)
-                virtio_gpu_fence_cleanup(vgfb->fence);
+        if (vgfb->fence) {
+                dma_fence_put(&vgfb->fence->f);
+                vgfb->fence = NULL;
+        }
 }
 
 static void virtio_gpu_cursor_plane_update(struct drm_plane *plane,
diff --git a/drivers/gpu/drm/virtio/virtgpu_vq.c b/drivers/gpu/drm/virtio/virtgpu_vq.c
index e27c4aedb809..6bc2008b0d0d 100644
--- a/drivers/gpu/drm/virtio/virtgpu_vq.c
+++ b/drivers/gpu/drm/virtio/virtgpu_vq.c
@@ -192,8 +192,16 @@ void virtio_gpu_dequeue_ctrl_func(struct work_struct *work)
 
         list_for_each_entry_safe(entry, tmp, &reclaim_list, list) {
                 resp = (struct virtio_gpu_ctrl_hdr *)entry->resp_buf;
-                if (resp->type != cpu_to_le32(VIRTIO_GPU_RESP_OK_NODATA))
-                        DRM_DEBUG("response 0x%x\n", le32_to_cpu(resp->type));
+                if (resp->type != cpu_to_le32(VIRTIO_GPU_RESP_OK_NODATA)) {
+                        if (resp->type >= cpu_to_le32(VIRTIO_GPU_RESP_ERR_UNSPEC)) {
+                                struct virtio_gpu_ctrl_hdr *cmd;
+                                cmd = (struct virtio_gpu_ctrl_hdr *)entry->buf;
+                                DRM_ERROR("response 0x%x (command 0x%x)\n",
+                                          le32_to_cpu(resp->type),
+                                          le32_to_cpu(cmd->type));
+                        } else
+                                DRM_DEBUG("response 0x%x\n", le32_to_cpu(resp->type));
+                }
                 if (resp->flags & cpu_to_le32(VIRTIO_GPU_FLAG_FENCE)) {
                         u64 f = le64_to_cpu(resp->fence_id);