drm/i915: Simplify expression &to_i915(dev)->drm
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 21 Jul 2023 21:21:33 +0000 (23:21 +0200)
committerAndi Shyti <andi.shyti@linux.intel.com>
Tue, 25 Jul 2023 13:31:34 +0000 (15:31 +0200)
to_i915 is defined as

container_of(dev, struct drm_i915_private, drm);

So for a struct drm_device *dev, to_i915(dev)->drm is just dev. Simplify
accordingly.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230721212133.271118-1-u.kleine-koenig@pengutronix.de
drivers/gpu/drm/i915/display/intel_display_debugfs.c
drivers/gpu/drm/i915/gt/intel_ggtt_fencing.c
drivers/gpu/drm/i915/i915_vma.c

index 165e2c7e3126a20893dbd93e2da14bba3e1b15c1..63c1fb9e479faa9a611f84517bc5b5d9f9a81d2b 100644 (file)
@@ -819,8 +819,7 @@ static ssize_t i915_displayport_test_active_write(struct file *file,
        if (IS_ERR(input_buffer))
                return PTR_ERR(input_buffer);
 
-       drm_dbg(&to_i915(dev)->drm,
-               "Copied %d bytes from user\n", (unsigned int)len);
+       drm_dbg(dev, "Copied %d bytes from user\n", (unsigned int)len);
 
        drm_connector_list_iter_begin(dev, &conn_iter);
        drm_for_each_connector_iter(connector, &conn_iter) {
@@ -839,8 +838,7 @@ static ssize_t i915_displayport_test_active_write(struct file *file,
                        status = kstrtoint(input_buffer, 10, &val);
                        if (status < 0)
                                break;
-                       drm_dbg(&to_i915(dev)->drm,
-                               "Got %d for test active\n", val);
+                       drm_dbg(dev, "Got %d for test active\n", val);
                        /* To prevent erroneous activation of the compliance
                         * testing code, only accept an actual value of 1 here
                         */
index 37d0b0fe791df6746cf8236c5d00b79030a42306..40371b8a9bbbd70ccf0e38184478df73f666856d 100644 (file)
@@ -818,7 +818,7 @@ i915_gem_object_save_bit_17_swizzle(struct drm_i915_gem_object *obj,
        if (obj->bit_17 == NULL) {
                obj->bit_17 = bitmap_zalloc(page_count, GFP_KERNEL);
                if (obj->bit_17 == NULL) {
-                       drm_err(&to_i915(obj->base.dev)->drm,
+                       drm_err(obj->base.dev,
                                "Failed to allocate memory for bit 17 record\n");
                        return;
                }
index ffb425ba591c7935649d80da8d6387090aab9e22..af2e49f90f2f855a2b7e21f74e9f7b4ecaaa9598 100644 (file)
@@ -74,14 +74,14 @@ static void vma_print_allocator(struct i915_vma *vma, const char *reason)
        char buf[512];
 
        if (!vma->node.stack) {
-               drm_dbg(&to_i915(vma->obj->base.dev)->drm,
+               drm_dbg(vma->obj->base.dev,
                        "vma.node [%08llx + %08llx] %s: unknown owner\n",
                        vma->node.start, vma->node.size, reason);
                return;
        }
 
        stack_depot_snprint(vma->node.stack, buf, sizeof(buf), 0);
-       drm_dbg(&to_i915(vma->obj->base.dev)->drm,
+       drm_dbg(vma->obj->base.dev,
                "vma.node [%08llx + %08llx] %s: inserted at %s\n",
                vma->node.start, vma->node.size, reason, buf);
 }
@@ -805,7 +805,7 @@ i915_vma_insert(struct i915_vma *vma, struct i915_gem_ww_ctx *ww,
         * attempt to find space.
         */
        if (size > end - 2 * guard) {
-               drm_dbg(&to_i915(vma->obj->base.dev)->drm,
+               drm_dbg(vma->obj->base.dev,
                        "Attempting to bind an object larger than the aperture: request=%llu > %s aperture=%llu\n",
                        size, flags & PIN_MAPPABLE ? "mappable" : "total", end);
                return -ENOSPC;