With the overlay IOCTLs now returning no overlay support and -ENODEV,
let's drop all the code for the display version 2-4 overlay support.

Cc: David Airlie <[email protected]>
Cc: Simona Vetter <[email protected]>
Cc: Ville Syrjälä <[email protected]>
Signed-off-by: Jani Nikula <[email protected]>
---
 drivers/gpu/drm/i915/Makefile                 |    1 -
 drivers/gpu/drm/i915/display/intel_display.c  |   13 -
 .../gpu/drm/i915/display/intel_display_core.h |    2 -
 .../drm/i915/display/intel_display_device.c   |   11 -
 .../drm/i915/display/intel_display_device.h   |    4 -
 .../drm/i915/display/intel_display_driver.c   |    5 -
 .../drm/i915/display/intel_display_snapshot.c |    5 -
 .../drm/i915/display/intel_display_types.h    |    1 -
 drivers/gpu/drm/i915/display/intel_overlay.c  | 1556 -----------------
 drivers/gpu/drm/i915/display/intel_overlay.h  |   75 -
 drivers/gpu/drm/i915/gt/intel_reset.c         |    4 -
 11 files changed, 1677 deletions(-)
 delete mode 100644 drivers/gpu/drm/i915/display/intel_overlay.c
 delete mode 100644 drivers/gpu/drm/i915/display/intel_overlay.h

diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
index 7d726e8c21bf..fbe5a1b8f062 100644
--- a/drivers/gpu/drm/i915/Makefile
+++ b/drivers/gpu/drm/i915/Makefile
@@ -296,7 +296,6 @@ i915-y += \
        display/intel_modeset_lock.o \
        display/intel_modeset_setup.o \
        display/intel_modeset_verify.o \
-       display/intel_overlay.o \
        display/intel_parent.o \
        display/intel_pch.o \
        display/intel_pch_display.o \
diff --git a/drivers/gpu/drm/i915/display/intel_display.c 
b/drivers/gpu/drm/i915/display/intel_display.c
index 564d11925af3..db8e95237f60 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -106,7 +106,6 @@
 #include "intel_lvds_regs.h"
 #include "intel_modeset_setup.h"
 #include "intel_modeset_verify.h"
-#include "intel_overlay.h"
 #include "intel_panel.h"
 #include "intel_pch_display.h"
 #include "intel_pch_refclk.h"
@@ -772,16 +771,6 @@ intel_get_crtc_new_encoder(const struct intel_atomic_state 
*state,
        return encoder;
 }
 
