From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A5A30F364A4 for ; Thu, 9 Apr 2026 17:09:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Cc:To:In-Reply-To:References :Message-Id:Content-Transfer-Encoding:Content-Type:MIME-Version:Subject:Date: From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=wX7pWpmziBNvFcnTaf8fwuS1/DS6EX90GHY6UzJaV7c=; b=XEALvmN76icba6fAcrTL0PLjc2 Y60TiZJx01MFUypD/zzxBz6pXqA4AAX8eBy4vfAwF1ZAAK0ruoXx1n0uBjghU/5X3LXuarcZSIr45 /HSIXRaw2OGM2+ZWuIb+xwwJto9WAJUQgOibjxZEbtRj/Ihm7O+AHyhYsrwJXwMObhyhNeAUZjzJ3 hzUgQ6aVhV8sQD8ea1f28IZnlNhR/mLBl2+uSZShQiyVOX5fNeFmW5pVa0ICHPC/PkCYxumBtbjEw rUmjD9lTq8KaJmqY5laaR8l0tIfLCnElmXMzUyIrnQdX4eLzfplk/eZtfKuZajahpMwhz3PGvASFJ rCtVJiEQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1wAssb-0000000Ayqx-1CHR; Thu, 09 Apr 2026 17:09:21 +0000 Received: from smtpout-03.galae.net ([185.246.85.4]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1wAssW-0000000Aykz-3jnJ for linux-arm-kernel@lists.infradead.org; Thu, 09 Apr 2026 17:09:19 +0000 Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-03.galae.net (Postfix) with ESMTPS id 72F6C4E429AB; Thu, 9 Apr 2026 17:09:15 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 3DCDD603E4; Thu, 9 Apr 2026 17:09:15 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 496B2104501B9; Thu, 9 Apr 2026 19:09:10 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1775754553; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=wX7pWpmziBNvFcnTaf8fwuS1/DS6EX90GHY6UzJaV7c=; b=DjUrLR5y0kyp5a69sBPx7vxDI68E2p+JWQxYiCEIdcla6HbZUbQbCnuGg5PCZ+KmSzMpNU 4SaxwgWrjgqzkivNaTqTWQrunfRupR7MlPkZq1rsyAa/GqaOLnta5kjeIkehhzZM6Zpsd6 OGOF2N15lIYlAprXd6Qykxh67y7KS2nDRSYiIn2NwkI7E+E2PofJBs5zHiO1JhopqW6plC QeqraZs69QyjajhNQkDnbzB8wEtKy7/xn44CRktZdvKafiMdLMsUxWbqGDGrAq0vJbgj0s QR5X/f+dXNO+HGqAW8cg7RoCKAiWczknwepO7OI16ymzEg1OrH6MrvEuTGj2bQ== From: Kory Maincent Date: Thu, 09 Apr 2026 19:08:22 +0200 Subject: [PATCH RFC 06/12] drm/i915/display: Switch to managed for plane MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260409-feat_link_cap-v1-6-7069e8199ce2@bootlin.com> References: <20260409-feat_link_cap-v1-0-7069e8199ce2@bootlin.com> In-Reply-To: <20260409-feat_link_cap-v1-0-7069e8199ce2@bootlin.com> To: Jani Nikula , Rodrigo Vivi , Joonas Lahtinen , Tvrtko Ursulin , David Airlie , Simona Vetter , Dave Airlie , Jesse Barnes , Eric Anholt , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Chun-Kuang Hu , Philipp Zabel , Matthias Brugger , AngeloGioacchino Del Regno , Chris Wilson Cc: Thomas Petazzoni , Mark Yacoub , Sean Paul , Louis Chauvet , intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, Simona Vetter , Kory Maincent X-Mailer: b4 0.14-dev-d4707 X-Last-TLS-Session-Version: TLSv1.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260409_100917_208127_E755AD2E X-CRM114-Status: GOOD ( 18.19 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org The current i915 driver uses non-managed function to create plane. It is not an issue yet, but in order to comply with the latest DRM requirement, convert this code to use drm and device managed helpers. Assisted-by: Claude Code:2.1.90 Signed-off-by: Kory Maincent --- drivers/gpu/drm/i915/display/i9xx_plane.c | 97 ++++++++--------- drivers/gpu/drm/i915/display/intel_cursor.c | 41 ++++--- drivers/gpu/drm/i915/display/intel_plane.c | 45 +------- drivers/gpu/drm/i915/display/intel_plane.h | 5 +- drivers/gpu/drm/i915/display/intel_sprite.c | 119 ++++++++++----------- drivers/gpu/drm/i915/display/skl_universal_plane.c | 102 +++++++++--------- 6 files changed, 181 insertions(+), 228 deletions(-) diff --git a/drivers/gpu/drm/i915/display/i9xx_plane.c b/drivers/gpu/drm/i915/display/i9xx_plane.c index 9c16753a1f3ba..032c56b478dfc 100644 --- a/drivers/gpu/drm/i915/display/i9xx_plane.c +++ b/drivers/gpu/drm/i915/display/i9xx_plane.c @@ -8,6 +8,7 @@ #include #include #include +#include #include #include "i9xx_plane.h" @@ -882,7 +883,6 @@ static unsigned int i9xx_plane_min_alignment(struct intel_plane *plane, static const struct drm_plane_funcs i965_plane_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = i965_plane_format_mod_supported, @@ -892,7 +892,6 @@ static const struct drm_plane_funcs i965_plane_funcs = { static const struct drm_plane_funcs i8xx_plane_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = i8xx_plane_format_mod_supported, @@ -923,32 +922,15 @@ static void i9xx_disable_tiling(struct intel_plane *plane) struct intel_plane * intel_primary_plane_create(struct intel_display *display, enum pipe pipe) { + struct intel_plane_state *plane_state; struct intel_plane *plane; const struct drm_plane_funcs *plane_funcs; unsigned int supported_rotations; const u64 *modifiers; const u32 *formats; int num_formats; - int ret, zpos; - - plane = intel_plane_alloc(); - if (IS_ERR(plane)) - return plane; - - plane->pipe = pipe; - /* - * On gen2/3 only plane A can do FBC, but the panel fitter and LVDS - * port is hooked to pipe B. Hence we want plane A feeding pipe B. - */ - if (HAS_FBC(display) && DISPLAY_VER(display) < 4 && - INTEL_NUM_PIPES(display) == 2) - plane->i9xx_plane = (enum i9xx_plane_id) !pipe; - else - plane->i9xx_plane = (enum i9xx_plane_id) pipe; - plane->id = PLANE_PRIMARY; - plane->frontbuffer_bit = INTEL_FRONTBUFFER(pipe, plane->id); - - intel_fbc_add_plane(i9xx_plane_fbc(display, plane->i9xx_plane), plane); + enum i9xx_plane_id i9xx_plane; + int zpos; if (display->platform.valleyview || display->platform.cherryview) { formats = vlv_primary_formats; @@ -984,6 +966,46 @@ intel_primary_plane_create(struct intel_display *display, enum pipe pipe) else plane_funcs = &i8xx_plane_funcs; + /* + * On gen2/3 only plane A can do FBC, but the panel fitter and LVDS + * port is hooked to pipe B. Hence we want plane A feeding pipe B. + */ + if (HAS_FBC(display) && DISPLAY_VER(display) < 4 && + INTEL_NUM_PIPES(display) == 2) + i9xx_plane = (enum i9xx_plane_id)!pipe; + else + i9xx_plane = (enum i9xx_plane_id)pipe; + + modifiers = intel_fb_plane_get_modifiers(display, INTEL_PLANE_CAP_TILING_X); + + if (DISPLAY_VER(display) >= 5 || display->platform.g4x) + plane = drmm_universal_plane_alloc(display->drm, struct intel_plane, base, + 0, plane_funcs, + formats, num_formats, + modifiers, + DRM_PLANE_TYPE_PRIMARY, + "primary %c", pipe_name(pipe)); + else + plane = drmm_universal_plane_alloc(display->drm, struct intel_plane, base, + 0, plane_funcs, + formats, num_formats, + modifiers, + DRM_PLANE_TYPE_PRIMARY, + "plane %c", + plane_name(i9xx_plane)); + + kfree(modifiers); + + if (IS_ERR(plane)) + return plane; + + plane->pipe = pipe; + plane->i9xx_plane = i9xx_plane; + plane->id = PLANE_PRIMARY; + plane->frontbuffer_bit = INTEL_FRONTBUFFER(pipe, plane->id); + + intel_fbc_add_plane(i9xx_plane_fbc(display, plane->i9xx_plane), plane); + if (display->platform.valleyview || display->platform.cherryview) plane->min_cdclk = vlv_plane_min_cdclk; else if (display->platform.broadwell || display->platform.haswell) @@ -1069,28 +1091,12 @@ intel_primary_plane_create(struct intel_display *display, enum pipe pipe) plane->disable_tiling = i9xx_disable_tiling; - modifiers = intel_fb_plane_get_modifiers(display, INTEL_PLANE_CAP_TILING_X); - - if (DISPLAY_VER(display) >= 5 || display->platform.g4x) - ret = drm_universal_plane_init(display->drm, &plane->base, - 0, plane_funcs, - formats, num_formats, - modifiers, - DRM_PLANE_TYPE_PRIMARY, - "primary %c", pipe_name(pipe)); - else - ret = drm_universal_plane_init(display->drm, &plane->base, - 0, plane_funcs, - formats, num_formats, - modifiers, - DRM_PLANE_TYPE_PRIMARY, - "plane %c", - plane_name(plane->i9xx_plane)); - - kfree(modifiers); + plane_state = kzalloc_obj(*plane_state); + if (!plane_state) + return ERR_PTR(-ENOMEM); - if (ret) - goto fail; + intel_plane_state_reset(plane_state, plane); + plane->base.state = &plane_state->uapi; if (display->platform.cherryview && pipe == PIPE_B) { supported_rotations = @@ -1114,11 +1120,6 @@ intel_primary_plane_create(struct intel_display *display, enum pipe pipe) intel_plane_helper_add(plane); return plane; - -fail: - intel_plane_free(plane); - - return ERR_PTR(ret); } static int i9xx_format_to_fourcc(int format) diff --git a/drivers/gpu/drm/i915/display/intel_cursor.c b/drivers/gpu/drm/i915/display/intel_cursor.c index 18d1014de3613..2493baf25fbe2 100644 --- a/drivers/gpu/drm/i915/display/intel_cursor.c +++ b/drivers/gpu/drm/i915/display/intel_cursor.c @@ -9,6 +9,7 @@ #include #include #include +#include #include #include @@ -971,7 +972,6 @@ intel_legacy_cursor_update(struct drm_plane *_plane, static const struct drm_plane_funcs intel_cursor_plane_funcs = { .update_plane = intel_legacy_cursor_update, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = intel_cursor_format_mod_supported, @@ -1004,11 +1004,23 @@ struct intel_plane * intel_cursor_plane_create(struct intel_display *display, enum pipe pipe) { + struct intel_plane_state *plane_state; struct intel_plane *cursor; - int ret, zpos; + int zpos; u64 *modifiers; - cursor = intel_plane_alloc(); + modifiers = intel_fb_plane_get_modifiers(display, INTEL_PLANE_CAP_NONE); + + cursor = drmm_universal_plane_alloc(display->drm, struct intel_plane, base, + 0, &intel_cursor_plane_funcs, + intel_cursor_formats, + ARRAY_SIZE(intel_cursor_formats), + modifiers, + DRM_PLANE_TYPE_CURSOR, + "cursor %c", pipe_name(pipe)); + + kfree(modifiers); + if (IS_ERR(cursor)) return cursor; @@ -1056,20 +1068,12 @@ intel_cursor_plane_create(struct intel_display *display, if (display->platform.i845g || display->platform.i865g || HAS_CUR_FBC(display)) cursor->cursor.size = ~0; - modifiers = intel_fb_plane_get_modifiers(display, INTEL_PLANE_CAP_NONE); - - ret = drm_universal_plane_init(display->drm, &cursor->base, - 0, &intel_cursor_plane_funcs, - intel_cursor_formats, - ARRAY_SIZE(intel_cursor_formats), - modifiers, - DRM_PLANE_TYPE_CURSOR, - "cursor %c", pipe_name(pipe)); - - kfree(modifiers); + plane_state = kzalloc_obj(*plane_state); + if (!plane_state) + return ERR_PTR(-ENOMEM); - if (ret) - goto fail; + intel_plane_state_reset(plane_state, cursor); + cursor->base.state = &plane_state->uapi; if (DISPLAY_VER(display) >= 4) drm_plane_create_rotation_property(&cursor->base, @@ -1088,11 +1092,6 @@ intel_cursor_plane_create(struct intel_display *display, intel_plane_helper_add(cursor); return cursor; - -fail: - intel_plane_free(cursor); - - return ERR_PTR(ret); } void intel_cursor_mode_config_init(struct intel_display *display) diff --git a/drivers/gpu/drm/i915/display/intel_plane.c b/drivers/gpu/drm/i915/display/intel_plane.c index 5390ceb21ca42..05c2dc0902f5c 100644 --- a/drivers/gpu/drm/i915/display/intel_plane.c +++ b/drivers/gpu/drm/i915/display/intel_plane.c @@ -62,8 +62,8 @@ #include "skl_universal_plane.h" #include "skl_watermark.h" -static void intel_plane_state_reset(struct intel_plane_state *plane_state, - struct intel_plane *plane) +void intel_plane_state_reset(struct intel_plane_state *plane_state, + struct intel_plane *plane) { memset(plane_state, 0, sizeof(*plane_state)); @@ -72,47 +72,6 @@ static void intel_plane_state_reset(struct intel_plane_state *plane_state, plane_state->scaler_id = -1; } -struct intel_plane *intel_plane_alloc(void) -{ - struct intel_plane_state *plane_state; - struct intel_plane *plane; - - plane = kzalloc_obj(*plane); - if (!plane) - return ERR_PTR(-ENOMEM); - - plane_state = kzalloc_obj(*plane_state); - if (!plane_state) { - kfree(plane); - return ERR_PTR(-ENOMEM); - } - - intel_plane_state_reset(plane_state, plane); - - plane->base.state = &plane_state->uapi; - - return plane; -} - -void intel_plane_free(struct intel_plane *plane) -{ - intel_plane_destroy_state(&plane->base, plane->base.state); - kfree(plane); -} - -/** - * intel_plane_destroy - destroy a plane - * @plane: plane to destroy - * - * Common destruction function for all types of planes (primary, cursor, - * sprite). - */ -void intel_plane_destroy(struct drm_plane *plane) -{ - drm_plane_cleanup(plane); - kfree(to_intel_plane(plane)); -} - /** * intel_plane_duplicate_state - duplicate plane state * @plane: drm plane diff --git a/drivers/gpu/drm/i915/display/intel_plane.h b/drivers/gpu/drm/i915/display/intel_plane.h index 5a8f2f3baab5f..56221619a2b29 100644 --- a/drivers/gpu/drm/i915/display/intel_plane.h +++ b/drivers/gpu/drm/i915/display/intel_plane.h @@ -55,9 +55,8 @@ void intel_plane_update_arm(struct intel_dsb *dsb, void intel_plane_disable_arm(struct intel_dsb *dsb, struct intel_plane *plane, const struct intel_crtc_state *crtc_state); -struct intel_plane *intel_plane_alloc(void); -void intel_plane_free(struct intel_plane *plane); -void intel_plane_destroy(struct drm_plane *plane); +void intel_plane_state_reset(struct intel_plane_state *plane_state, + struct intel_plane *plane); struct drm_plane_state *intel_plane_duplicate_state(struct drm_plane *plane); void intel_plane_destroy_state(struct drm_plane *plane, struct drm_plane_state *state); diff --git a/drivers/gpu/drm/i915/display/intel_sprite.c b/drivers/gpu/drm/i915/display/intel_sprite.c index 6a65f92e8a031..f285d15734ee5 100644 --- a/drivers/gpu/drm/i915/display/intel_sprite.c +++ b/drivers/gpu/drm/i915/display/intel_sprite.c @@ -36,6 +36,7 @@ #include #include #include +#include #include #include @@ -1563,7 +1564,6 @@ static bool vlv_sprite_format_mod_supported(struct drm_plane *_plane, static const struct drm_plane_funcs g4x_sprite_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = g4x_sprite_format_mod_supported, @@ -1573,7 +1573,6 @@ static const struct drm_plane_funcs g4x_sprite_funcs = { static const struct drm_plane_funcs snb_sprite_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = snb_sprite_format_mod_supported, @@ -1583,7 +1582,6 @@ static const struct drm_plane_funcs snb_sprite_funcs = { static const struct drm_plane_funcs vlv_sprite_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = vlv_sprite_format_mod_supported, @@ -1594,18 +1592,69 @@ struct intel_plane * intel_sprite_plane_create(struct intel_display *display, enum pipe pipe, int sprite) { + struct intel_plane_state *plane_state; struct intel_plane *plane; const struct drm_plane_funcs *plane_funcs; unsigned int supported_rotations; const u64 *modifiers; const u32 *formats; int num_formats; - int ret, zpos; + int zpos; + + if (display->platform.valleyview || display->platform.cherryview) { + if (display->platform.cherryview && pipe == PIPE_B) { + formats = chv_pipe_b_sprite_formats; + num_formats = ARRAY_SIZE(chv_pipe_b_sprite_formats); + } else { + formats = vlv_sprite_formats; + num_formats = ARRAY_SIZE(vlv_sprite_formats); + } + + plane_funcs = &vlv_sprite_funcs; + } else if (DISPLAY_VER(display) >= 7) { + formats = snb_sprite_formats; + num_formats = ARRAY_SIZE(snb_sprite_formats); + + plane_funcs = &snb_sprite_funcs; + } else { + if (display->platform.sandybridge) { + formats = snb_sprite_formats; + num_formats = ARRAY_SIZE(snb_sprite_formats); + + plane_funcs = &snb_sprite_funcs; + } else { + formats = g4x_sprite_formats; + num_formats = ARRAY_SIZE(g4x_sprite_formats); + + plane_funcs = &g4x_sprite_funcs; + } + } + + if (display->platform.cherryview && pipe == PIPE_B) { + supported_rotations = + DRM_MODE_ROTATE_0 | DRM_MODE_ROTATE_180 | + DRM_MODE_REFLECT_X; + } else { + supported_rotations = + DRM_MODE_ROTATE_0 | DRM_MODE_ROTATE_180; + } + + modifiers = intel_fb_plane_get_modifiers(display, INTEL_PLANE_CAP_TILING_X); + + plane = drmm_universal_plane_alloc(display->drm, struct intel_plane, base, + 0, plane_funcs, + formats, num_formats, modifiers, + DRM_PLANE_TYPE_OVERLAY, + "sprite %c", sprite_name(display, pipe, sprite)); + kfree(modifiers); - plane = intel_plane_alloc(); if (IS_ERR(plane)) return plane; + plane->pipe = pipe; + plane->id = PLANE_SPRITE0 + sprite; + plane->frontbuffer_bit = INTEL_FRONTBUFFER(pipe, plane->id); + if (display->platform.valleyview || display->platform.cherryview) { plane->update_noarm = vlv_sprite_update_noarm; plane->update_arm = vlv_sprite_update_arm; @@ -1621,16 +1670,6 @@ intel_sprite_plane_create(struct intel_display *display, /* FIXME undocumented for VLV/CHV so not sure what's actually needed */ if (intel_scanout_needs_vtd_wa(display)) plane->vtd_guard = 128; - - if (display->platform.cherryview && pipe == PIPE_B) { - formats = chv_pipe_b_sprite_formats; - num_formats = ARRAY_SIZE(chv_pipe_b_sprite_formats); - } else { - formats = vlv_sprite_formats; - num_formats = ARRAY_SIZE(vlv_sprite_formats); - } - - plane_funcs = &vlv_sprite_funcs; } else if (DISPLAY_VER(display) >= 7) { plane->update_noarm = ivb_sprite_update_noarm; plane->update_arm = ivb_sprite_update_arm; @@ -1652,11 +1691,6 @@ intel_sprite_plane_create(struct intel_display *display, if (intel_scanout_needs_vtd_wa(display)) plane->vtd_guard = 64; - - formats = snb_sprite_formats; - num_formats = ARRAY_SIZE(snb_sprite_formats); - - plane_funcs = &snb_sprite_funcs; } else { plane->update_noarm = g4x_sprite_update_noarm; plane->update_arm = g4x_sprite_update_arm; @@ -1671,44 +1705,14 @@ intel_sprite_plane_create(struct intel_display *display, if (intel_scanout_needs_vtd_wa(display)) plane->vtd_guard = 64; - - if (display->platform.sandybridge) { - formats = snb_sprite_formats; - num_formats = ARRAY_SIZE(snb_sprite_formats); - - plane_funcs = &snb_sprite_funcs; - } else { - formats = g4x_sprite_formats; - num_formats = ARRAY_SIZE(g4x_sprite_formats); - - plane_funcs = &g4x_sprite_funcs; - } } - if (display->platform.cherryview && pipe == PIPE_B) { - supported_rotations = - DRM_MODE_ROTATE_0 | DRM_MODE_ROTATE_180 | - DRM_MODE_REFLECT_X; - } else { - supported_rotations = - DRM_MODE_ROTATE_0 | DRM_MODE_ROTATE_180; - } + plane_state = kzalloc_obj(*plane_state); + if (!plane_state) + return ERR_PTR(-ENOMEM); - plane->pipe = pipe; - plane->id = PLANE_SPRITE0 + sprite; - plane->frontbuffer_bit = INTEL_FRONTBUFFER(pipe, plane->id); - - modifiers = intel_fb_plane_get_modifiers(display, INTEL_PLANE_CAP_TILING_X); - - ret = drm_universal_plane_init(display->drm, &plane->base, - 0, plane_funcs, - formats, num_formats, modifiers, - DRM_PLANE_TYPE_OVERLAY, - "sprite %c", sprite_name(display, pipe, sprite)); - kfree(modifiers); - - if (ret) - goto fail; + intel_plane_state_reset(plane_state, plane); + plane->base.state = &plane_state->uapi; drm_plane_create_rotation_property(&plane->base, DRM_MODE_ROTATE_0, @@ -1728,9 +1732,4 @@ intel_sprite_plane_create(struct intel_display *display, intel_plane_helper_add(plane); return plane; - -fail: - intel_plane_free(plane); - - return ERR_PTR(ret); } diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c b/drivers/gpu/drm/i915/display/skl_universal_plane.c index 11ba42c67e3ed..6d6b108bf7e46 100644 --- a/drivers/gpu/drm/i915/display/skl_universal_plane.c +++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c @@ -7,6 +7,7 @@ #include #include #include +#include #include #include "intel_bo.h" @@ -2690,7 +2691,6 @@ static bool tgl_plane_format_mod_supported(struct drm_plane *_plane, static const struct drm_plane_funcs skl_plane_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = skl_plane_format_mod_supported, @@ -2700,7 +2700,6 @@ static const struct drm_plane_funcs skl_plane_funcs = { static const struct drm_plane_funcs icl_plane_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = icl_plane_format_mod_supported, @@ -2710,7 +2709,6 @@ static const struct drm_plane_funcs icl_plane_funcs = { static const struct drm_plane_funcs tgl_plane_funcs = { .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, - .destroy = intel_plane_destroy, .atomic_duplicate_state = intel_plane_duplicate_state, .atomic_destroy_state = intel_plane_destroy_state, .format_mod_supported = tgl_plane_format_mod_supported, @@ -2850,6 +2848,7 @@ struct intel_plane * skl_universal_plane_create(struct intel_display *display, enum pipe pipe, enum plane_id plane_id) { + struct intel_plane_state *plane_state; const struct drm_plane_funcs *plane_funcs; struct intel_plane *plane; enum drm_plane_type plane_type; @@ -2858,10 +2857,50 @@ skl_universal_plane_create(struct intel_display *display, const u64 *modifiers; const u32 *formats; int num_formats; - int ret; u8 caps; - plane = intel_plane_alloc(); + if (DISPLAY_VER(display) >= 11) + formats = icl_get_plane_formats(display, pipe, + plane_id, &num_formats); + else if (DISPLAY_VER(display) >= 10) + formats = glk_get_plane_formats(display, pipe, + plane_id, &num_formats); + else + formats = skl_get_plane_formats(display, pipe, + plane_id, &num_formats); + + if (DISPLAY_VER(display) >= 12) + plane_funcs = &tgl_plane_funcs; + else if (DISPLAY_VER(display) == 11) + plane_funcs = &icl_plane_funcs; + else + plane_funcs = &skl_plane_funcs; + + if (plane_id == PLANE_1) + plane_type = DRM_PLANE_TYPE_PRIMARY; + else + plane_type = DRM_PLANE_TYPE_OVERLAY; + + if (DISPLAY_VER(display) >= 12) + caps = tgl_plane_caps(display, pipe, plane_id); + else if (DISPLAY_VER(display) == 11) + caps = icl_plane_caps(display, pipe, plane_id); + else if (DISPLAY_VER(display) == 10) + caps = glk_plane_caps(display, pipe, plane_id); + else + caps = skl_plane_caps(display, pipe, plane_id); + + modifiers = intel_fb_plane_get_modifiers(display, caps); + + plane = drmm_universal_plane_alloc(display->drm, struct intel_plane, base, + 0, plane_funcs, + formats, num_formats, modifiers, + plane_type, + "plane %d%c", plane_id + 1, + pipe_name(pipe)); + + kfree(modifiers); + if (IS_ERR(plane)) return plane; @@ -2940,50 +2979,12 @@ skl_universal_plane_create(struct intel_display *display, plane->can_async_flip = skl_plane_can_async_flip; } - if (DISPLAY_VER(display) >= 11) - formats = icl_get_plane_formats(display, pipe, - plane_id, &num_formats); - else if (DISPLAY_VER(display) >= 10) - formats = glk_get_plane_formats(display, pipe, - plane_id, &num_formats); - else - formats = skl_get_plane_formats(display, pipe, - plane_id, &num_formats); + plane_state = kzalloc_obj(*plane_state); + if (!plane_state) + return ERR_PTR(-ENOMEM); - if (DISPLAY_VER(display) >= 12) - plane_funcs = &tgl_plane_funcs; - else if (DISPLAY_VER(display) == 11) - plane_funcs = &icl_plane_funcs; - else - plane_funcs = &skl_plane_funcs; - - if (plane_id == PLANE_1) - plane_type = DRM_PLANE_TYPE_PRIMARY; - else - plane_type = DRM_PLANE_TYPE_OVERLAY; - - if (DISPLAY_VER(display) >= 12) - caps = tgl_plane_caps(display, pipe, plane_id); - else if (DISPLAY_VER(display) == 11) - caps = icl_plane_caps(display, pipe, plane_id); - else if (DISPLAY_VER(display) == 10) - caps = glk_plane_caps(display, pipe, plane_id); - else - caps = skl_plane_caps(display, pipe, plane_id); - - modifiers = intel_fb_plane_get_modifiers(display, caps); - - ret = drm_universal_plane_init(display->drm, &plane->base, - 0, plane_funcs, - formats, num_formats, modifiers, - plane_type, - "plane %d%c", plane_id + 1, - pipe_name(pipe)); - - kfree(modifiers); - - if (ret) - goto fail; + intel_plane_state_reset(plane_state, plane); + plane->base.state = &plane_state->uapi; if (DISPLAY_VER(display) >= 13) supported_rotations = DRM_MODE_ROTATE_0 | DRM_MODE_ROTATE_180; @@ -3033,11 +3034,6 @@ skl_universal_plane_create(struct intel_display *display, intel_plane_helper_add(plane); return plane; - -fail: - intel_plane_free(plane); - - return ERR_PTR(ret); } void -- 2.43.0