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 6C164CD4851 for ; Wed, 13 May 2026 13:57:16 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 0953210EEA2; Wed, 13 May 2026 13:57:16 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="VP0QdOb2"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.15]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5C96710E5DC; Wed, 13 May 2026 13:57:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1778680635; x=1810216635; h=date:from:to:cc:subject:message-id:references: mime-version:content-transfer-encoding:in-reply-to; bh=GiSOX1wnJVA4Z74UHhgS/GsqbkHc4Trrb+bLjqvMHjc=; b=VP0QdOb2ksNdEzvY/L6BrUME8grzS3dK0LoN6VK1xHYX10J4z239QogY sbnXB65qhS+eR+K3ceL9cUEYLwx85H0D5Q7Q9cwQlVJp0YYHW2XjA7Q3Z ohGQKnSDSBPvmNaes5iocCZkX26794oBNO2ARPvR7RnrHmKxH4I/osReu EPG8AOUpXIGw982Y3JPplNDcg7bhvFvNFdezTyf+VBGYFbPcME4uZG+ax W2Qch/EVUFu2ny2uapvYt61JBAsLUs2lsdnyjwr3hXIrXlXXWlpH7qF/r cbx/OzU5P+bnN2PXxojLMHyeoJqPatVq3X+y5xRQcXRb3b5fwkhT5SFQa g==; X-CSE-ConnectionGUID: 3nKiKpwRQSihcPNORYoeng== X-CSE-MsgGUID: g14b5op+QO6CEDBQVyYHKA== X-IronPort-AV: E=McAfee;i="6800,10657,11784"; a="79723067" X-IronPort-AV: E=Sophos;i="6.23,232,1770624000"; d="scan'208";a="79723067" Received: from fmviesa004.fm.intel.com ([10.60.135.144]) by fmvoesa109.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 May 2026 06:57:15 -0700 X-CSE-ConnectionGUID: daTnlwi8TAmY+aIzeQcnZA== X-CSE-MsgGUID: reaNVbWMQL2tpbPv1+VNTw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,232,1770624000"; d="scan'208";a="239901244" Received: from amilburn-desk.amilburn-desk (HELO localhost) ([10.245.244.10]) by fmviesa004-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 May 2026 06:57:13 -0700 Date: Wed, 13 May 2026 16:57:10 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Jani Nikula Cc: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org Subject: Re: [PATCH v3 4/6] drm/i915/irq: add intel_display_irq_postinstall() to irq funcs Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Patchwork-Hint: comment Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs Bertel Jungin Aukio 5, 02600 Espoo, Finland 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 Wed, May 13, 2026 at 01:10:09PM +0300, Jani Nikula wrote: > Call the platform specific display irq postinstall hooks via > intel_display_irq_postinstall(). > > Relocate the gen11 HAS_DISPLAY() check to > intel_display_irq_postinstall(), as the funcs pointer won't be > initialized for no display. > > v2: > - relocate HAS_DISPLAY() (Sashiko) > > Signed-off-by: Jani Nikula Reviewed-by: Ville Syrjälä > --- > .../gpu/drm/i915/display/intel_display_irq.c | 30 +++++++++++++------ > .../gpu/drm/i915/display/intel_display_irq.h | 7 +---- > drivers/gpu/drm/i915/i915_irq.c | 16 +++++----- > drivers/gpu/drm/xe/display/xe_display.c | 2 +- > 4 files changed, 31 insertions(+), 24 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_display_irq.c b/drivers/gpu/drm/i915/display/intel_display_irq.c > index 62a849673454..1c3c8095765d 100644 > --- a/drivers/gpu/drm/i915/display/intel_display_irq.c > +++ b/drivers/gpu/drm/i915/display/intel_display_irq.c > @@ -1981,7 +1981,7 @@ u32 i9xx_display_irq_enable_mask(struct intel_display *display) > return enable_mask; > } > > -void i915_display_irq_postinstall(struct intel_display *display) > +static void i915_display_irq_postinstall(struct intel_display *display) > { > /* > * Interrupt setup is already guaranteed to be single-threaded, this is > @@ -1995,7 +1995,7 @@ void i915_display_irq_postinstall(struct intel_display *display) > i915_enable_asle_pipestat(display); > } > > -void i965_display_irq_postinstall(struct intel_display *display) > +static void i965_display_irq_postinstall(struct intel_display *display) > { > /* > * Interrupt setup is already guaranteed to be single-threaded, this is > @@ -2057,7 +2057,7 @@ static void _vlv_display_irq_postinstall(struct intel_display *display) > irq_init(display, VLV_IRQ_REGS, display->irq.vlv_imr_mask, enable_mask); > } > > -void vlv_display_irq_postinstall(struct intel_display *display) > +static void vlv_display_irq_postinstall(struct intel_display *display) > { > spin_lock_irq(&display->irq.lock); > if (display->irq.vlv_display_irqs_enabled) > @@ -2262,7 +2262,7 @@ void valleyview_disable_display_irqs(struct intel_display *display) > spin_unlock_irq(&display->irq.lock); > } > > -void ilk_de_irq_postinstall(struct intel_display *display) > +static void ilk_de_irq_postinstall(struct intel_display *display) > { > u32 display_mask, extra_mask; > > @@ -2306,7 +2306,7 @@ void ilk_de_irq_postinstall(struct intel_display *display) > static void mtp_irq_postinstall(struct intel_display *display); > static void icp_irq_postinstall(struct intel_display *display); > > -void gen8_de_irq_postinstall(struct intel_display *display) > +static void gen8_de_irq_postinstall(struct intel_display *display) > { > u32 de_pipe_masked = gen8_de_pipe_fault_mask(display) | > GEN8_PIPE_CDCLK_CRC_DONE; > @@ -2433,11 +2433,8 @@ static void icp_irq_postinstall(struct intel_display *display) > irq_init(display, SDE_IRQ_REGS, ~mask, 0xffffffff); > } > > -void gen11_de_irq_postinstall(struct intel_display *display) > +static void gen11_de_irq_postinstall(struct intel_display *display) > { > - if (!HAS_DISPLAY(display)) > - return; > - > gen8_de_irq_postinstall(display); > > intel_de_write(display, GEN11_DISPLAY_INT_CTL, GEN11_DISPLAY_IRQ_ENABLE); > @@ -2445,30 +2442,37 @@ void gen11_de_irq_postinstall(struct intel_display *display) > > struct intel_display_irq_funcs { > void (*reset)(struct intel_display *display); > + void (*postinstall)(struct intel_display *display); > }; > > static const struct intel_display_irq_funcs gen11_display_irq_funcs = { > .reset = gen11_display_irq_reset, > + .postinstall = gen11_de_irq_postinstall, > }; > > static const struct intel_display_irq_funcs gen8_display_irq_funcs = { > .reset = gen8_display_irq_reset, > + .postinstall = gen8_de_irq_postinstall, > }; > > static const struct intel_display_irq_funcs vlv_display_irq_funcs = { > .reset = vlv_display_irq_reset, > + .postinstall = vlv_display_irq_postinstall, > }; > > static const struct intel_display_irq_funcs ilk_display_irq_funcs = { > .reset = ilk_display_irq_reset, > + .postinstall = ilk_de_irq_postinstall, > }; > > static const struct intel_display_irq_funcs i965_display_irq_funcs = { > .reset = i9xx_display_irq_reset, > + .postinstall = i965_display_irq_postinstall, > }; > > static const struct intel_display_irq_funcs i915_display_irq_funcs = { > .reset = i9xx_display_irq_reset, > + .postinstall = i915_display_irq_postinstall, > }; > > void intel_display_irq_reset(struct intel_display *display) > @@ -2479,6 +2483,14 @@ void intel_display_irq_reset(struct intel_display *display) > display->irq.funcs->reset(display); > } > > +void intel_display_irq_postinstall(struct intel_display *display) > +{ > + if (!HAS_DISPLAY(display)) > + return; > + > + display->irq.funcs->postinstall(display); > +} > + > void intel_display_irq_init(struct intel_display *display) > { > spin_lock_init(&display->irq.lock); > diff --git a/drivers/gpu/drm/i915/display/intel_display_irq.h b/drivers/gpu/drm/i915/display/intel_display_irq.h > index 21b2145656cd..fd9873ce9755 100644 > --- a/drivers/gpu/drm/i915/display/intel_display_irq.h > +++ b/drivers/gpu/drm/i915/display/intel_display_irq.h > @@ -59,14 +59,9 @@ u32 gen11_gu_misc_irq_ack(struct intel_display *display, const u32 master_ctl); > void gen11_gu_misc_irq_handler(struct intel_display *display, const u32 iir); > > void intel_display_irq_reset(struct intel_display *display); > +void intel_display_irq_postinstall(struct intel_display *display); > > u32 i9xx_display_irq_enable_mask(struct intel_display *display); > -void i915_display_irq_postinstall(struct intel_display *display); > -void i965_display_irq_postinstall(struct intel_display *display); > -void vlv_display_irq_postinstall(struct intel_display *display); > -void ilk_de_irq_postinstall(struct intel_display *display); > -void gen8_de_irq_postinstall(struct intel_display *display); > -void gen11_de_irq_postinstall(struct intel_display *display); > > u32 i915_pipestat_enable_mask(struct intel_display *display, enum pipe pipe); > void i915_enable_pipestat(struct intel_display *display, enum pipe pipe, u32 status_mask); > diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c > index c4f56a869910..c21b289b8007 100644 > --- a/drivers/gpu/drm/i915/i915_irq.c > +++ b/drivers/gpu/drm/i915/i915_irq.c > @@ -724,7 +724,7 @@ static void ilk_irq_postinstall(struct drm_i915_private *dev_priv) > > gen5_gt_irq_postinstall(to_gt(dev_priv)); > > - ilk_de_irq_postinstall(display); > + intel_display_irq_postinstall(display); > } > > static void valleyview_irq_postinstall(struct drm_i915_private *dev_priv) > @@ -733,7 +733,7 @@ static void valleyview_irq_postinstall(struct drm_i915_private *dev_priv) > > gen5_gt_irq_postinstall(to_gt(dev_priv)); > > - vlv_display_irq_postinstall(display); > + intel_display_irq_postinstall(display); > > intel_uncore_write(&dev_priv->uncore, VLV_MASTER_IER, MASTER_INTERRUPT_ENABLE); > intel_uncore_posting_read(&dev_priv->uncore, VLV_MASTER_IER); > @@ -744,7 +744,7 @@ static void gen8_irq_postinstall(struct drm_i915_private *dev_priv) > struct intel_display *display = dev_priv->display; > > gen8_gt_irq_postinstall(to_gt(dev_priv)); > - gen8_de_irq_postinstall(display); > + intel_display_irq_postinstall(display); > > gen8_master_intr_enable(intel_uncore_regs(&dev_priv->uncore)); > } > @@ -757,7 +757,7 @@ static void gen11_irq_postinstall(struct drm_i915_private *dev_priv) > u32 gu_misc_masked = GEN11_GU_MISC_GSE; > > gen11_gt_irq_postinstall(gt); > - gen11_de_irq_postinstall(display); > + intel_display_irq_postinstall(display); > > gen2_irq_init(uncore, GEN11_GU_MISC_IRQ_REGS, ~gu_misc_masked, gu_misc_masked); > > @@ -778,7 +778,7 @@ static void dg1_irq_postinstall(struct drm_i915_private *dev_priv) > > gen2_irq_init(uncore, GEN11_GU_MISC_IRQ_REGS, ~gu_misc_masked, gu_misc_masked); > > - gen11_de_irq_postinstall(display); > + intel_display_irq_postinstall(display); > > dg1_master_intr_enable(intel_uncore_regs(uncore)); > intel_uncore_posting_read(uncore, DG1_MSTR_TILE_INTR); > @@ -790,7 +790,7 @@ static void cherryview_irq_postinstall(struct drm_i915_private *dev_priv) > > gen8_gt_irq_postinstall(to_gt(dev_priv)); > > - vlv_display_irq_postinstall(display); > + intel_display_irq_postinstall(display); > > intel_uncore_write(&dev_priv->uncore, GEN8_MASTER_IRQ, GEN8_MASTER_IRQ_CONTROL); > intel_uncore_posting_read(&dev_priv->uncore, GEN8_MASTER_IRQ); > @@ -888,7 +888,7 @@ static void i915_irq_postinstall(struct drm_i915_private *dev_priv) > > gen2_irq_init(uncore, GEN2_IRQ_REGS, dev_priv->gen2_imr_mask, enable_mask); > > - i915_display_irq_postinstall(display); > + intel_display_irq_postinstall(display); > } > > static irqreturn_t i915_irq_handler(int irq, void *arg) > @@ -997,7 +997,7 @@ static void i965_irq_postinstall(struct drm_i915_private *dev_priv) > > gen2_irq_init(uncore, GEN2_IRQ_REGS, dev_priv->gen2_imr_mask, enable_mask); > > - i965_display_irq_postinstall(display); > + intel_display_irq_postinstall(display); > } > > static irqreturn_t i965_irq_handler(int irq, void *arg) > diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c > index ba3225878c61..62e5d38938eb 100644 > --- a/drivers/gpu/drm/xe/display/xe_display.c > +++ b/drivers/gpu/drm/xe/display/xe_display.c > @@ -246,7 +246,7 @@ void xe_display_irq_postinstall(struct xe_device *xe) > if (!xe->info.probe_display) > return; > > - gen11_de_irq_postinstall(display); > + intel_display_irq_postinstall(display); > } > > static bool suspend_to_idle(void) > -- > 2.47.3 -- Ville Syrjälä Intel