diff mbox series

[07/16] drm/i915/gvt: Use the full PLANE_KEY*() defines

Message ID 20240510152329.24098-8-ville.syrjala@linux.intel.com (mailing list archive)
State New
Headers show
Series drm/i915: skl+ plane register stuff | expand

Commit Message

Ville Syrjala May 10, 2024, 3:23 p.m. UTC
From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Stop hand rolling PLANE_KEY*() register defines and just
use the real thing.

Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
CC: Zhi Wang <zhi.wang.linux@gmail.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 drivers/gpu/drm/i915/intel_gvt_mmio_table.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

Comments

Jani Nikula May 13, 2024, 10:25 a.m. UTC | #1
On Fri, 10 May 2024, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Stop hand rolling PLANE_KEY*() register defines and just
> use the real thing.
>
> Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
> CC: Zhi Wang <zhi.wang.linux@gmail.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Reviewed-by: Jani Nikula <jani.nikula@intel.com>

> ---
>  drivers/gpu/drm/i915/intel_gvt_mmio_table.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c b/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
> index ad3bf60855bc..b53c98cd6d7f 100644
> --- a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
> +++ b/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
> @@ -1075,15 +1075,15 @@ static int iterate_skl_plus_mmio(struct intel_gvt_mmio_table_iter *iter)
>  	MMIO_D(_MMIO(0x70034));
>  	MMIO_D(_MMIO(0x71034));
>  	MMIO_D(_MMIO(0x72034));
> -	MMIO_D(_MMIO(_PLANE_KEYVAL_1(PIPE_A)));
> -	MMIO_D(_MMIO(_PLANE_KEYVAL_1(PIPE_B)));
> -	MMIO_D(_MMIO(_PLANE_KEYVAL_1(PIPE_C)));
> -	MMIO_D(_MMIO(_PLANE_KEYMAX_1(PIPE_A)));
> -	MMIO_D(_MMIO(_PLANE_KEYMAX_1(PIPE_B)));
> -	MMIO_D(_MMIO(_PLANE_KEYMAX_1(PIPE_C)));
> -	MMIO_D(_MMIO(_PLANE_KEYMSK_1(PIPE_A)));
> -	MMIO_D(_MMIO(_PLANE_KEYMSK_1(PIPE_B)));
> -	MMIO_D(_MMIO(_PLANE_KEYMSK_1(PIPE_C)));
> +	MMIO_D(PLANE_KEYVAL(PIPE_A, 0));
> +	MMIO_D(PLANE_KEYVAL(PIPE_B, 0));
> +	MMIO_D(PLANE_KEYVAL(PIPE_C, 0));
> +	MMIO_D(PLANE_KEYMAX(PIPE_A, 0));
> +	MMIO_D(PLANE_KEYMAX(PIPE_B, 0));
> +	MMIO_D(PLANE_KEYMAX(PIPE_C, 0));
> +	MMIO_D(PLANE_KEYMSK(PIPE_A, 0));
> +	MMIO_D(PLANE_KEYMSK(PIPE_B, 0));
> +	MMIO_D(PLANE_KEYMSK(PIPE_C, 0));
>  	MMIO_D(_MMIO(0x44500));
>  #define CSFE_CHICKEN1_REG(base) _MMIO((base) + 0xD4)
>  	MMIO_RING_D(CSFE_CHICKEN1_REG);
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c b/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
index ad3bf60855bc..b53c98cd6d7f 100644
--- a/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
+++ b/drivers/gpu/drm/i915/intel_gvt_mmio_table.c
@@ -1075,15 +1075,15 @@  static int iterate_skl_plus_mmio(struct intel_gvt_mmio_table_iter *iter)
 	MMIO_D(_MMIO(0x70034));
 	MMIO_D(_MMIO(0x71034));
 	MMIO_D(_MMIO(0x72034));
-	MMIO_D(_MMIO(_PLANE_KEYVAL_1(PIPE_A)));
-	MMIO_D(_MMIO(_PLANE_KEYVAL_1(PIPE_B)));
-	MMIO_D(_MMIO(_PLANE_KEYVAL_1(PIPE_C)));
-	MMIO_D(_MMIO(_PLANE_KEYMAX_1(PIPE_A)));
-	MMIO_D(_MMIO(_PLANE_KEYMAX_1(PIPE_B)));
-	MMIO_D(_MMIO(_PLANE_KEYMAX_1(PIPE_C)));
-	MMIO_D(_MMIO(_PLANE_KEYMSK_1(PIPE_A)));
-	MMIO_D(_MMIO(_PLANE_KEYMSK_1(PIPE_B)));
-	MMIO_D(_MMIO(_PLANE_KEYMSK_1(PIPE_C)));
+	MMIO_D(PLANE_KEYVAL(PIPE_A, 0));
+	MMIO_D(PLANE_KEYVAL(PIPE_B, 0));
+	MMIO_D(PLANE_KEYVAL(PIPE_C, 0));
+	MMIO_D(PLANE_KEYMAX(PIPE_A, 0));
+	MMIO_D(PLANE_KEYMAX(PIPE_B, 0));
+	MMIO_D(PLANE_KEYMAX(PIPE_C, 0));
+	MMIO_D(PLANE_KEYMSK(PIPE_A, 0));
+	MMIO_D(PLANE_KEYMSK(PIPE_B, 0));
+	MMIO_D(PLANE_KEYMSK(PIPE_C, 0));
 	MMIO_D(_MMIO(0x44500));
 #define CSFE_CHICKEN1_REG(base) _MMIO((base) + 0xD4)
 	MMIO_RING_D(CSFE_CHICKEN1_REG);