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 X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C2FCDC433E0 for ; Mon, 11 Jan 2021 21:26:04 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 6159522BED for ; Mon, 11 Jan 2021 21:26:04 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6159522BED Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=intel-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 8EA248994A; Mon, 11 Jan 2021 21:26:03 +0000 (UTC) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTPS id 09A9C898FD for ; Mon, 11 Jan 2021 21:26:01 +0000 (UTC) IronPort-SDR: O0inYptRz5o0UDw/oyYD9BY5d4gI4Rs3r0Px6oZ5OkF6wwSLxBoaRAqYdH7qQ6IWzYzhAyNw46 uO69jppZulSA== X-IronPort-AV: E=McAfee;i="6000,8403,9861"; a="174429159" X-IronPort-AV: E=Sophos;i="5.79,339,1602572400"; d="scan'208";a="174429159" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jan 2021 13:25:59 -0800 IronPort-SDR: ADqzXzKhJnmHnNyAy2krguNWL9X3J4eM1TfjXCPM53JQT/Z2ix1wfwaNYw09dc/n1j0969NXjH JQ3aGcwqp5lg== X-IronPort-AV: E=Sophos;i="5.79,339,1602572400"; d="scan'208";a="363274084" Received: from jraghubi-mobl.amr.corp.intel.com (HELO ldmartin-desk1) ([10.212.23.93]) by orsmga002-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jan 2021 13:25:57 -0800 Date: Mon, 11 Jan 2021 13:25:53 -0800 From: Lucas De Marchi To: Matt Roper Message-ID: <20210111212553.brclyuex7dgzeryu@ldmartin-desk1> References: <20210108231853.2859646-1-aditya.swarup@intel.com> <20210108231853.2859646-2-aditya.swarup@intel.com> <20210108234440.GO3894148@mdroper-desk1.amr.corp.intel.com> <878s8zw7tw.fsf@intel.com> <875z43w7kq.fsf@intel.com> <20210111205743.GC4758@mdroper-desk1.amr.corp.intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210111205743.GC4758@mdroper-desk1.amr.corp.intel.com> Subject: Re: [Intel-gfx] [PATCH 1/2] drm/i915/tgl: Use TGL stepping info for applying WAs 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: , Cc: intel-gfx@lists.freedesktop.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1"; Format="flowed" Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Mon, Jan 11, 2021 at 12:57:43PM -0800, Matt Roper wrote: >On Mon, Jan 11, 2021 at 10:18:45PM +0200, Jani Nikula wrote: >> On Mon, 11 Jan 2021, Jani Nikula wrote: >> > On Fri, 08 Jan 2021, Matt Roper wrote: >> >> On Fri, Jan 08, 2021 at 03:18:52PM -0800, Aditya Swarup wrote: >> >>> TGL adds another level of indirection for applying WA based on stepp= ing >> >>> information rather than PCI REVID. So change TGL_REVID enum into >> >>> stepping enum and use PCI REVID as index into revid to stepping tabl= e to >> >>> fetch correct display and GT stepping for application of WAs as >> >>> suggested by Matt Roper. >> >> >> >> So to clarify the goal is to rename "revid" -> "stepping" because the >> >> values like "A1," "C0," etc. are't the actual PCI revision ID, but >> >> rather descriptions of the stepping of a given IP block; the enum val= ues >> >> we use to represent those are arbitrary and don't matter as long as >> >> they're monotonically increasing for comparisons. The PCI revision ID >> >> is just the input we use today to deduce what the IP steppings are, a= nd >> >> there's talk that we could determine the IP steppings in a different = way >> >> at some point in the future. >> >> >> >> Furthermore, since the same scheme will be used at least for ADL-S, we >> >> should drop the "TGL" prefix since there's no need to name these gene= ral >> >> enum values in a platform-specific manner. >> >> >> >> Reviewed-by: Matt Roper >> >> >> >> We should probably make the same kind of change to KBL (and use the s= ame >> >> stepping enum) too since it has the same kind of extra indirection as >> >> TGL/ADL-S, but we can do that as a followup patch. >> > >> > FWIW I have a wip series changing the whole thing to abstract steppings >> > enums that are shared between platforms, but it's in a bit of limbo >> > because the previous revid changes were applied to drm-intel-gt-next, >> > and it's fallen pretty far out of sync with drm-intel-next. All of this >> > really belongs to drm-intel-next, but can't do that until the branches >> > sync up again. >> >> Btw this series doesn't apply to drm-intel-next either, for the same >> reason, and the ADL-S platform definition and PCI IDs must *not* be >> applied to drm-intel-gt-next. > >So to clarify, it looks like we have a bunch of revid changes to the >display code that got merged to the gt-next tree but not to the >intel-next tree? Should we be going back and also merging / >cherry-picking those over to intel-next since that's where the display >changes are supposed to go, or is it too late to do that cleanly at this >point? it was my mistake to merge them to drm-intel-gt-next. They should have been in drm-intel-next. > >Going forward, what should the general strategy be for stuff like >platform definitions and such? Merge such enablement patches to both last time we talked about this was regarding dg1 AFAIR and the consensus was to create a topic branch and that topic branch to be merged in both branches. That would avoid having 2 commits in different branches. Not sure if it would work out nicely for getting test on CI though. Since the changes are spread through the codebase, we could very easily hit a situation that this topic branch creates conflicts for other patches getting merged on either drm-intel-next or drm-intel-gt-next. +Joonas, +Rodrigo Lucas De Marchi >intel-next and gt-next at the same time so that the basic definitions >are available to both trees? It seems like the whole split into two >trees really isn't working well and is just leading to more mistakes and >bottlenecks. What benefit are we supposed to be getting from this >split? > > >Matt > > >> >> BR, >> Jani. >> >> > >> > My series also completely hides the arrays into a separate .c file, >> > because the externs with direct array access are turning into >> > nightmare. The ARRAY_SIZE() checks rely on the extern declaration and >> > the actual array definition to have the sizes in sync, but the compiler >> > does not check that. Really. >> > >> > IDK, feels like this merging this series is going to be extra churn. >> > >> > >> > BR, >> > Jani. >> > >> > >> >> >> >> >> >> Matt >> >> >> >>> >> >>> Cc: Matt Roper >> >>> Cc: Lucas De Marchi >> >>> Cc: Jos=E9 Roberto de Souza >> >>> Signed-off-by: Aditya Swarup >> >>> --- >> >>> .../drm/i915/display/intel_display_power.c | 2 +- >> >>> drivers/gpu/drm/i915/display/intel_psr.c | 4 +- >> >>> drivers/gpu/drm/i915/display/intel_sprite.c | 2 +- >> >>> drivers/gpu/drm/i915/gt/intel_workarounds.c | 26 +++++----- >> >>> drivers/gpu/drm/i915/i915_drv.h | 50 +++++++++-------= --- >> >>> drivers/gpu/drm/i915/intel_pm.c | 2 +- >> >>> 6 files changed, 43 insertions(+), 43 deletions(-) >> >>> >> >>> diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/dr= ivers/gpu/drm/i915/display/intel_display_power.c >> >>> index d52374f01316..bb04b502a442 100644 >> >>> --- a/drivers/gpu/drm/i915/display/intel_display_power.c >> >>> +++ b/drivers/gpu/drm/i915/display/intel_display_power.c >> >>> @@ -5340,7 +5340,7 @@ static void tgl_bw_buddy_init(struct drm_i915_= private *dev_priv) >> >>> int config, i; >> >>> >> >>> if (IS_DG1_REVID(dev_priv, DG1_REVID_A0, DG1_REVID_A0) || >> >>> - IS_TGL_DISP_REVID(dev_priv, TGL_REVID_A0, TGL_REVID_B0)) >> >>> + IS_TGL_DISP_STEPPING(dev_priv, STEP_A0, STEP_B0)) >> >>> /* Wa_1409767108:tgl,dg1 */ >> >>> table =3D wa_1409767108_buddy_page_masks; >> >>> else >> >>> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/= drm/i915/display/intel_psr.c >> >>> index c24ae69426cf..a93717178957 100644 >> >>> --- a/drivers/gpu/drm/i915/display/intel_psr.c >> >>> +++ b/drivers/gpu/drm/i915/display/intel_psr.c >> >>> @@ -550,7 +550,7 @@ static void hsw_activate_psr2(struct intel_dp *i= ntel_dp) >> >>> >> >>> if (dev_priv->psr.psr2_sel_fetch_enabled) { >> >>> /* WA 1408330847 */ >> >>> - if (IS_TGL_DISP_REVID(dev_priv, TGL_REVID_A0, TGL_REVID_A0) || >> >>> + if (IS_TGL_DISP_STEPPING(dev_priv, STEP_A0, STEP_A0) || >> >>> IS_RKL_REVID(dev_priv, RKL_REVID_A0, RKL_REVID_A0)) >> >>> intel_de_rmw(dev_priv, CHICKEN_PAR1_1, >> >>> DIS_RAM_BYPASS_PSR2_MAN_TRACK, >> >>> @@ -1102,7 +1102,7 @@ static void intel_psr_disable_locked(struct in= tel_dp *intel_dp) >> >>> >> >>> /* WA 1408330847 */ >> >>> if (dev_priv->psr.psr2_sel_fetch_enabled && >> >>> - (IS_TGL_DISP_REVID(dev_priv, TGL_REVID_A0, TGL_REVID_A0) || >> >>> + (IS_TGL_DISP_STEPPING(dev_priv, STEP_A0, STEP_A0) || >> >>> IS_RKL_REVID(dev_priv, RKL_REVID_A0, RKL_REVID_A0))) >> >>> intel_de_rmw(dev_priv, CHICKEN_PAR1_1, >> >>> DIS_RAM_BYPASS_PSR2_MAN_TRACK, 0); >> >>> diff --git a/drivers/gpu/drm/i915/display/intel_sprite.c b/drivers/g= pu/drm/i915/display/intel_sprite.c >> >>> index cf3589fd0ddb..4ce32df3855f 100644 >> >>> --- a/drivers/gpu/drm/i915/display/intel_sprite.c >> >>> +++ b/drivers/gpu/drm/i915/display/intel_sprite.c >> >>> @@ -3033,7 +3033,7 @@ static bool gen12_plane_supports_mc_ccs(struct= drm_i915_private *dev_priv, >> >>> { >> >>> /* Wa_14010477008:tgl[a0..c0],rkl[all],dg1[all] */ >> >>> if (IS_DG1(dev_priv) || IS_ROCKETLAKE(dev_priv) || >> >>> - IS_TGL_DISP_REVID(dev_priv, TGL_REVID_A0, TGL_REVID_C0)) >> >>> + IS_TGL_DISP_STEPPING(dev_priv, STEP_A0, STEP_C0)) >> >>> return false; >> >>> >> >>> return plane_id < PLANE_SPRITE4; >> >>> diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/g= pu/drm/i915/gt/intel_workarounds.c >> >>> index c21a9726326a..111d01e2f81e 100644 >> >>> --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c >> >>> +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c >> >>> @@ -71,17 +71,17 @@ const struct i915_rev_steppings kbl_revids[] =3D= { >> >>> [7] =3D { .gt_stepping =3D KBL_REVID_G0, .disp_stepping =3D KBL_RE= VID_C0 }, >> >>> }; >> >>> >> >>> -const struct i915_rev_steppings tgl_uy_revids[] =3D { >> >>> - [0] =3D { .gt_stepping =3D TGL_REVID_A0, .disp_stepping =3D TGL_RE= VID_A0 }, >> >>> - [1] =3D { .gt_stepping =3D TGL_REVID_B0, .disp_stepping =3D TGL_RE= VID_C0 }, >> >>> - [2] =3D { .gt_stepping =3D TGL_REVID_B1, .disp_stepping =3D TGL_RE= VID_C0 }, >> >>> - [3] =3D { .gt_stepping =3D TGL_REVID_C0, .disp_stepping =3D TGL_RE= VID_D0 }, >> >>> +const struct i915_rev_steppings tgl_uy_revid_step_tbl[] =3D { >> >>> + [0] =3D { .gt_stepping =3D STEP_A0, .disp_stepping =3D STEP_A0 }, >> >>> + [1] =3D { .gt_stepping =3D STEP_B0, .disp_stepping =3D STEP_C0 }, >> >>> + [2] =3D { .gt_stepping =3D STEP_B1, .disp_stepping =3D STEP_C0 }, >> >>> + [3] =3D { .gt_stepping =3D STEP_C0, .disp_stepping =3D STEP_D0 }, >> >>> }; >> >>> >> >>> /* Same GT stepping between tgl_uy_revids and tgl_revids don't mean= the same HW */ >> >>> -const struct i915_rev_steppings tgl_revids[] =3D { >> >>> - [0] =3D { .gt_stepping =3D TGL_REVID_A0, .disp_stepping =3D TGL_RE= VID_B0 }, >> >>> - [1] =3D { .gt_stepping =3D TGL_REVID_B0, .disp_stepping =3D TGL_RE= VID_D0 }, >> >>> +const struct i915_rev_steppings tgl_revid_step_tbl[] =3D { >> >>> + [0] =3D { .gt_stepping =3D STEP_A0, .disp_stepping =3D STEP_B0 }, >> >>> + [1] =3D { .gt_stepping =3D STEP_B0, .disp_stepping =3D STEP_D0 }, >> >>> }; >> >>> >> >>> static void wa_init_start(struct i915_wa_list *wal, const char *nam= e, const char *engine_name) >> >>> @@ -1211,19 +1211,19 @@ tgl_gt_workarounds_init(struct drm_i915_priv= ate *i915, struct i915_wa_list *wal) >> >>> gen12_gt_workarounds_init(i915, wal); >> >>> >> >>> /* Wa_1409420604:tgl */ >> >>> - if (IS_TGL_UY_GT_REVID(i915, TGL_REVID_A0, TGL_REVID_A0)) >> >>> + if (IS_TGL_UY_GT_STEPPING(i915, STEP_A0, STEP_A0)) >> >>> wa_write_or(wal, >> >>> SUBSLICE_UNIT_LEVEL_CLKGATE2, >> >>> CPSSUNIT_CLKGATE_DIS); >> >>> >> >>> /* Wa_1607087056:tgl also know as BUG:1409180338 */ >> >>> - if (IS_TGL_UY_GT_REVID(i915, TGL_REVID_A0, TGL_REVID_A0)) >> >>> + if (IS_TGL_UY_GT_STEPPING(i915, STEP_A0, STEP_A0)) >> >>> wa_write_or(wal, >> >>> SLICE_UNIT_LEVEL_CLKGATE, >> >>> L3_CLKGATE_DIS | L3_CR2X_CLKGATE_DIS); >> >>> >> >>> /* Wa_1408615072:tgl[a0] */ >> >>> - if (IS_TGL_UY_GT_REVID(i915, TGL_REVID_A0, TGL_REVID_A0)) >> >>> + if (IS_TGL_UY_GT_STEPPING(i915, STEP_A0, STEP_A0)) >> >>> wa_write_or(wal, UNSLICE_UNIT_LEVEL_CLKGATE2, >> >>> VSUNIT_CLKGATE_DIS_TGL); >> >>> } >> >>> @@ -1700,7 +1700,7 @@ rcs_engine_wa_init(struct intel_engine_cs *eng= ine, struct i915_wa_list *wal) >> >>> struct drm_i915_private *i915 =3D engine->i915; >> >>> >> >>> if (IS_DG1_REVID(i915, DG1_REVID_A0, DG1_REVID_A0) || >> >>> - IS_TGL_UY_GT_REVID(i915, TGL_REVID_A0, TGL_REVID_A0)) { >> >>> + IS_TGL_UY_GT_STEPPING(i915, STEP_A0, STEP_A0)) { >> >>> /* >> >>> * Wa_1607138336:tgl[a0],dg1[a0] >> >>> * Wa_1607063988:tgl[a0],dg1[a0] >> >>> @@ -1710,7 +1710,7 @@ rcs_engine_wa_init(struct intel_engine_cs *eng= ine, struct i915_wa_list *wal) >> >>> GEN12_DISABLE_POSH_BUSY_FF_DOP_CG); >> >>> } >> >>> >> >>> - if (IS_TGL_UY_GT_REVID(i915, TGL_REVID_A0, TGL_REVID_A0)) { >> >>> + if (IS_TGL_UY_GT_STEPPING(i915, STEP_A0, STEP_A0)) { >> >>> /* >> >>> * Wa_1606679103:tgl >> >>> * (see also Wa_1606682166:icl) >> >>> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/= i915_drv.h >> >>> index 5e5bcef20e33..11d6e8abde46 100644 >> >>> --- a/drivers/gpu/drm/i915/i915_drv.h >> >>> +++ b/drivers/gpu/drm/i915/i915_drv.h >> >>> @@ -1559,54 +1559,54 @@ extern const struct i915_rev_steppings kbl_r= evids[]; >> >>> (IS_JSL_EHL(p) && IS_REVID(p, since, until)) >> >>> >> >>> enum { >> >>> - TGL_REVID_A0, >> >>> - TGL_REVID_B0, >> >>> - TGL_REVID_B1, >> >>> - TGL_REVID_C0, >> >>> - TGL_REVID_D0, >> >>> + STEP_A0, >> >>> + STEP_B0, >> >>> + STEP_B1, >> >>> + STEP_C0, >> >>> + STEP_D0, >> >>> }; >> >>> >> >>> -#define TGL_UY_REVIDS_SIZE 4 >> >>> -#define TGL_REVIDS_SIZE 2 >> >>> +#define TGL_UY_REVID_STEP_TBL_SIZE 4 >> >>> +#define TGL_REVID_STEP_TBL_SIZE 2 >> >>> >> >>> -extern const struct i915_rev_steppings tgl_uy_revids[TGL_UY_REVIDS_= SIZE]; >> >>> -extern const struct i915_rev_steppings tgl_revids[TGL_REVIDS_SIZE]; >> >>> +extern const struct i915_rev_steppings tgl_uy_revid_step_tbl[TGL_UY= _REVID_STEP_TBL_SIZE]; >> >>> +extern const struct i915_rev_steppings tgl_revid_step_tbl[TGL_REVID= _STEP_TBL_SIZE]; >> >>> >> >>> static inline const struct i915_rev_steppings * >> >>> -tgl_revids_get(struct drm_i915_private *dev_priv) >> >>> +tgl_stepping_get(struct drm_i915_private *dev_priv) >> >>> { >> >>> u8 revid =3D INTEL_REVID(dev_priv); >> >>> u8 size; >> >>> - const struct i915_rev_steppings *tgl_revid_tbl; >> >>> + const struct i915_rev_steppings *revid_step_tbl; >> >>> >> >>> if (IS_TGL_U(dev_priv) || IS_TGL_Y(dev_priv)) { >> >>> - tgl_revid_tbl =3D tgl_uy_revids; >> >>> - size =3D ARRAY_SIZE(tgl_uy_revids); >> >>> + revid_step_tbl =3D tgl_uy_revid_step_tbl; >> >>> + size =3D ARRAY_SIZE(tgl_uy_revid_step_tbl); >> >>> } else { >> >>> - tgl_revid_tbl =3D tgl_revids; >> >>> - size =3D ARRAY_SIZE(tgl_revids); >> >>> + revid_step_tbl =3D tgl_revid_step_tbl; >> >>> + size =3D ARRAY_SIZE(tgl_revid_step_tbl); >> >>> } >> >>> >> >>> revid =3D min_t(u8, revid, size - 1); >> >>> >> >>> - return &tgl_revid_tbl[revid]; >> >>> + return &revid_step_tbl[revid]; >> >>> } >> >>> >> >>> -#define IS_TGL_DISP_REVID(p, since, until) \ >> >>> +#define IS_TGL_DISP_STEPPING(p, since, until) \ >> >>> (IS_TIGERLAKE(p) && \ >> >>> - tgl_revids_get(p)->disp_stepping >=3D (since) && \ >> >>> - tgl_revids_get(p)->disp_stepping <=3D (until)) >> >>> + tgl_stepping_get(p)->disp_stepping >=3D (since) && \ >> >>> + tgl_stepping_get(p)->disp_stepping <=3D (until)) >> >>> >> >>> -#define IS_TGL_UY_GT_REVID(p, since, until) \ >> >>> +#define IS_TGL_UY_GT_STEPPING(p, since, until) \ >> >>> ((IS_TGL_U(p) || IS_TGL_Y(p)) && \ >> >>> - tgl_revids_get(p)->gt_stepping >=3D (since) && \ >> >>> - tgl_revids_get(p)->gt_stepping <=3D (until)) >> >>> + tgl_stepping_get(p)->gt_stepping >=3D (since) && \ >> >>> + tgl_stepping_get(p)->gt_stepping <=3D (until)) >> >>> >> >>> -#define IS_TGL_GT_REVID(p, since, until) \ >> >>> +#define IS_TGL_GT_STEPPING(p, since, until) \ >> >>> (IS_TIGERLAKE(p) && \ >> >>> !(IS_TGL_U(p) || IS_TGL_Y(p)) && \ >> >>> - tgl_revids_get(p)->gt_stepping >=3D (since) && \ >> >>> - tgl_revids_get(p)->gt_stepping <=3D (until)) >> >>> + tgl_stepping_get(p)->gt_stepping >=3D (since) && \ >> >>> + tgl_stepping_get(p)->gt_stepping <=3D (until)) >> >>> >> >>> #define RKL_REVID_A0 0x0 >> >>> #define RKL_REVID_B0 0x1 >> >>> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/= intel_pm.c >> >>> index bbc73df7f753..319acca2630b 100644 >> >>> --- a/drivers/gpu/drm/i915/intel_pm.c >> >>> +++ b/drivers/gpu/drm/i915/intel_pm.c >> >>> @@ -7110,7 +7110,7 @@ static void tgl_init_clock_gating(struct drm_i= 915_private *dev_priv) >> >>> ILK_DPFC_CHICKEN_COMP_DUMMY_PIXEL); >> >>> >> >>> /* Wa_1409825376:tgl (pre-prod)*/ >> >>> - if (IS_TGL_DISP_REVID(dev_priv, TGL_REVID_A0, TGL_REVID_B1)) >> >>> + if (IS_TGL_DISP_STEPPING(dev_priv, STEP_A0, STEP_B1)) >> >>> intel_uncore_write(&dev_priv->uncore, GEN9_CLKGATE_DIS_3, intel_u= ncore_read(&dev_priv->uncore, GEN9_CLKGATE_DIS_3) | >> >>> TGL_VRH_GATING_DIS); >> >>> >> >>> -- >> >>> 2.27.0 >> >>> >> >> -- >> Jani Nikula, Intel Open Source Graphics Center > >-- = >Matt Roper >Graphics Software Engineer >VTT-OSGC Platform Enablement >Intel Corporation >(916) 356-2795 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx