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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 22886C25B10 for ; Mon, 13 May 2024 10:10:53 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id A28BD10E461; Mon, 13 May 2024 10:10:52 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="TsqngODo"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.18]) by gabe.freedesktop.org (Postfix) with ESMTPS id D1FF910E461 for ; Mon, 13 May 2024 10:10:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1715595051; x=1747131051; h=from:to:cc:subject:in-reply-to:references:date: message-id:mime-version:content-transfer-encoding; bh=GAWkJmuurwoXl9rT9yxN/zutfJ/u6m63dm4ORKv4/4o=; b=TsqngODohTdRLInymup0UkEPIgw+iZMU1r7US1ctJgYOquN4JfzxGsqU zOaRkCfi2r2c7YSb0ALUquxIhd98+WH/Xj/o4GdTISgSTzSJSuOiU/X8a eK+jkdaUzVHV817EmkV2UtwFpbi2+ot8cDm0o1QBrd3qug75hTxtxnRve yHEaCE4HbeA3A6fA0Vr/s5losW4ZJ8PT1qAjmB2UTBOUpX31ZfABBF94h OlZ6qp+8g6CZoQCO4yAlmTFFb4EubC0tjxDMtU+BjogutQ/biPlrt0Vl+ pkEH8GO99wNOqTLqtXpWTjPRxK+RD8VXLipxkdoiSwoJtfJ96HaQi7pPD Q==; X-CSE-ConnectionGUID: QrkLcZTgRRyE6kdJ7gz1HA== X-CSE-MsgGUID: xAY8aIFsRj63Cxz4+5sOxA== X-IronPort-AV: E=McAfee;i="6600,9927,11071"; a="11681739" X-IronPort-AV: E=Sophos;i="6.08,158,1712646000"; d="scan'208";a="11681739" Received: from orviesa005.jf.intel.com ([10.64.159.145]) by orvoesa110.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 May 2024 03:10:51 -0700 X-CSE-ConnectionGUID: mxamVmO5SFSWSlpc4Th0jQ== X-CSE-MsgGUID: iltxAAz1RjS6Wk4y3Ac/Jg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,158,1712646000"; d="scan'208";a="35052551" Received: from fdefranc-mobl3.ger.corp.intel.com (HELO localhost) ([10.245.246.7]) by orviesa005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 May 2024 03:10:48 -0700 From: Jani Nikula To: Ville Syrjala , intel-gfx@lists.freedesktop.org Cc: Zhenyu Wang , Zhi Wang Subject: Re: [PATCH 03/16] drm/i915: Extract intel_cursor_regs.h In-Reply-To: <20240510152329.24098-4-ville.syrjala@linux.intel.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20240510152329.24098-1-ville.syrjala@linux.intel.com> <20240510152329.24098-4-ville.syrjala@linux.intel.com> Date: Mon, 13 May 2024 13:10:45 +0300 Message-ID: <871q66ggyi.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Fri, 10 May 2024, Ville Syrjala wrote: > From: Ville Syrj=C3=A4l=C3=A4 > > Move most cursor register definitions into their own file. > Declutters i915_reg.h a bit more. > > Cc: Zhenyu Wang > CC: Zhi Wang > Signed-off-by: Ville Syrj=C3=A4l=C3=A4 Reviewed-by: Jani Nikula > --- > drivers/gpu/drm/i915/display/intel_cursor.c | 1 + > .../gpu/drm/i915/display/intel_cursor_regs.h | 78 +++++++++++++++++++ > drivers/gpu/drm/i915/display/intel_display.c | 1 + > drivers/gpu/drm/i915/display/intel_psr.c | 1 + > drivers/gpu/drm/i915/gvt/display.c | 1 + > drivers/gpu/drm/i915/gvt/fb_decoder.c | 1 + > drivers/gpu/drm/i915/i915_reg.h | 70 ----------------- > drivers/gpu/drm/i915/intel_gvt_mmio_table.c | 1 + > 8 files changed, 84 insertions(+), 70 deletions(-) > create mode 100644 drivers/gpu/drm/i915/display/intel_cursor_regs.h > > diff --git a/drivers/gpu/drm/i915/display/intel_cursor.c b/drivers/gpu/dr= m/i915/display/intel_cursor.c > index 2118b87ccb10..d2b459634732 100644 > --- a/drivers/gpu/drm/i915/display/intel_cursor.c > +++ b/drivers/gpu/drm/i915/display/intel_cursor.c > @@ -14,6 +14,7 @@ > #include "intel_atomic.h" > #include "intel_atomic_plane.h" > #include "intel_cursor.h" > +#include "intel_cursor_regs.h" > #include "intel_de.h" > #include "intel_display.h" > #include "intel_display_types.h" > diff --git a/drivers/gpu/drm/i915/display/intel_cursor_regs.h b/drivers/g= pu/drm/i915/display/intel_cursor_regs.h > new file mode 100644 > index 000000000000..62f7fb5c3f10 > --- /dev/null > +++ b/drivers/gpu/drm/i915/display/intel_cursor_regs.h > @@ -0,0 +1,78 @@ > +/* SPDX-License-Identifier: MIT */ > +/* > + * Copyright =C2=A9 2024 Intel Corporation > + */ > + > +#ifndef __INTEL_CURSOR_REGS_H__ > +#define __INTEL_CURSOR_REGS_H__ > + > +#include "intel_display_reg_defs.h" > + > +#define _CURACNTR 0x70080 > +/* Old style CUR*CNTR flags (desktop 8xx) */ > +#define CURSOR_ENABLE REG_BIT(31) > +#define CURSOR_PIPE_GAMMA_ENABLE REG_BIT(30) > +#define CURSOR_STRIDE_MASK REG_GENMASK(29, 28) > +#define CURSOR_STRIDE(stride) REG_FIELD_PREP(CURSOR_STRIDE_MASK, ffs(s= tride) - 9) /* 256,512,1k,2k */ > +#define CURSOR_FORMAT_MASK REG_GENMASK(26, 24) > +#define CURSOR_FORMAT_2C REG_FIELD_PREP(CURSOR_FORMAT_MASK, 0) > +#define CURSOR_FORMAT_3C REG_FIELD_PREP(CURSOR_FORMAT_MASK, 1) > +#define CURSOR_FORMAT_4C REG_FIELD_PREP(CURSOR_FORMAT_MASK, 2) > +#define CURSOR_FORMAT_ARGB REG_FIELD_PREP(CURSOR_FORMAT_MASK, 4) > +#define CURSOR_FORMAT_XRGB REG_FIELD_PREP(CURSOR_FORMAT_MASK, 5) > +/* New style CUR*CNTR flags */ > +#define MCURSOR_ARB_SLOTS_MASK REG_GENMASK(30, 28) /* icl+ */ > +#define MCURSOR_ARB_SLOTS(x) REG_FIELD_PREP(MCURSOR_ARB_SLOTS_MASK, (= x)) /* icl+ */ > +#define MCURSOR_PIPE_SEL_MASK REG_GENMASK(29, 28) > +#define MCURSOR_PIPE_SEL(pipe) REG_FIELD_PREP(MCURSOR_PIPE_SEL_MASK, (= pipe)) > +#define MCURSOR_PIPE_GAMMA_ENABLE REG_BIT(26) > +#define MCURSOR_PIPE_CSC_ENABLE REG_BIT(24) /* ilk+ */ > +#define MCURSOR_ROTATE_180 REG_BIT(15) > +#define MCURSOR_TRICKLE_FEED_DISABLE REG_BIT(14) > +#define MCURSOR_MODE_MASK 0x27 > +#define MCURSOR_MODE_DISABLE 0x00 > +#define MCURSOR_MODE_128_32B_AX 0x02 > +#define MCURSOR_MODE_256_32B_AX 0x03 > +#define MCURSOR_MODE_64_2B 0x04 > +#define MCURSOR_MODE_64_32B_AX 0x07 > +#define MCURSOR_MODE_128_ARGB_AX (0x20 | MCURSOR_MODE_128_32B_AX) > +#define MCURSOR_MODE_256_ARGB_AX (0x20 | MCURSOR_MODE_256_32B_AX) > +#define MCURSOR_MODE_64_ARGB_AX (0x20 | MCURSOR_MODE_64_32B_AX) > +#define _CURABASE 0x70084 > +#define _CURAPOS 0x70088 > +#define _CURAPOS_ERLY_TPT 0x7008c > +#define CURSOR_POS_Y_SIGN REG_BIT(31) > +#define CURSOR_POS_Y_MASK REG_GENMASK(30, 16) > +#define CURSOR_POS_Y(y) REG_FIELD_PREP(CURSOR_POS_Y_MASK, (y)) > +#define CURSOR_POS_X_SIGN REG_BIT(15) > +#define CURSOR_POS_X_MASK REG_GENMASK(14, 0) > +#define CURSOR_POS_X(x) REG_FIELD_PREP(CURSOR_POS_X_MASK, (x)) > +#define _CURASIZE 0x700a0 /* 845/865 */ > +#define CURSOR_HEIGHT_MASK REG_GENMASK(21, 12) > +#define CURSOR_HEIGHT(h) REG_FIELD_PREP(CURSOR_HEIGHT_MASK, (h)) > +#define CURSOR_WIDTH_MASK REG_GENMASK(9, 0) > +#define CURSOR_WIDTH(w) REG_FIELD_PREP(CURSOR_WIDTH_MASK, (w)) > +#define _CUR_FBC_CTL_A 0x700a0 /* ivb+ */ > +#define CUR_FBC_EN REG_BIT(31) > +#define CUR_FBC_HEIGHT_MASK REG_GENMASK(7, 0) > +#define CUR_FBC_HEIGHT(h) REG_FIELD_PREP(CUR_FBC_HEIGHT_MASK, (h)) > +#define _CUR_CHICKEN_A 0x700a4 /* mtl+ */ > +#define _CURASURFLIVE 0x700ac /* g4x+ */ > +#define _CURBCNTR 0x700c0 > +#define _CURBBASE 0x700c4 > +#define _CURBPOS 0x700c8 > + > +#define _CURBCNTR_IVB 0x71080 > +#define _CURBBASE_IVB 0x71084 > +#define _CURBPOS_IVB 0x71088 > + > +#define CURCNTR(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURACNTR) > +#define CURBASE(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURABASE) > +#define CURPOS(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURAPOS) > +#define CURPOS_ERLY_TPT(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURAPOS_ERL= Y_TPT) > +#define CURSIZE(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURASIZE) > +#define CUR_FBC_CTL(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CUR_FBC_CTL_A) > +#define CUR_CHICKEN(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CUR_CHICKEN_A) > +#define CURSURFLIVE(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURASURFLIVE) > + > +#endif /* __INTEL_CURSOR_REGS_H__ */ > diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/d= rm/i915/display/intel_display.c > index a2c331c696fe..839643c07566 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -65,6 +65,7 @@ > #include "intel_crt.h" > #include "intel_crtc.h" > #include "intel_crtc_state_dump.h" > +#include "intel_cursor_regs.h" > #include "intel_ddi.h" > #include "intel_de.h" > #include "intel_display_driver.h" > diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i= 915/display/intel_psr.c > index 595eb1b3b6c6..8cb12e7b157c 100644 > --- a/drivers/gpu/drm/i915/display/intel_psr.c > +++ b/drivers/gpu/drm/i915/display/intel_psr.c > @@ -29,6 +29,7 @@ > #include "i915_reg.h" > #include "intel_atomic.h" > #include "intel_crtc.h" > +#include "intel_cursor_regs.h" > #include "intel_ddi.h" > #include "intel_de.h" > #include "intel_display_types.h" > diff --git a/drivers/gpu/drm/i915/gvt/display.c b/drivers/gpu/drm/i915/gv= t/display.c > index 2b7df7fcf369..527e0bb2b15e 100644 > --- a/drivers/gpu/drm/i915/gvt/display.c > +++ b/drivers/gpu/drm/i915/gvt/display.c > @@ -37,6 +37,7 @@ > #include "gvt.h" >=20=20 > #include "display/bxt_dpio_phy_regs.h" > +#include "display/intel_cursor_regs.h" > #include "display/intel_display.h" > #include "display/intel_dpio_phy.h" > #include "display/intel_sprite_regs.h" > diff --git a/drivers/gpu/drm/i915/gvt/fb_decoder.c b/drivers/gpu/drm/i915= /gvt/fb_decoder.c > index dd6eb0ae003a..ac3ec9d1ef5f 100644 > --- a/drivers/gpu/drm/i915/gvt/fb_decoder.c > +++ b/drivers/gpu/drm/i915/gvt/fb_decoder.c > @@ -40,6 +40,7 @@ > #include "i915_pvinfo.h" > #include "i915_reg.h" >=20=20 > +#include "display/intel_cursor_regs.h" > #include "display/intel_sprite_regs.h" > #include "display/skl_universal_plane_regs.h" >=20=20 > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_= reg.h > index 08725dc7883e..52b029cd3981 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -1762,8 +1762,6 @@ > #define PIPE_LINK_M_G4X(pipe) _MMIO_PIPE(pipe, _PIPEA_LINK_M_G4X, _PIPEB= _LINK_M_G4X) > #define PIPE_LINK_N_G4X(pipe) _MMIO_PIPE(pipe, _PIPEA_LINK_N_G4X, _PIPEB= _LINK_N_G4X) >=20=20 > -/* Display & cursor control */ > - > /* Pipe A */ > #define _PIPEADSL 0x70000 > #define PIPEDSL_CURR_FIELD REG_BIT(31) /* ctg+ */ > @@ -2266,74 +2264,6 @@ > #define PIPE_FRMCOUNT_G4X(pipe) _MMIO_PIPE2(dev_priv, pipe, _PIPEA_FRMCO= UNT_G4X) > #define PIPE_FLIPCOUNT_G4X(pipe) _MMIO_PIPE2(dev_priv, pipe, _PIPEA_FLIP= COUNT_G4X) >=20=20 > -/* Cursor A & B regs */ > -#define _CURACNTR 0x70080 > -/* Old style CUR*CNTR flags (desktop 8xx) */ > -#define CURSOR_ENABLE REG_BIT(31) > -#define CURSOR_PIPE_GAMMA_ENABLE REG_BIT(30) > -#define CURSOR_STRIDE_MASK REG_GENMASK(29, 28) > -#define CURSOR_STRIDE(stride) REG_FIELD_PREP(CURSOR_STRIDE_MASK, ffs(s= tride) - 9) /* 256,512,1k,2k */ > -#define CURSOR_FORMAT_MASK REG_GENMASK(26, 24) > -#define CURSOR_FORMAT_2C REG_FIELD_PREP(CURSOR_FORMAT_MASK, 0) > -#define CURSOR_FORMAT_3C REG_FIELD_PREP(CURSOR_FORMAT_MASK, 1) > -#define CURSOR_FORMAT_4C REG_FIELD_PREP(CURSOR_FORMAT_MASK, 2) > -#define CURSOR_FORMAT_ARGB REG_FIELD_PREP(CURSOR_FORMAT_MASK, 4) > -#define CURSOR_FORMAT_XRGB REG_FIELD_PREP(CURSOR_FORMAT_MASK, 5) > -/* New style CUR*CNTR flags */ > -#define MCURSOR_ARB_SLOTS_MASK REG_GENMASK(30, 28) /* icl+ */ > -#define MCURSOR_ARB_SLOTS(x) REG_FIELD_PREP(MCURSOR_ARB_SLOTS_MASK, (= x)) /* icl+ */ > -#define MCURSOR_PIPE_SEL_MASK REG_GENMASK(29, 28) > -#define MCURSOR_PIPE_SEL(pipe) REG_FIELD_PREP(MCURSOR_PIPE_SEL_MASK, (= pipe)) > -#define MCURSOR_PIPE_GAMMA_ENABLE REG_BIT(26) > -#define MCURSOR_PIPE_CSC_ENABLE REG_BIT(24) /* ilk+ */ > -#define MCURSOR_ROTATE_180 REG_BIT(15) > -#define MCURSOR_TRICKLE_FEED_DISABLE REG_BIT(14) > -#define MCURSOR_MODE_MASK 0x27 > -#define MCURSOR_MODE_DISABLE 0x00 > -#define MCURSOR_MODE_128_32B_AX 0x02 > -#define MCURSOR_MODE_256_32B_AX 0x03 > -#define MCURSOR_MODE_64_2B 0x04 > -#define MCURSOR_MODE_64_32B_AX 0x07 > -#define MCURSOR_MODE_128_ARGB_AX (0x20 | MCURSOR_MODE_128_32B_AX) > -#define MCURSOR_MODE_256_ARGB_AX (0x20 | MCURSOR_MODE_256_32B_AX) > -#define MCURSOR_MODE_64_ARGB_AX (0x20 | MCURSOR_MODE_64_32B_AX) > -#define _CURABASE 0x70084 > -#define _CURAPOS 0x70088 > -#define _CURAPOS_ERLY_TPT 0x7008c > -#define CURSOR_POS_Y_SIGN REG_BIT(31) > -#define CURSOR_POS_Y_MASK REG_GENMASK(30, 16) > -#define CURSOR_POS_Y(y) REG_FIELD_PREP(CURSOR_POS_Y_MASK, (y)) > -#define CURSOR_POS_X_SIGN REG_BIT(15) > -#define CURSOR_POS_X_MASK REG_GENMASK(14, 0) > -#define CURSOR_POS_X(x) REG_FIELD_PREP(CURSOR_POS_X_MASK, (x)) > -#define _CURASIZE 0x700a0 /* 845/865 */ > -#define CURSOR_HEIGHT_MASK REG_GENMASK(21, 12) > -#define CURSOR_HEIGHT(h) REG_FIELD_PREP(CURSOR_HEIGHT_MASK, (h)) > -#define CURSOR_WIDTH_MASK REG_GENMASK(9, 0) > -#define CURSOR_WIDTH(w) REG_FIELD_PREP(CURSOR_WIDTH_MASK, (w)) > -#define _CUR_FBC_CTL_A 0x700a0 /* ivb+ */ > -#define CUR_FBC_EN REG_BIT(31) > -#define CUR_FBC_HEIGHT_MASK REG_GENMASK(7, 0) > -#define CUR_FBC_HEIGHT(h) REG_FIELD_PREP(CUR_FBC_HEIGHT_MASK, (h)) > -#define _CUR_CHICKEN_A 0x700a4 /* mtl+ */ > -#define _CURASURFLIVE 0x700ac /* g4x+ */ > -#define _CURBCNTR 0x700c0 > -#define _CURBBASE 0x700c4 > -#define _CURBPOS 0x700c8 > - > -#define _CURBCNTR_IVB 0x71080 > -#define _CURBBASE_IVB 0x71084 > -#define _CURBPOS_IVB 0x71088 > - > -#define CURCNTR(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURACNTR) > -#define CURBASE(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURABASE) > -#define CURPOS(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURAPOS) > -#define CURPOS_ERLY_TPT(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURAPOS_ERL= Y_TPT) > -#define CURSIZE(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURASIZE) > -#define CUR_FBC_CTL(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CUR_FBC_CTL_A) > -#define CUR_CHICKEN(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CUR_CHICKEN_A) > -#define CURSURFLIVE(pipe) _MMIO_CURSOR2(dev_priv, pipe, _CURASURFLIVE) > - > /* Display A control */ > #define _DSPAADDR_VLV 0x7017C /* vlv/chv */ > #define _DSPACNTR 0x70180 > diff --git a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c b/drivers/gpu/dr= m/i915/intel_gvt_mmio_table.c > index 178eba121881..3b79c1c84b79 100644 > --- a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c > +++ b/drivers/gpu/drm/i915/intel_gvt_mmio_table.c > @@ -7,6 +7,7 @@ > #include "display/intel_audio_regs.h" > #include "display/intel_backlight_regs.h" > #include "display/intel_color_regs.h" > +#include "display/intel_cursor_regs.h" > #include "display/intel_display_types.h" > #include "display/intel_dmc_regs.h" > #include "display/intel_dp_aux_regs.h" --=20 Jani Nikula, Intel