-static void intel_crtc_dpms_overlay_disable(struct intel_crtc *crtc)
-{
-       if (crtc->overlay)
-               (void) intel_overlay_switch_off(crtc->overlay);
-
-       /* Let userspace switch the overlay on again. In most cases userspace
-        * has to recompute where to put it anyway.
-        */
-}
-
 static bool needs_nv12_wa(const struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(crtc_state);
@@ -1273,8 +1262,6 @@ static void intel_crtc_disable_planes(struct 
intel_atomic_state *state,
        unsigned fb_bits = 0;
        int i;
 
-       intel_crtc_dpms_overlay_disable(crtc);
-
        for_each_old_intel_plane_in_state(state, plane, old_plane_state, i) {
                if (crtc->pipe != plane->pipe ||
                    !(update_mask & BIT(plane->id)))
diff --git a/drivers/gpu/drm/i915/display/intel_display_core.h 
b/drivers/gpu/drm/i915/display/intel_display_core.h
index d708d322aa85..5a60ded6d5ce 100644
--- a/drivers/gpu/drm/i915/display/intel_display_core.h
+++ b/drivers/gpu/drm/i915/display/intel_display_core.h
@@ -50,7 +50,6 @@ struct intel_fdi_funcs;
 struct intel_hotplug_funcs;
 struct intel_initial_plane_config;
 struct intel_opregion;
-struct intel_overlay;
 struct task_struct;
 
 /* Amount of SAGV/QGV points, BSpec precisely defines this */
@@ -629,7 +628,6 @@ struct intel_display {
        struct intel_frontbuffer_tracking fb_tracking;
        struct intel_hotplug hotplug;
        struct intel_opregion *opregion;
-       struct intel_overlay *overlay;
        struct intel_display_params params;
        struct intel_vbt_data vbt;
        struct intel_dmc_wl wl;
diff --git a/drivers/gpu/drm/i915/display/intel_display_device.c 
b/drivers/gpu/drm/i915/display/intel_display_device.c
index 471f236c9ddf..62c76ab22942 100644
--- a/drivers/gpu/drm/i915/display/intel_display_device.c
+++ b/drivers/gpu/drm/i915/display/intel_display_device.c
@@ -229,9 +229,7 @@ static const struct intel_display_device_info no_display = 
{};
        }
 
 #define I830_DISPLAY \
-       .has_overlay = 1, \
        .cursor_needs_physical = 1, \
-       .overlay_needs_physical = 1, \
        .has_gmch = 1, \
        I9XX_PIPE_OFFSETS, \
        I9XX_CURSOR_OFFSETS, \
@@ -243,8 +241,6 @@ static const struct intel_display_device_info no_display = 
{};
                BIT(TRANSCODER_A) | BIT(TRANSCODER_B)
 
 #define I845_DISPLAY \
-       .has_overlay = 1, \
-       .overlay_needs_physical = 1, \
        .has_gmch = 1, \
        I845_PIPE_OFFSETS, \
        I845_CURSOR_OFFSETS, \
@@ -296,7 +292,6 @@ static const struct platform_desc i865g_desc = {
 
 #define GEN3_DISPLAY   \
        .has_gmch = 1, \
-       .has_overlay = 1, \
        I9XX_PIPE_OFFSETS, \
        I9XX_CURSOR_OFFSETS, \
        \
@@ -312,7 +307,6 @@ static const struct platform_desc i915g_desc = {
                GEN3_DISPLAY,
                I845_COLORS,
                .cursor_needs_physical = 1,
-               .overlay_needs_physical = 1,
        },
 };
 
@@ -323,7 +317,6 @@ static const struct platform_desc i915gm_desc = {
                GEN3_DISPLAY,
                I9XX_COLORS,
                .cursor_needs_physical = 1,
-               .overlay_needs_physical = 1,
                .supports_tv = 1,
 
                .__runtime_defaults.fbc_mask = BIT(INTEL_FBC_A),
@@ -337,7 +330,6 @@ static const struct platform_desc i945g_desc = {
                I845_COLORS,
                .has_hotplug = 1,
                .cursor_needs_physical = 1,
-               .overlay_needs_physical = 1,
        },
 };
 
@@ -349,7 +341,6 @@ static const struct platform_desc i945gm_desc = {
                I9XX_COLORS,
                .has_hotplug = 1,
                .cursor_needs_physical = 1,
-               .overlay_needs_physical = 1,
                .supports_tv = 1,
 
                .__runtime_defaults.fbc_mask = BIT(INTEL_FBC_A),
@@ -398,7 +389,6 @@ static const struct platform_desc i965g_desc = {
        PLATFORM(i965g),
        .info = &(const struct intel_display_device_info) {
                GEN4_DISPLAY,
-               .has_overlay = 1,
 
                .__runtime_defaults.port_mask = BIT(PORT_B) | BIT(PORT_C), /* 
SDVO B/C */
        },
@@ -409,7 +399,6 @@ static const struct platform_desc i965gm_desc = {
        PLATFORM_GROUP(mobile),
        .info = &(const struct intel_display_device_info) {
                GEN4_DISPLAY,
-               .has_overlay = 1,
                .supports_tv = 1,
 
                .__runtime_defaults.port_mask = BIT(PORT_B) | BIT(PORT_C), /* 
SDVO B/C */
diff --git a/drivers/gpu/drm/i915/display/intel_display_device.h 
b/drivers/gpu/drm/i915/display/intel_display_device.h
index e84c190dcc4f..8ece4cf194bd 100644
--- a/drivers/gpu/drm/i915/display/intel_display_device.h
+++ b/drivers/gpu/drm/i915/display/intel_display_device.h
@@ -139,10 +139,8 @@ struct intel_display_platforms {
        func(has_hotplug); \
        func(has_hti); \
        func(has_ipc); \
-       func(has_overlay); \
        func(has_psr); \
        func(has_psr_hw_tracking); \
-       func(overlay_needs_physical); \
        func(supports_tv);
 
 #define HAS_128B_Y_TILING(__display)   (!(__display)->platform.i915g && 
!(__display)->platform.i915gm)
@@ -191,7 +189,6 @@ struct intel_display_platforms {
 #define HAS_LT_PHY(__display)          ((__display)->platform.novalake)
 #define HAS_MBUS_JOINING(__display)    ((__display)->platform.alderlake_p || 
DISPLAY_VER(__display) >= 14)
 #define HAS_MSO(__display)             (DISPLAY_VER(__display) >= 12)
-#define HAS_OVERLAY(__display)         (DISPLAY_INFO(__display)->has_overlay)
 #define HAS_PIPEDMC(__display)         (DISPLAY_VER(__display) >= 12)
 #define HAS_PIXEL_NORMALIZER(__display)        (DISPLAY_VER(__display) >= 35)
 #define HAS_PSR(__display)             (DISPLAY_INFO(__display)->has_psr)
@@ -209,7 +206,6 @@ struct intel_display_platforms {
 #define HAS_VRR(__display)             (DISPLAY_VER(__display) >= 11)
 #define HAS_VRR_DC_BALANCE(__display)  (DISPLAY_VER(__display) >= 30)
 #define INTEL_NUM_PIPES(__display)     
(hweight8(DISPLAY_RUNTIME_INFO(__display)->pipe_mask))
-#define OVERLAY_NEEDS_PHYSICAL(__display)      
(DISPLAY_INFO(__display)->overlay_needs_physical)
 #define SUPPORTS_TV(__display)         (DISPLAY_INFO(__display)->supports_tv)
 
 /* Check that device has a display IP version within the specific range. */
diff --git a/drivers/gpu/drm/i915/display/intel_display_driver.c 
b/drivers/gpu/drm/i915/display/intel_display_driver.c
index 967da2b41316..229c7ed7a060 100644
--- a/drivers/gpu/drm/i915/display/intel_display_driver.c
+++ b/drivers/gpu/drm/i915/display/intel_display_driver.c
@@ -56,7 +56,6 @@
 #include "intel_modeset_lock.h"
 #include "intel_modeset_setup.h"
 #include "intel_opregion.h"
-#include "intel_overlay.h"
 #include "intel_pmdemand.h"
 #include "intel_pps.h"
 #include "intel_psr.h"
@@ -530,8 +529,6 @@ int intel_display_driver_probe(struct intel_display 
*display)
        if (ret)
                drm_dbg_kms(display->drm, "Initial modeset failed, %d\n", ret);
 
-       intel_overlay_setup(display);
-
        /* Only enable hotplug handling once the fbdev is fully set up. */
        intel_hpd_init(display);
 
@@ -622,8 +619,6 @@ void intel_display_driver_remove_noirq(struct intel_display 
*display)
 
        intel_dp_tunnel_mgr_cleanup(display);
 
-       intel_overlay_cleanup(display);
-
        intel_gmbus_teardown(display);
 
        destroy_workqueue(display->hotplug.dp_wq);
diff --git a/drivers/gpu/drm/i915/display/intel_display_snapshot.c 
b/drivers/gpu/drm/i915/display/intel_display_snapshot.c
index 66087302fdbc..a8022c02dfe3 100644
--- a/drivers/gpu/drm/i915/display/intel_display_snapshot.c
+++ b/drivers/gpu/drm/i915/display/intel_display_snapshot.c
@@ -11,7 +11,6 @@
 #include "intel_display_params.h"
 #include "intel_display_snapshot.h"
 #include "intel_dmc.h"
-#include "intel_overlay.h"
 
 struct intel_display_snapshot {
        struct intel_display *display;
@@ -19,7 +18,6 @@ struct intel_display_snapshot {
        struct intel_display_device_info info;
        struct intel_display_runtime_info runtime_info;
        struct intel_display_params params;
-       struct intel_overlay_snapshot *overlay;
        struct intel_dmc_snapshot *dmc;
        struct intel_display_irq_snapshot *irq;
 };
@@ -41,7 +39,6 @@ struct intel_display_snapshot 
*intel_display_snapshot_capture(struct intel_displ
        intel_display_params_copy(&snapshot->params);
 
        snapshot->irq = intel_display_irq_snapshot_capture(display);
-       snapshot->overlay = intel_overlay_snapshot_capture(display);
        snapshot->dmc = intel_dmc_snapshot_capture(display);
 
        return snapshot;
@@ -61,7 +58,6 @@ void intel_display_snapshot_print(const struct 
intel_display_snapshot *snapshot,
        intel_display_params_dump(&snapshot->params, 
display->drm->driver->name, p);
 
        intel_display_irq_snapshot_print(snapshot->irq, p);
-       intel_overlay_snapshot_print(snapshot->overlay, p);
        intel_dmc_snapshot_print(snapshot->dmc, p);
 }
 
@@ -73,7 +69,6 @@ void intel_display_snapshot_free(struct 
intel_display_snapshot *snapshot)
        intel_display_params_free(&snapshot->params);
 
        kfree(snapshot->irq);
-       kfree(snapshot->overlay);
        kfree(snapshot->dmc);
        kfree(snapshot);
 }
diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h 
b/drivers/gpu/drm/i915/display/intel_display_types.h
index e6298279dc89..31c88e205634 100644
--- a/drivers/gpu/drm/i915/display/intel_display_types.h
+++ b/drivers/gpu/drm/i915/display/intel_display_types.h
@@ -1499,7 +1499,6 @@ struct intel_crtc {
 
        struct intel_display_power_domain_set enabled_power_domains;
        struct intel_display_power_domain_set hw_readout_power_domains;
-       struct intel_overlay *overlay;
 
        struct intel_crtc_state *config;
 
diff --git a/drivers/gpu/drm/i915/display/intel_overlay.c 
b/drivers/gpu/drm/i915/display/intel_overlay.c
deleted file mode 100644
index 88eb7ae5765c..000000000000
--- a/drivers/gpu/drm/i915/display/intel_overlay.c
+++ /dev/null
@@ -1,1556 +0,0 @@
-/*
- * Copyright © 2009
- *
- * 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 AUTHORS OR COPYRIGHT HOLDERS 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.
- *
- * Authors:
- *    Daniel Vetter <[email protected]>
- *
- * Derived from Xorg ddx, xf86-video-intel, src/i830_video.c
- */
-
-#include <drm/drm_fourcc.h>
-#include <drm/drm_print.h>
-
-#include "gem/i915_gem_internal.h"
-#include "gem/i915_gem_object_frontbuffer.h"
-#include "gem/i915_gem_pm.h"
-
-#include "gt/intel_gpu_commands.h"
-#include "gt/intel_ring.h"
-
-#include "i915_drv.h"
-#include "i915_reg.h"
-#include "intel_color_regs.h"
-#include "intel_de.h"
-#include "intel_display_regs.h"
-#include "intel_display_types.h"
-#include "intel_frontbuffer.h"
-#include "intel_overlay.h"
-#include "intel_pci_config.h"
-#include "intel_pfit_regs.h"
-
-/* Limits for overlay size. According to intel doc, the real limits are:
- * Y width: 4095, UV width (planar): 2047, Y height: 2047,
- * UV width (planar): * 1023. But the xorg thinks 2048 for height and width. 
Use
- * the minimum of both.
- */
-#define IMAGE_MAX_WIDTH                2048
-#define IMAGE_MAX_HEIGHT       2046 /* 2 * 1023 */
-/* on 830 and 845 these large limits result in the card hanging */
-#define IMAGE_MAX_WIDTH_LEGACY 1024
-#define IMAGE_MAX_HEIGHT_LEGACY        1088
-
-/* overlay register definitions */
-/* OCMD register */
-#define OCMD_TILED_SURFACE     (0x1<<19)
-#define OCMD_MIRROR_MASK       (0x3<<17)
-#define OCMD_MIRROR_MODE       (0x3<<17)
-#define OCMD_MIRROR_HORIZONTAL (0x1<<17)
-#define OCMD_MIRROR_VERTICAL   (0x2<<17)
-#define OCMD_MIRROR_BOTH       (0x3<<17)
-#define OCMD_BYTEORDER_MASK    (0x3<<14) /* zero for YUYV or FOURCC YUY2 */
-#define OCMD_UV_SWAP           (0x1<<14) /* YVYU */
-#define OCMD_Y_SWAP            (0x2<<14) /* UYVY or FOURCC UYVY */
-#define OCMD_Y_AND_UV_SWAP     (0x3<<14) /* VYUY */
-#define OCMD_SOURCE_FORMAT_MASK (0xf<<10)
-#define OCMD_RGB_888           (0x1<<10) /* not in i965 Intel docs */
-#define OCMD_RGB_555           (0x2<<10) /* not in i965 Intel docs */
-#define OCMD_RGB_565           (0x3<<10) /* not in i965 Intel docs */
-#define OCMD_YUV_422_PACKED    (0x8<<10)
-#define OCMD_YUV_411_PACKED    (0x9<<10) /* not in i965 Intel docs */
-#define OCMD_YUV_420_PLANAR    (0xc<<10)
-#define OCMD_YUV_422_PLANAR    (0xd<<10)
-#define OCMD_YUV_410_PLANAR    (0xe<<10) /* also 411 */
-#define OCMD_TVSYNCFLIP_PARITY (0x1<<9)
-#define OCMD_TVSYNCFLIP_ENABLE (0x1<<7)
-#define OCMD_BUF_TYPE_MASK     (0x1<<5)
-#define OCMD_BUF_TYPE_FRAME    (0x0<<5)
-#define OCMD_BUF_TYPE_FIELD    (0x1<<5)
-#define OCMD_TEST_MODE         (0x1<<4)
-#define OCMD_BUFFER_SELECT     (0x3<<2)
-#define OCMD_BUFFER0           (0x0<<2)
-#define OCMD_BUFFER1           (0x1<<2)
-#define OCMD_FIELD_SELECT      (0x1<<2)
-#define OCMD_FIELD0            (0x0<<1)
-#define OCMD_FIELD1            (0x1<<1)
-#define OCMD_ENABLE            (0x1<<0)
-
-/* OCONFIG register */
-#define OCONF_PIPE_MASK                (0x1<<18)
-#define OCONF_PIPE_A           (0x0<<18)
-#define OCONF_PIPE_B           (0x1<<18)
-#define OCONF_GAMMA2_ENABLE    (0x1<<16)
-#define OCONF_CSC_MODE_BT601   (0x0<<5)
-#define OCONF_CSC_MODE_BT709   (0x1<<5)
-#define OCONF_CSC_BYPASS       (0x1<<4)
-#define OCONF_CC_OUT_8BIT      (0x1<<3)
-#define OCONF_TEST_MODE                (0x1<<2)
-#define OCONF_THREE_LINE_BUFFER        (0x1<<0)
-#define OCONF_TWO_LINE_BUFFER  (0x0<<0)
-
-/* DCLRKM (dst-key) register */
-#define DST_KEY_ENABLE         (0x1<<31)
-#define CLK_RGB24_MASK         0x0
-#define CLK_RGB16_MASK         0x070307
-#define CLK_RGB15_MASK         0x070707
-
-#define RGB30_TO_COLORKEY(c) \
-       ((((c) & 0x3fc00000) >> 6) | (((c) & 0x000ff000) >> 4) | (((c) & 
0x000003fc) >> 2))
-#define RGB16_TO_COLORKEY(c) \
-       ((((c) & 0xf800) << 8) | (((c) & 0x07e0) << 5) | (((c) & 0x001f) << 3))
-#define RGB15_TO_COLORKEY(c) \
-       ((((c) & 0x7c00) << 9) | (((c) & 0x03e0) << 6) | (((c) & 0x001f) << 3))
-#define RGB8I_TO_COLORKEY(c) \
-       ((((c) & 0xff) << 16) | (((c) & 0xff) << 8) | (((c) & 0xff) << 0))
-
-/* overlay flip addr flag */
-#define OFC_UPDATE             0x1
-
-/* polyphase filter coefficients */
-#define N_HORIZ_Y_TAPS          5
-#define N_VERT_Y_TAPS           3
-#define N_HORIZ_UV_TAPS         3
-#define N_VERT_UV_TAPS          3
-#define N_PHASES                17
-#define MAX_TAPS                5
-
-/* memory bufferd overlay registers */
-struct overlay_registers {
-       u32 OBUF_0Y;
-       u32 OBUF_1Y;
-       u32 OBUF_0U;
-       u32 OBUF_0V;
-       u32 OBUF_1U;
-       u32 OBUF_1V;
-       u32 OSTRIDE;
-       u32 YRGB_VPH;
-       u32 UV_VPH;
-       u32 HORZ_PH;
-       u32 INIT_PHS;
-       u32 DWINPOS;
-       u32 DWINSZ;
-       u32 SWIDTH;
-       u32 SWIDTHSW;
-       u32 SHEIGHT;
-       u32 YRGBSCALE;
-       u32 UVSCALE;
-       u32 OCLRC0;
-       u32 OCLRC1;
-       u32 DCLRKV;
-       u32 DCLRKM;
-       u32 SCLRKVH;
-       u32 SCLRKVL;
-       u32 SCLRKEN;
-       u32 OCONFIG;
-       u32 OCMD;
-       u32 RESERVED1; /* 0x6C */
-       u32 OSTART_0Y;
-       u32 OSTART_1Y;
-       u32 OSTART_0U;
-       u32 OSTART_0V;
-       u32 OSTART_1U;
-       u32 OSTART_1V;
-       u32 OTILEOFF_0Y;
-       u32 OTILEOFF_1Y;
-       u32 OTILEOFF_0U;
-       u32 OTILEOFF_0V;
-       u32 OTILEOFF_1U;
-       u32 OTILEOFF_1V;
-       u32 FASTHSCALE; /* 0xA0 */
-       u32 UVSCALEV; /* 0xA4 */
-       u32 RESERVEDC[(0x200 - 0xA8) / 4]; /* 0xA8 - 0x1FC */
-       u16 Y_VCOEFS[N_VERT_Y_TAPS * N_PHASES]; /* 0x200 */
-       u16 RESERVEDD[0x100 / 2 - N_VERT_Y_TAPS * N_PHASES];
-       u16 Y_HCOEFS[N_HORIZ_Y_TAPS * N_PHASES]; /* 0x300 */
-       u16 RESERVEDE[0x200 / 2 - N_HORIZ_Y_TAPS * N_PHASES];
-       u16 UV_VCOEFS[N_VERT_UV_TAPS * N_PHASES]; /* 0x500 */
-       u16 RESERVEDF[0x100 / 2 - N_VERT_UV_TAPS * N_PHASES];
-       u16 UV_HCOEFS[N_HORIZ_UV_TAPS * N_PHASES]; /* 0x600 */
-       u16 RESERVEDG[0x100 / 2 - N_HORIZ_UV_TAPS * N_PHASES];
-};
-
-struct intel_overlay {
-       struct intel_display *display;
-       struct intel_context *context;
-       struct intel_crtc *crtc;
-       struct i915_vma *vma;
-       struct i915_vma *old_vma;
-       struct intel_frontbuffer *frontbuffer;
-       bool active;
-       bool pfit_active;
-       u32 pfit_vscale_ratio; /* shifted-point number, (1<<12) == 1.0 */
-       u32 color_key:24;
-       u32 color_key_enabled:1;
-       u32 brightness, contrast, saturation;
-       u32 old_xscale, old_yscale;
-       /* register access */
-       struct drm_i915_gem_object *reg_bo;
-       struct overlay_registers __iomem *regs;
-       u32 flip_addr;
-       /* flip handling */
-       struct i915_active last_flip;
-       void (*flip_complete)(struct intel_overlay *ovl);
-};
-
-static void i830_overlay_clock_gating(struct intel_display *display,
-                                     bool enable)
-{
-       struct pci_dev *pdev = to_pci_dev(display->drm->dev);
-       u8 val;
-
-       /* WA_OVERLAY_CLKGATE:alm */
-       if (enable)
-               intel_de_write(display, DSPCLK_GATE_D, 0);
-       else
-               intel_de_write(display, DSPCLK_GATE_D, 
OVRUNIT_CLOCK_GATE_DISABLE);
-
-       /* WA_DISABLE_L2CACHE_CLOCK_GATING:alm */
-       pci_bus_read_config_byte(pdev->bus,
-                                PCI_DEVFN(0, 0), I830_CLOCK_GATE, &val);
-       if (enable)
-               val &= ~I830_L2_CACHE_CLOCK_GATE_DISABLE;
-       else
-               val |= I830_L2_CACHE_CLOCK_GATE_DISABLE;
-       pci_bus_write_config_byte(pdev->bus,
-                                 PCI_DEVFN(0, 0), I830_CLOCK_GATE, val);
-}
-
-static struct i915_request *
-alloc_request(struct intel_overlay *overlay, void (*fn)(struct intel_overlay 
*))
-{
-       struct i915_request *rq;
-       int err;
-
-       overlay->flip_complete = fn;
-
-       rq = i915_request_create(overlay->context);
-       if (IS_ERR(rq))
-               return rq;
-
-       err = i915_active_add_request(&overlay->last_flip, rq);
-       if (err) {
-               i915_request_add(rq);
-               return ERR_PTR(err);
-       }
-
-       return rq;
-}
-
-/* overlay needs to be disable in OCMD reg */
-static int intel_overlay_on(struct intel_overlay *overlay)
-{
-       struct intel_display *display = overlay->display;
-       struct i915_request *rq;
-       u32 *cs;
-
-       drm_WARN_ON(display->drm, overlay->active);
-
-       rq = alloc_request(overlay, NULL);
-       if (IS_ERR(rq))
-               return PTR_ERR(rq);
-
-       cs = intel_ring_begin(rq, 4);
-       if (IS_ERR(cs)) {
-               i915_request_add(rq);
-               return PTR_ERR(cs);
-       }
-
-       overlay->active = true;
-
-       if (display->platform.i830)
-               i830_overlay_clock_gating(display, false);
-
-       *cs++ = MI_OVERLAY_FLIP | MI_OVERLAY_ON;
-       *cs++ = overlay->flip_addr | OFC_UPDATE;
-       *cs++ = MI_WAIT_FOR_EVENT | MI_WAIT_FOR_OVERLAY_FLIP;
-       *cs++ = MI_NOOP;
-       intel_ring_advance(rq, cs);
-
-       i915_request_add(rq);
-
-       return i915_active_wait(&overlay->last_flip);
-}
-
-static void intel_overlay_flip_prepare(struct intel_overlay *overlay,
-                                      struct i915_vma *vma)
-{
-       struct intel_display *display = overlay->display;
-       enum pipe pipe = overlay->crtc->pipe;
-       struct intel_frontbuffer *frontbuffer = NULL;
-
-       drm_WARN_ON(display->drm, overlay->old_vma);
-
-       if (vma)
-               frontbuffer = 
intel_frontbuffer_get(intel_bo_to_drm_bo(vma->obj));
-
-       intel_frontbuffer_track(overlay->frontbuffer, frontbuffer,
-                               INTEL_FRONTBUFFER_OVERLAY(pipe));
-
-       if (overlay->frontbuffer)
-               intel_frontbuffer_put(overlay->frontbuffer);
-       overlay->frontbuffer = frontbuffer;
-
-       overlay->old_vma = overlay->vma;
-       if (vma)
-               overlay->vma = i915_vma_get(vma);
-       else
-               overlay->vma = NULL;
-}
-
-/* overlay needs to be enabled in OCMD reg */
-static int intel_overlay_continue(struct intel_overlay *overlay,
-                                 struct i915_vma *vma,
-                                 bool load_polyphase_filter)
-{
-       struct intel_display *display = overlay->display;
-       struct i915_request *rq;
-       u32 flip_addr = overlay->flip_addr;
-       u32 tmp, *cs;
-
-       drm_WARN_ON(display->drm, !overlay->active);
-
-       if (load_polyphase_filter)
-               flip_addr |= OFC_UPDATE;
-
-       /* check for underruns */
-       tmp = intel_de_read(display, DOVSTA);
-       if (tmp & (1 << 17))
-               drm_dbg(display->drm, "overlay underrun, DOVSTA: %x\n", tmp);
-
-       rq = alloc_request(overlay, NULL);
-       if (IS_ERR(rq))
-               return PTR_ERR(rq);
-
-       cs = intel_ring_begin(rq, 2);
-       if (IS_ERR(cs)) {
-               i915_request_add(rq);
-               return PTR_ERR(cs);
-       }
-
-       *cs++ = MI_OVERLAY_FLIP | MI_OVERLAY_CONTINUE;
-       *cs++ = flip_addr;
-       intel_ring_advance(rq, cs);
-
-       intel_overlay_flip_prepare(overlay, vma);
-       i915_request_add(rq);
-
-       return 0;
-}
-
-static void intel_overlay_release_old_vma(struct intel_overlay *overlay)
-{
-       struct intel_display *display = overlay->display;
-       struct i915_vma *vma;
-
-       vma = fetch_and_zero(&overlay->old_vma);
-       if (drm_WARN_ON(display->drm, !vma))
-               return;
-
-       intel_frontbuffer_flip(display, 
INTEL_FRONTBUFFER_OVERLAY(overlay->crtc->pipe));
-
-       i915_vma_unpin(vma);
-       i915_vma_put(vma);
-}
-
-static void
-intel_overlay_release_old_vid_tail(struct intel_overlay *overlay)
-{
-       intel_overlay_release_old_vma(overlay);
-}
-
-static void intel_overlay_off_tail(struct intel_overlay *overlay)
-{
-       struct intel_display *display = overlay->display;
-
-       intel_overlay_release_old_vma(overlay);
-
-       overlay->crtc->overlay = NULL;
-       overlay->crtc = NULL;
-       overlay->active = false;
-
-       if (display->platform.i830)
-               i830_overlay_clock_gating(display, true);
-}
-
-static void intel_overlay_last_flip_retire(struct i915_active *active)
-{
-       struct intel_overlay *overlay =
-               container_of(active, typeof(*overlay), last_flip);
-
-       if (overlay->flip_complete)
-               overlay->flip_complete(overlay);
-}
-
-/* overlay needs to be disabled in OCMD reg */
-static int intel_overlay_off(struct intel_overlay *overlay)
-{
-       struct intel_display *display = overlay->display;
-       struct i915_request *rq;
-       u32 *cs, flip_addr = overlay->flip_addr;
-
-       drm_WARN_ON(display->drm, !overlay->active);
-
-       /*
-        * According to intel docs the overlay hw may hang (when switching
-        * off) without loading the filter coeffs. It is however unclear whether
-        * this applies to the disabling of the overlay or to the switching off
-        * of the hw. Do it in both cases.
-        */
-       flip_addr |= OFC_UPDATE;
-
-       rq = alloc_request(overlay, intel_overlay_off_tail);
-       if (IS_ERR(rq))
-               return PTR_ERR(rq);
-
-       cs = intel_ring_begin(rq, 6);
-       if (IS_ERR(cs)) {
-               i915_request_add(rq);
-               return PTR_ERR(cs);
-       }
-
-       /* wait for overlay to go idle */
-       *cs++ = MI_OVERLAY_FLIP | MI_OVERLAY_CONTINUE;
-       *cs++ = flip_addr;
-       *cs++ = MI_WAIT_FOR_EVENT | MI_WAIT_FOR_OVERLAY_FLIP;
-
-       /* turn overlay off */
-       *cs++ = MI_OVERLAY_FLIP | MI_OVERLAY_OFF;
-       *cs++ = flip_addr;
-       *cs++ = MI_WAIT_FOR_EVENT | MI_WAIT_FOR_OVERLAY_FLIP;
-
-       intel_ring_advance(rq, cs);
-
-       intel_overlay_flip_prepare(overlay, NULL);
-       i915_request_add(rq);
-
-       return i915_active_wait(&overlay->last_flip);
-}
-
-/*
- * Recover from an interruption due to a signal.
- * We have to be careful not to repeat work forever an make forward progress.
- */
-static int intel_overlay_recover_from_interrupt(struct intel_overlay *overlay)
-{
-       return i915_active_wait(&overlay->last_flip);
-}
-
-/*
- * Wait for pending overlay flip and release old frame.
- * Needs to be called before the overlay register are changed
- * via intel_overlay_(un)map_regs.
- */
-static int intel_overlay_release_old_vid(struct intel_overlay *overlay)
-{
-       struct intel_display *display = overlay->display;
-       struct i915_request *rq;
-       u32 *cs;
-
-       /*
-        * Only wait if there is actually an old frame to release to
-        * guarantee forward progress.
-        */
-       if (!overlay->old_vma)
-               return 0;
-
-       if (!(intel_de_read(display, GEN2_ISR) & 
I915_OVERLAY_PLANE_FLIP_PENDING_INTERRUPT)) {
-               intel_overlay_release_old_vid_tail(overlay);
-               return 0;
-       }
-
-       rq = alloc_request(overlay, intel_overlay_release_old_vid_tail);
-       if (IS_ERR(rq))
-               return PTR_ERR(rq);
-
-       cs = intel_ring_begin(rq, 2);
-       if (IS_ERR(cs)) {
-               i915_request_add(rq);
-               return PTR_ERR(cs);
-       }
-
-       *cs++ = MI_WAIT_FOR_EVENT | MI_WAIT_FOR_OVERLAY_FLIP;
-       *cs++ = MI_NOOP;
-       intel_ring_advance(rq, cs);
-
-       i915_request_add(rq);
-
-       return i915_active_wait(&overlay->last_flip);
-}
-
-void intel_overlay_reset(struct intel_display *display)
-{
-       struct intel_overlay *overlay = display->overlay;
-
-       if (!overlay)
-               return;
-
-       overlay->old_xscale = 0;
-       overlay->old_yscale = 0;
-       overlay->crtc = NULL;
-       overlay->active = false;
-}
-
-static int packed_depth_bytes(u32 format)
-{
-       switch (format & I915_OVERLAY_DEPTH_MASK) {
-       case I915_OVERLAY_YUV422:
-               return 4;
-       case I915_OVERLAY_YUV411:
-               /* return 6; not implemented */
-       default:
-               return -EINVAL;
-       }
-}
-
-static int packed_width_bytes(u32 format, short width)
-{
-       switch (format & I915_OVERLAY_DEPTH_MASK) {
-       case I915_OVERLAY_YUV422:
-               return width << 1;
-       default:
-               return -EINVAL;
-       }
-}
-
-static int uv_hsubsampling(u32 format)
-{
-       switch (format & I915_OVERLAY_DEPTH_MASK) {
-       case I915_OVERLAY_YUV422:
-       case I915_OVERLAY_YUV420:
-               return 2;
-       case I915_OVERLAY_YUV411:
-       case I915_OVERLAY_YUV410:
-               return 4;
-       default:
-               return -EINVAL;
-       }
-}
-
-static int uv_vsubsampling(u32 format)
-{
-       switch (format & I915_OVERLAY_DEPTH_MASK) {
-       case I915_OVERLAY_YUV420:
-       case I915_OVERLAY_YUV410:
-               return 2;
-       case I915_OVERLAY_YUV422:
-       case I915_OVERLAY_YUV411:
-               return 1;
-       default:
-               return -EINVAL;
-       }
-}
-
-static u32 calc_swidthsw(struct intel_display *display, u32 offset, u32 width)
-{
-       u32 sw;
-
-       if (DISPLAY_VER(display) == 2)
-               sw = ALIGN((offset & 31) + width, 32);
-       else
-               sw = ALIGN((offset & 63) + width, 64);
-
-       if (sw == 0)
-               return 0;
-
-       return (sw - 32) >> 3;
-}
-
-static const u16 y_static_hcoeffs[N_PHASES][N_HORIZ_Y_TAPS] = {
-       [ 0] = { 0x3000, 0xb4a0, 0x1930, 0x1920, 0xb4a0, },
-       [ 1] = { 0x3000, 0xb500, 0x19d0, 0x1880, 0xb440, },
-       [ 2] = { 0x3000, 0xb540, 0x1a88, 0x2f80, 0xb3e0, },
-       [ 3] = { 0x3000, 0xb580, 0x1b30, 0x2e20, 0xb380, },
-       [ 4] = { 0x3000, 0xb5c0, 0x1bd8, 0x2cc0, 0xb320, },
-       [ 5] = { 0x3020, 0xb5e0, 0x1c60, 0x2b80, 0xb2c0, },
-       [ 6] = { 0x3020, 0xb5e0, 0x1cf8, 0x2a20, 0xb260, },
-       [ 7] = { 0x3020, 0xb5e0, 0x1d80, 0x28e0, 0xb200, },
-       [ 8] = { 0x3020, 0xb5c0, 0x1e08, 0x3f40, 0xb1c0, },
-       [ 9] = { 0x3020, 0xb580, 0x1e78, 0x3ce0, 0xb160, },
-       [10] = { 0x3040, 0xb520, 0x1ed8, 0x3aa0, 0xb120, },
-       [11] = { 0x3040, 0xb4a0, 0x1f30, 0x3880, 0xb0e0, },
-       [12] = { 0x3040, 0xb400, 0x1f78, 0x3680, 0xb0a0, },
-       [13] = { 0x3020, 0xb340, 0x1fb8, 0x34a0, 0xb060, },
-       [14] = { 0x3020, 0xb240, 0x1fe0, 0x32e0, 0xb040, },
-       [15] = { 0x3020, 0xb140, 0x1ff8, 0x3160, 0xb020, },
-       [16] = { 0xb000, 0x3000, 0x0800, 0x3000, 0xb000, },
-};
-
-static const u16 uv_static_hcoeffs[N_PHASES][N_HORIZ_UV_TAPS] = {
-       [ 0] = { 0x3000, 0x1800, 0x1800, },
-       [ 1] = { 0xb000, 0x18d0, 0x2e60, },
-       [ 2] = { 0xb000, 0x1990, 0x2ce0, },
-       [ 3] = { 0xb020, 0x1a68, 0x2b40, },
-       [ 4] = { 0xb040, 0x1b20, 0x29e0, },
-       [ 5] = { 0xb060, 0x1bd8, 0x2880, },
-       [ 6] = { 0xb080, 0x1c88, 0x3e60, },
-       [ 7] = { 0xb0a0, 0x1d28, 0x3c00, },
-       [ 8] = { 0xb0c0, 0x1db8, 0x39e0, },
-       [ 9] = { 0xb0e0, 0x1e40, 0x37e0, },
-       [10] = { 0xb100, 0x1eb8, 0x3620, },
-       [11] = { 0xb100, 0x1f18, 0x34a0, },
-       [12] = { 0xb100, 0x1f68, 0x3360, },
-       [13] = { 0xb0e0, 0x1fa8, 0x3240, },
-       [14] = { 0xb0c0, 0x1fe0, 0x3140, },
-       [15] = { 0xb060, 0x1ff0, 0x30a0, },
-       [16] = { 0x3000, 0x0800, 0x3000, },
-};
-
-static void update_polyphase_filter(struct overlay_registers __iomem *regs)
-{
-       memcpy_toio(regs->Y_HCOEFS, y_static_hcoeffs, sizeof(y_static_hcoeffs));
-       memcpy_toio(regs->UV_HCOEFS, uv_static_hcoeffs,
-                   sizeof(uv_static_hcoeffs));
-}
-
-static bool update_scaling_factors(struct intel_overlay *overlay,
-                                  struct overlay_registers __iomem *regs,
-                                  struct drm_intel_overlay_put_image *params)
-{
-       /* fixed point with a 12 bit shift */
-       u32 xscale, yscale, xscale_UV, yscale_UV;
-#define FP_SHIFT 12
-#define FRACT_MASK 0xfff
-       bool scale_changed = false;
-       int uv_hscale = uv_hsubsampling(params->flags);
-       int uv_vscale = uv_vsubsampling(params->flags);
-
-       if (params->dst_width > 1)
-               xscale = ((params->src_scan_width - 1) << FP_SHIFT) /
-                       params->dst_width;
-       else
-               xscale = 1 << FP_SHIFT;
-
-       if (params->dst_height > 1)
-               yscale = ((params->src_scan_height - 1) << FP_SHIFT) /
-                       params->dst_height;
-       else
-               yscale = 1 << FP_SHIFT;
-
-       /*if (params->format & I915_OVERLAY_YUV_PLANAR) {*/
-       xscale_UV = xscale/uv_hscale;
-       yscale_UV = yscale/uv_vscale;
-       /* make the Y scale to UV scale ratio an exact multiply */
-       xscale = xscale_UV * uv_hscale;
-       yscale = yscale_UV * uv_vscale;
-       /*} else {
-         xscale_UV = 0;
-         yscale_UV = 0;
-         }*/
-
-       if (xscale != overlay->old_xscale || yscale != overlay->old_yscale)
-               scale_changed = true;
-       overlay->old_xscale = xscale;
-       overlay->old_yscale = yscale;
-
-       iowrite32(((yscale & FRACT_MASK) << 20) |
-                 ((xscale >> FP_SHIFT)  << 16) |
-                 ((xscale & FRACT_MASK) << 3),
-                &regs->YRGBSCALE);
-
-       iowrite32(((yscale_UV & FRACT_MASK) << 20) |
-                 ((xscale_UV >> FP_SHIFT)  << 16) |
-                 ((xscale_UV & FRACT_MASK) << 3),
-                &regs->UVSCALE);
-
-       iowrite32((((yscale    >> FP_SHIFT) << 16) |
-                  ((yscale_UV >> FP_SHIFT) << 0)),
-                &regs->UVSCALEV);
-
-       if (scale_changed)
-               update_polyphase_filter(regs);
-
-       return scale_changed;
-}
-
-static void update_colorkey(struct intel_overlay *overlay,
-                           struct overlay_registers __iomem *regs)
-{
-       const struct intel_plane_state *state =
-               to_intel_plane_state(overlay->crtc->base.primary->state);
-       u32 key = overlay->color_key;
-       u32 format = 0;
-       u32 flags = 0;
-
-       if (overlay->color_key_enabled)
-               flags |= DST_KEY_ENABLE;
-
-       if (state->uapi.visible)
-               format = state->hw.fb->format->format;
-
-       switch (format) {
-       case DRM_FORMAT_C8:
-               key = RGB8I_TO_COLORKEY(key);
-               flags |= CLK_RGB24_MASK;
-               break;
-       case DRM_FORMAT_XRGB1555:
-               key = RGB15_TO_COLORKEY(key);
-               flags |= CLK_RGB15_MASK;
-               break;
-       case DRM_FORMAT_RGB565:
-               key = RGB16_TO_COLORKEY(key);
-               flags |= CLK_RGB16_MASK;
-               break;
-       case DRM_FORMAT_XRGB2101010:
-       case DRM_FORMAT_XBGR2101010:
-               key = RGB30_TO_COLORKEY(key);
-               flags |= CLK_RGB24_MASK;
-               break;
-       default:
-               flags |= CLK_RGB24_MASK;
-               break;
-       }
-
-       iowrite32(key, &regs->DCLRKV);
-       iowrite32(flags, &regs->DCLRKM);
-}
-
-static u32 overlay_cmd_reg(struct drm_intel_overlay_put_image *params)
-{
-       u32 cmd = OCMD_ENABLE | OCMD_BUF_TYPE_FRAME | OCMD_BUFFER0;
-
-       if (params->flags & I915_OVERLAY_YUV_PLANAR) {
-               switch (params->flags & I915_OVERLAY_DEPTH_MASK) {
-               case I915_OVERLAY_YUV422:
-                       cmd |= OCMD_YUV_422_PLANAR;
-                       break;
-               case I915_OVERLAY_YUV420:
-                       cmd |= OCMD_YUV_420_PLANAR;
-                       break;
-               case I915_OVERLAY_YUV411:
-               case I915_OVERLAY_YUV410:
-                       cmd |= OCMD_YUV_410_PLANAR;
-                       break;
-               }
-       } else { /* YUV packed */
-               switch (params->flags & I915_OVERLAY_DEPTH_MASK) {
-               case I915_OVERLAY_YUV422:
-                       cmd |= OCMD_YUV_422_PACKED;
-                       break;
-               case I915_OVERLAY_YUV411:
-                       cmd |= OCMD_YUV_411_PACKED;
-                       break;
-               }
-
-               switch (params->flags & I915_OVERLAY_SWAP_MASK) {
-               case I915_OVERLAY_NO_SWAP:
-                       break;
-               case I915_OVERLAY_UV_SWAP:
-                       cmd |= OCMD_UV_SWAP;
-                       break;
-               case I915_OVERLAY_Y_SWAP:
-                       cmd |= OCMD_Y_SWAP;
-                       break;
-               case I915_OVERLAY_Y_AND_UV_SWAP:
-                       cmd |= OCMD_Y_AND_UV_SWAP;
-                       break;
-               }
-       }
-
-       return cmd;
-}
-
-static struct i915_vma *intel_overlay_pin_fb(struct drm_i915_gem_object 
*new_bo)
-{
-       struct i915_gem_ww_ctx ww;
-       struct i915_vma *vma;
-       int ret;
-
-       i915_gem_ww_ctx_init(&ww, true);
-retry:
-       ret = i915_gem_object_lock(new_bo, &ww);
-       if (!ret) {
-               vma = i915_gem_object_pin_to_display_plane(new_bo, &ww, 0, 0,
-                                                          NULL, PIN_MAPPABLE);
-               ret = PTR_ERR_OR_ZERO(vma);
-       }
-       if (ret == -EDEADLK) {
-               ret = i915_gem_ww_ctx_backoff(&ww);
-               if (!ret)
-                       goto retry;
-       }
-       i915_gem_ww_ctx_fini(&ww);
-       if (ret)
-               return ERR_PTR(ret);
-
-       return vma;
-}
-
-static int intel_overlay_do_put_image(struct intel_overlay *overlay,
-                                     struct drm_i915_gem_object *new_bo,
-                                     struct drm_intel_overlay_put_image 
*params)
-{
-       struct intel_display *display = overlay->display;
-       struct overlay_registers __iomem *regs = overlay->regs;
-       u32 swidth, swidthsw, sheight, ostride;
-       enum pipe pipe = overlay->crtc->pipe;
-       bool scale_changed = false;
-       struct i915_vma *vma;
-       int ret, tmp_width;
-
-       drm_WARN_ON(display->drm,
-                   
!drm_modeset_is_locked(&display->drm->mode_config.connection_mutex));
-
-       ret = intel_overlay_release_old_vid(overlay);
-       if (ret != 0)
-               return ret;
-
-       atomic_inc(&display->restore.pending_fb_pin);
-
-       vma = intel_overlay_pin_fb(new_bo);
-       if (IS_ERR(vma)) {
-               ret = PTR_ERR(vma);
-               goto out_pin_section;
-       }
-
-       if (!overlay->active) {
-               const struct intel_crtc_state *crtc_state =
-                       overlay->crtc->config;
-               u32 oconfig = 0;
-
-               if (crtc_state->gamma_enable &&
-                   crtc_state->gamma_mode == GAMMA_MODE_MODE_8BIT)
-                       oconfig |= OCONF_CC_OUT_8BIT;
-               if (crtc_state->gamma_enable)
-                       oconfig |= OCONF_GAMMA2_ENABLE;
-               if (DISPLAY_VER(display) == 4)
-                       oconfig |= OCONF_CSC_MODE_BT709;
-               oconfig |= pipe == 0 ?
-                       OCONF_PIPE_A : OCONF_PIPE_B;
-               iowrite32(oconfig, &regs->OCONFIG);
-
-               ret = intel_overlay_on(overlay);
-               if (ret != 0)
-                       goto out_unpin;
-       }
-
-       iowrite32(params->dst_y << 16 | params->dst_x, &regs->DWINPOS);
-       iowrite32(params->dst_height << 16 | params->dst_width, &regs->DWINSZ);
-
-       if (params->flags & I915_OVERLAY_YUV_PACKED)
-               tmp_width = packed_width_bytes(params->flags,
-                                              params->src_width);
-       else
-               tmp_width = params->src_width;
-
-       swidth = params->src_width;
-       swidthsw = calc_swidthsw(display, params->offset_Y, tmp_width);
-       sheight = params->src_height;
-       iowrite32(i915_ggtt_offset(vma) + params->offset_Y, &regs->OBUF_0Y);
-       ostride = params->stride_Y;
-
-       if (params->flags & I915_OVERLAY_YUV_PLANAR) {
-               int uv_hscale = uv_hsubsampling(params->flags);
-               int uv_vscale = uv_vsubsampling(params->flags);
-               u32 tmp_U, tmp_V;
-
-               swidth |= (params->src_width / uv_hscale) << 16;
-               sheight |= (params->src_height / uv_vscale) << 16;
-
-               tmp_U = calc_swidthsw(display, params->offset_U,
-                                     params->src_width / uv_hscale);
-               tmp_V = calc_swidthsw(display, params->offset_V,
-                                     params->src_width / uv_hscale);
-               swidthsw |= max(tmp_U, tmp_V) << 16;
-
-               iowrite32(i915_ggtt_offset(vma) + params->offset_U,
-                         &regs->OBUF_0U);
-               iowrite32(i915_ggtt_offset(vma) + params->offset_V,
-                         &regs->OBUF_0V);
-
-               ostride |= params->stride_UV << 16;
-       }
-
-       iowrite32(swidth, &regs->SWIDTH);
-       iowrite32(swidthsw, &regs->SWIDTHSW);
-       iowrite32(sheight, &regs->SHEIGHT);
-       iowrite32(ostride, &regs->OSTRIDE);
-
-       scale_changed = update_scaling_factors(overlay, regs, params);
-
-       update_colorkey(overlay, regs);
-
-       iowrite32(overlay_cmd_reg(params), &regs->OCMD);
-
-       ret = intel_overlay_continue(overlay, vma, scale_changed);
-       if (ret)
-               goto out_unpin;
-
-       return 0;
-
-out_unpin:
-       i915_vma_unpin(vma);
-out_pin_section:
-       atomic_dec(&display->restore.pending_fb_pin);
-
-       return ret;
-}
-
-int intel_overlay_switch_off(struct intel_overlay *overlay)
-{
-       struct intel_display *display = overlay->display;
-       int ret;
-
-       drm_WARN_ON(display->drm,
-                   
!drm_modeset_is_locked(&display->drm->mode_config.connection_mutex));
-
-       ret = intel_overlay_recover_from_interrupt(overlay);
-       if (ret != 0)
-               return ret;
-
-       if (!overlay->active)
-               return 0;
-
-       ret = intel_overlay_release_old_vid(overlay);
-       if (ret != 0)
-               return ret;
-
-       iowrite32(0, &overlay->regs->OCMD);
-
-       return intel_overlay_off(overlay);
-}
-
-static int check_overlay_possible_on_crtc(struct intel_overlay *overlay,
-                                         struct intel_crtc *crtc)
-{
-       if (!crtc->active)
-               return -EINVAL;
-
-       /* can't use the overlay with double wide pipe */
-       if (crtc->config->double_wide)
-               return -EINVAL;
-
-       return 0;
-}
-
-static void update_pfit_vscale_ratio(struct intel_overlay *overlay)
-{
-       struct intel_display *display = overlay->display;
-       u32 ratio;
-
-       /* XXX: This is not the same logic as in the xorg driver, but more in
-        * line with the intel documentation for the i965
-        */
-       if (DISPLAY_VER(display) >= 4) {
-               u32 tmp = intel_de_read(display, PFIT_PGM_RATIOS(display));
-
-               /* on i965 use the PGM reg to read out the autoscaler values */
-               ratio = REG_FIELD_GET(PFIT_VERT_SCALE_MASK_965, tmp);
-       } else {
-               u32 tmp;
-
-               if (intel_de_read(display, PFIT_CONTROL(display)) & 
PFIT_VERT_AUTO_SCALE)
-                       tmp = intel_de_read(display, PFIT_AUTO_RATIOS(display));
-               else
-                       tmp = intel_de_read(display, PFIT_PGM_RATIOS(display));
-
-               ratio = REG_FIELD_GET(PFIT_VERT_SCALE_MASK, tmp);
-       }
-
-       overlay->pfit_vscale_ratio = ratio;
-}
-
-static int check_overlay_dst(struct intel_overlay *overlay,
-                            struct drm_intel_overlay_put_image *rec)
-{
-       const struct intel_crtc_state *crtc_state =
-               overlay->crtc->config;
-       struct drm_rect req, clipped;
-
-       drm_rect_init(&req, rec->dst_x, rec->dst_y,
-                     rec->dst_width, rec->dst_height);
-
-       clipped = req;
-
-       if (!drm_rect_intersect(&clipped, &crtc_state->pipe_src))
-               return -EINVAL;
-
-       if (!drm_rect_equals(&clipped, &req))
-               return -EINVAL;
-
-       return 0;
-}
-
-static int check_overlay_scaling(struct drm_intel_overlay_put_image *rec)
-{
-       u32 tmp;
-
-       /* downscaling limit is 8.0 */
-       tmp = ((rec->src_scan_height << 16) / rec->dst_height) >> 16;
-       if (tmp > 7)
-               return -EINVAL;
-
-       tmp = ((rec->src_scan_width << 16) / rec->dst_width) >> 16;
-       if (tmp > 7)
-               return -EINVAL;
-
-       return 0;
-}
-
-static int check_overlay_src(struct intel_display *display,
-                            struct drm_intel_overlay_put_image *rec,
-                            struct drm_i915_gem_object *new_bo)
-{
-       int uv_hscale = uv_hsubsampling(rec->flags);
-       int uv_vscale = uv_vsubsampling(rec->flags);
-       u32 stride_mask;
-       int depth;
-       u32 tmp;
-
-       /* check src dimensions */
-       if (display->platform.i845g || display->platform.i830) {
-               if (rec->src_height > IMAGE_MAX_HEIGHT_LEGACY ||
-                   rec->src_width  > IMAGE_MAX_WIDTH_LEGACY)
-                       return -EINVAL;
-       } else {
-               if (rec->src_height > IMAGE_MAX_HEIGHT ||
-                   rec->src_width  > IMAGE_MAX_WIDTH)
-                       return -EINVAL;
-       }
-
-       /* better safe than sorry, use 4 as the maximal subsampling ratio */
-       if (rec->src_height < N_VERT_Y_TAPS*4 ||
-           rec->src_width  < N_HORIZ_Y_TAPS*4)
-               return -EINVAL;
-
-       /* check alignment constraints */
-       switch (rec->flags & I915_OVERLAY_TYPE_MASK) {
-       case I915_OVERLAY_RGB:
-               /* not implemented */
-               return -EINVAL;
-
-       case I915_OVERLAY_YUV_PACKED:
-               if (uv_vscale != 1)
-                       return -EINVAL;
-
-               depth = packed_depth_bytes(rec->flags);
-               if (depth < 0)
-                       return depth;
-
-               /* ignore UV planes */
-               rec->stride_UV = 0;
-               rec->offset_U = 0;
-               rec->offset_V = 0;
-               /* check pixel alignment */
-               if (rec->offset_Y % depth)
-                       return -EINVAL;
-               break;
-
-       case I915_OVERLAY_YUV_PLANAR:
-               if (uv_vscale < 0 || uv_hscale < 0)
-                       return -EINVAL;
-               /* no offset restrictions for planar formats */
-               break;
-
-       default:
-               return -EINVAL;
-       }
-
-       if (rec->src_width % uv_hscale)
-               return -EINVAL;
-
-       /* stride checking */
-       if (display->platform.i830 || display->platform.i845g)
-               stride_mask = 255;
-       else
-               stride_mask = 63;
-
-       if (rec->stride_Y & stride_mask || rec->stride_UV & stride_mask)
-               return -EINVAL;
-       if (DISPLAY_VER(display) == 4 && rec->stride_Y < 512)
-               return -EINVAL;
-
-       tmp = (rec->flags & I915_OVERLAY_TYPE_MASK) == I915_OVERLAY_YUV_PLANAR ?
-               4096 : 8192;
-       if (rec->stride_Y > tmp || rec->stride_UV > 2*1024)
-               return -EINVAL;
-
-       /* check buffer dimensions */
-       switch (rec->flags & I915_OVERLAY_TYPE_MASK) {
-       case I915_OVERLAY_RGB:
-       case I915_OVERLAY_YUV_PACKED:
-               /* always 4 Y values per depth pixels */
-               if (packed_width_bytes(rec->flags, rec->src_width) > 
rec->stride_Y)
-                       return -EINVAL;
-
-               tmp = rec->stride_Y*rec->src_height;
-               if (rec->offset_Y + tmp > new_bo->base.size)
-                       return -EINVAL;
-               break;
-
-       case I915_OVERLAY_YUV_PLANAR:
-               if (rec->src_width > rec->stride_Y)
-                       return -EINVAL;
-               if (rec->src_width/uv_hscale > rec->stride_UV)
-                       return -EINVAL;
-
-               tmp = rec->stride_Y * rec->src_height;
-               if (rec->offset_Y + tmp > new_bo->base.size)
-                       return -EINVAL;
-
-               tmp = rec->stride_UV * (rec->src_height / uv_vscale);
-               if (rec->offset_U + tmp > new_bo->base.size ||
-                   rec->offset_V + tmp > new_bo->base.size)
-                       return -EINVAL;
-               break;
-       }
-
-       return 0;
-}
-
-int intel_overlay_put_image_ioctl(struct drm_device *dev, void *data,
-                                 struct drm_file *file_priv)
-{
-       struct intel_display *display = to_intel_display(dev);
-       struct drm_intel_overlay_put_image *params = data;
-       struct intel_overlay *overlay;
-       struct drm_crtc *drmmode_crtc;
-       struct intel_crtc *crtc;
-       struct drm_i915_gem_object *new_bo;
-       int ret;
-
-       overlay = display->overlay;
-       if (!overlay) {
-               drm_dbg(display->drm, "userspace bug: no overlay\n");
-               return -ENODEV;
-       }
-
-       if (!(params->flags & I915_OVERLAY_ENABLE)) {
-               drm_modeset_lock_all(dev);
-               ret = intel_overlay_switch_off(overlay);
-               drm_modeset_unlock_all(dev);
-
-               return ret;
-       }
-
-       drmmode_crtc = drm_crtc_find(dev, file_priv, params->crtc_id);
-       if (!drmmode_crtc)
-               return -ENOENT;
-       crtc = to_intel_crtc(drmmode_crtc);
-
-       new_bo = i915_gem_object_lookup(file_priv, params->bo_handle);
-       if (!new_bo)
-               return -ENOENT;
-
-       drm_modeset_lock_all(dev);
-
-       if (i915_gem_object_is_tiled(new_bo)) {
-               drm_dbg_kms(display->drm,
-                           "buffer used for overlay image can not be tiled\n");
-               ret = -EINVAL;
-               goto out_unlock;
-       }
-
-       ret = intel_overlay_recover_from_interrupt(overlay);
-       if (ret != 0)
-               goto out_unlock;
-
-       if (overlay->crtc != crtc) {
-               ret = intel_overlay_switch_off(overlay);
-               if (ret != 0)
-                       goto out_unlock;
-
-               ret = check_overlay_possible_on_crtc(overlay, crtc);
-               if (ret != 0)
-                       goto out_unlock;
-
-               overlay->crtc = crtc;
-               crtc->overlay = overlay;
-
-               /* line too wide, i.e. one-line-mode */
-               if (drm_rect_width(&crtc->config->pipe_src) > 1024 &&
-                   crtc->config->gmch_pfit.control & PFIT_ENABLE) {
-                       overlay->pfit_active = true;
-                       update_pfit_vscale_ratio(overlay);
-               } else
-                       overlay->pfit_active = false;
-       }
-
-       ret = check_overlay_dst(overlay, params);
-       if (ret != 0)
-               goto out_unlock;
-
-       if (overlay->pfit_active) {
-               params->dst_y = (((u32)params->dst_y << 12) /
-                                overlay->pfit_vscale_ratio);
-               /* shifting right rounds downwards, so add 1 */
-               params->dst_height = (((u32)params->dst_height << 12) /
-                                overlay->pfit_vscale_ratio) + 1;
-       }
-
-       if (params->src_scan_height > params->src_height ||
-           params->src_scan_width > params->src_width) {
-               ret = -EINVAL;
-               goto out_unlock;
-       }
-
-       ret = check_overlay_src(display, params, new_bo);
-       if (ret != 0)
-               goto out_unlock;
-
-       /* Check scaling after src size to prevent a divide-by-zero. */
-       ret = check_overlay_scaling(params);
-       if (ret != 0)
-               goto out_unlock;
-
-       ret = intel_overlay_do_put_image(overlay, new_bo, params);
-       if (ret != 0)
-               goto out_unlock;
-
-       drm_modeset_unlock_all(dev);
-       i915_gem_object_put(new_bo);
-
-       return 0;
-
-out_unlock:
-       drm_modeset_unlock_all(dev);
-       i915_gem_object_put(new_bo);
-
-       return ret;
-}
-
-static void update_reg_attrs(struct intel_overlay *overlay,
-                            struct overlay_registers __iomem *regs)
-{
-       iowrite32((overlay->contrast << 18) | (overlay->brightness & 0xff),
-                 &regs->OCLRC0);
-       iowrite32(overlay->saturation, &regs->OCLRC1);
-}
-
-static bool check_gamma_bounds(u32 gamma1, u32 gamma2)
-{
-       int i;
-
-       if (gamma1 & 0xff000000 || gamma2 & 0xff000000)
-               return false;
-
-       for (i = 0; i < 3; i++) {
-               if (((gamma1 >> i*8) & 0xff) >= ((gamma2 >> i*8) & 0xff))
-                       return false;
-       }
-
-       return true;
-}
-
-static bool check_gamma5_errata(u32 gamma5)
-{
-       int i;
-
-       for (i = 0; i < 3; i++) {
-               if (((gamma5 >> i*8) & 0xff) == 0x80)
-                       return false;
-       }
-
-       return true;
-}
-
-static int check_gamma(struct drm_intel_overlay_attrs *attrs)
-{
-       if (!check_gamma_bounds(0, attrs->gamma0) ||
-           !check_gamma_bounds(attrs->gamma0, attrs->gamma1) ||
-           !check_gamma_bounds(attrs->gamma1, attrs->gamma2) ||
-           !check_gamma_bounds(attrs->gamma2, attrs->gamma3) ||
-           !check_gamma_bounds(attrs->gamma3, attrs->gamma4) ||
-           !check_gamma_bounds(attrs->gamma4, attrs->gamma5) ||
-           !check_gamma_bounds(attrs->gamma5, 0x00ffffff))
-               return -EINVAL;
-
-       if (!check_gamma5_errata(attrs->gamma5))
-               return -EINVAL;
-
-       return 0;
-}
-
-int intel_overlay_attrs_ioctl(struct drm_device *dev, void *data,
-                             struct drm_file *file_priv)
-{
-       struct intel_display *display = to_intel_display(dev);
-       struct drm_intel_overlay_attrs *attrs = data;
-       struct intel_overlay *overlay;
-       int ret;
-
-       overlay = display->overlay;
-       if (!overlay) {
-               drm_dbg(display->drm, "userspace bug: no overlay\n");
-               return -ENODEV;
-       }
-
-       drm_modeset_lock_all(dev);
-
-       ret = -EINVAL;
-       if (!(attrs->flags & I915_OVERLAY_UPDATE_ATTRS)) {
-               attrs->color_key  = overlay->color_key;
-               attrs->brightness = overlay->brightness;
-               attrs->contrast   = overlay->contrast;
-               attrs->saturation = overlay->saturation;
-
-               if (DISPLAY_VER(display) != 2) {
-                       attrs->gamma0 = intel_de_read(display, OGAMC0);
-                       attrs->gamma1 = intel_de_read(display, OGAMC1);
-                       attrs->gamma2 = intel_de_read(display, OGAMC2);
-                       attrs->gamma3 = intel_de_read(display, OGAMC3);
-                       attrs->gamma4 = intel_de_read(display, OGAMC4);
-                       attrs->gamma5 = intel_de_read(display, OGAMC5);
-               }
-       } else {
-               if (attrs->brightness < -128 || attrs->brightness > 127)
-                       goto out_unlock;
-               if (attrs->contrast > 255)
-                       goto out_unlock;
-               if (attrs->saturation > 1023)
-                       goto out_unlock;
-
-               overlay->color_key  = attrs->color_key;
-               overlay->brightness = attrs->brightness;
-               overlay->contrast   = attrs->contrast;
-               overlay->saturation = attrs->saturation;
-
-               update_reg_attrs(overlay, overlay->regs);
-
-               if (attrs->flags & I915_OVERLAY_UPDATE_GAMMA) {
-                       if (DISPLAY_VER(display) == 2)
-                               goto out_unlock;
-
-                       if (overlay->active) {
-                               ret = -EBUSY;
-                               goto out_unlock;
-                       }
-
-                       ret = check_gamma(attrs);
-                       if (ret)
-                               goto out_unlock;
-
-                       intel_de_write(display, OGAMC0, attrs->gamma0);
-                       intel_de_write(display, OGAMC1, attrs->gamma1);
-                       intel_de_write(display, OGAMC2, attrs->gamma2);
-                       intel_de_write(display, OGAMC3, attrs->gamma3);
-                       intel_de_write(display, OGAMC4, attrs->gamma4);
-                       intel_de_write(display, OGAMC5, attrs->gamma5);
-               }
-       }
-       overlay->color_key_enabled = (attrs->flags & 
I915_OVERLAY_DISABLE_DEST_COLORKEY) == 0;
-
-       ret = 0;
-out_unlock:
-       drm_modeset_unlock_all(dev);
-
-       return ret;
-}
-
-static int get_registers(struct intel_overlay *overlay, bool use_phys)
-{
-       struct intel_display *display = overlay->display;
-       struct drm_i915_private *i915 = to_i915(display->drm);
-       struct drm_i915_gem_object *obj = ERR_PTR(-ENODEV);
-       struct i915_vma *vma;
-       int err;
-
-       if (!display->platform.meteorlake) /* Wa_22018444074 */
-               obj = i915_gem_object_create_stolen(i915, PAGE_SIZE);
-       if (IS_ERR(obj))
-               obj = i915_gem_object_create_internal(i915, PAGE_SIZE);
-       if (IS_ERR(obj))
-               return PTR_ERR(obj);
-
-       vma = i915_gem_object_ggtt_pin(obj, NULL, 0, 0, PIN_MAPPABLE);
-       if (IS_ERR(vma)) {
-               err = PTR_ERR(vma);
-               goto err_put_bo;
-       }
-
-       if (use_phys)
-               overlay->flip_addr = sg_dma_address(obj->mm.pages->sgl);
-       else
-               overlay->flip_addr = i915_ggtt_offset(vma);
-       overlay->regs = i915_vma_pin_iomap(vma);
-       i915_vma_unpin(vma);
-
-       if (IS_ERR(overlay->regs)) {
-               err = PTR_ERR(overlay->regs);
-               goto err_put_bo;
-       }
-
-       overlay->reg_bo = obj;
-       return 0;
-
-err_put_bo:
-       i915_gem_object_put(obj);
-       return err;
-}
-
-void intel_overlay_setup(struct intel_display *display)
-{
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-       struct intel_overlay *overlay;
-       struct intel_engine_cs *engine;
-       int ret;
-
-       if (!HAS_OVERLAY(display))
-               return;
-
-       engine = to_gt(dev_priv)->engine[RCS0];
-       if (!engine || !engine->kernel_context)
-               return;
-
-       overlay = kzalloc(sizeof(*overlay), GFP_KERNEL);
-       if (!overlay)
-               return;
-
-       overlay->display = display;
-       overlay->context = engine->kernel_context;
-       overlay->color_key = 0x0101fe;
-       overlay->color_key_enabled = true;
-       overlay->brightness = -19;
-       overlay->contrast = 75;
-       overlay->saturation = 146;
-
-       i915_active_init(&overlay->last_flip,
-                        NULL, intel_overlay_last_flip_retire, 0);
-
-       ret = get_registers(overlay, OVERLAY_NEEDS_PHYSICAL(display));
-       if (ret)
-               goto out_free;
-
-       memset_io(overlay->regs, 0, sizeof(struct overlay_registers));
-       update_polyphase_filter(overlay->regs);
-       update_reg_attrs(overlay, overlay->regs);
-
-       display->overlay = overlay;
-       drm_info(display->drm, "Initialized overlay support.\n");
-       return;
-
-out_free:
-       kfree(overlay);
-}
-
-bool intel_overlay_available(struct intel_display *display)
-{
-       return display->overlay;
-}
-
-void intel_overlay_cleanup(struct intel_display *display)
-{
-       struct intel_overlay *overlay;
-
-       overlay = fetch_and_zero(&display->overlay);
-       if (!overlay)
-               return;
-
-       /*
-        * The bo's should be free'd by the generic code already.
-        * Furthermore modesetting teardown happens beforehand so the
-        * hardware should be off already.
-        */
-       drm_WARN_ON(display->drm, overlay->active);
-
-       i915_gem_object_put(overlay->reg_bo);
-       i915_active_fini(&overlay->last_flip);
-
-       kfree(overlay);
-}
-
-#if IS_ENABLED(CONFIG_DRM_I915_CAPTURE_ERROR)
-
-struct intel_overlay_snapshot {
-       struct overlay_registers regs;
-       unsigned long base;
-       u32 dovsta;
-       u32 isr;
-};
-
-struct intel_overlay_snapshot *
-intel_overlay_snapshot_capture(struct intel_display *display)
-{
-       struct intel_overlay *overlay = display->overlay;
-       struct intel_overlay_snapshot *error;
-
-       if (!overlay || !overlay->active)
-               return NULL;
-
-       error = kmalloc(sizeof(*error), GFP_ATOMIC);
-       if (error == NULL)
-               return NULL;
-
-       error->dovsta = intel_de_read(display, DOVSTA);
-       error->isr = intel_de_read(display, GEN2_ISR);
-       error->base = overlay->flip_addr;
-
-       memcpy_fromio(&error->regs, overlay->regs, sizeof(error->regs));
-
-       return error;
-}
-
-void
-intel_overlay_snapshot_print(const struct intel_overlay_snapshot *error,
-                            struct drm_printer *p)
-{
-       if (!error)
-               return;
-
-       drm_printf(p, "Overlay, status: 0x%08x, interrupt: 0x%08x\n",
-                  error->dovsta, error->isr);
-       drm_printf(p, "  Register file at 0x%08lx:\n", error->base);
-
-#define P(x) drm_printf(p, "    " #x ": 0x%08x\n", error->regs.x)
-       P(OBUF_0Y);
-       P(OBUF_1Y);
-       P(OBUF_0U);
-       P(OBUF_0V);
-       P(OBUF_1U);
-       P(OBUF_1V);
-       P(OSTRIDE);
-       P(YRGB_VPH);
-       P(UV_VPH);
-       P(HORZ_PH);
-       P(INIT_PHS);
-       P(DWINPOS);
-       P(DWINSZ);
-       P(SWIDTH);
-       P(SWIDTHSW);
-       P(SHEIGHT);
-       P(YRGBSCALE);
-       P(UVSCALE);
-       P(OCLRC0);
-       P(OCLRC1);
-       P(DCLRKV);
-       P(DCLRKM);
-       P(SCLRKVH);
-       P(SCLRKVL);
-       P(SCLRKEN);
-       P(OCONFIG);
-       P(OCMD);
-       P(OSTART_0Y);
-       P(OSTART_1Y);
-       P(OSTART_0U);
-       P(OSTART_0V);
-       P(OSTART_1U);
-       P(OSTART_1V);
-       P(OTILEOFF_0Y);
-       P(OTILEOFF_1Y);
-       P(OTILEOFF_0U);
-       P(OTILEOFF_0V);
-       P(OTILEOFF_1U);
-       P(OTILEOFF_1V);
-       P(FASTHSCALE);
-       P(UVSCALEV);
-#undef P
-}
-
-#endif
diff --git a/drivers/gpu/drm/i915/display/intel_overlay.h 
b/drivers/gpu/drm/i915/display/intel_overlay.h
deleted file mode 100644
index d259e4c74b03..000000000000
--- a/drivers/gpu/drm/i915/display/intel_overlay.h
+++ /dev/null
@@ -1,75 +0,0 @@
-/* SPDX-License-Identifier: MIT */
-/*
- * Copyright © 2019 Intel Corporation
- */
-
-#ifndef __INTEL_OVERLAY_H__
-#define __INTEL_OVERLAY_H__
-
-#include <linux/types.h>
-
-struct drm_device;
-struct drm_file;
-struct drm_printer;
-struct intel_display;
-struct intel_overlay;
-struct intel_overlay_snapshot;
-
-#ifdef I915
-void intel_overlay_setup(struct intel_display *display);
-bool intel_overlay_available(struct intel_display *display);
-void intel_overlay_cleanup(struct intel_display *display);
-int intel_overlay_switch_off(struct intel_overlay *overlay);
-int intel_overlay_put_image_ioctl(struct drm_device *dev, void *data,
-                                 struct drm_file *file_priv);
-int intel_overlay_attrs_ioctl(struct drm_device *dev, void *data,
-                             struct drm_file *file_priv);
-void intel_overlay_reset(struct intel_display *display);
-#else
-static inline void intel_overlay_setup(struct intel_display *display)
-{
-}
-static inline bool intel_overlay_available(struct intel_display *display)
-{
-       return false;
-}
-static inline void intel_overlay_cleanup(struct intel_display *display)
-{
-}
-static inline int intel_overlay_switch_off(struct intel_overlay *overlay)
-{
-       return 0;
-}
-static inline int intel_overlay_put_image_ioctl(struct drm_device *dev, void 
*data,
-                                               struct drm_file *file_priv)
-{
-       return 0;
-}
-static inline int intel_overlay_attrs_ioctl(struct drm_device *dev, void *data,
-                                           struct drm_file *file_priv)
-{
-       return 0;
-}
-static inline void intel_overlay_reset(struct intel_display *display)
-{
-}
-#endif
-
-#if IS_ENABLED(CONFIG_DRM_I915_CAPTURE_ERROR) && defined(I915)
-struct intel_overlay_snapshot *
-intel_overlay_snapshot_capture(struct intel_display *display);
-void intel_overlay_snapshot_print(const struct intel_overlay_snapshot *error,
-                                 struct drm_printer *p);
-#else
-static inline struct intel_overlay_snapshot *
-intel_overlay_snapshot_capture(struct intel_display *display)
-{
-       return NULL;
-}
-static inline void intel_overlay_snapshot_print(const struct 
intel_overlay_snapshot *error,
-                                               struct drm_printer *p)
-{
-}
-#endif
-
-#endif /* __INTEL_OVERLAY_H__ */
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c 
b/drivers/gpu/drm/i915/gt/intel_reset.c
index 41b5036dc538..7dde1fbc3850 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.c
+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
@@ -8,7 +8,6 @@
 #include <linux/string_helpers.h>
 
 #include "display/intel_display_reset.h"
-#include "display/intel_overlay.h"
 #include "gem/i915_gem_context.h"
 #include "gt/intel_gt_regs.h"
 #include "gt/uc/intel_gsc_fw.h"
@@ -1202,7 +1201,6 @@ void intel_gt_reset(struct intel_gt *gt,
                    intel_engine_mask_t stalled_mask,
                    const char *reason)
 {
-       struct intel_display *display = gt->i915->display;
        intel_engine_mask_t awake;
        int ret;
 
@@ -1248,8 +1246,6 @@ void intel_gt_reset(struct intel_gt *gt,
        if (intel_gt_gpu_reset_clobbers_display(gt))
                intel_irq_resume(gt->i915);
 
-       intel_overlay_reset(display);
-
        /* sanitize uC after engine reset */
        if (!intel_uc_uses_guc_submission(&gt->uc))
                intel_uc_reset_prepare(&gt->uc);
-- 
2.47.3

Reply via email to