From: "Summers, Stuart" <stuart.summers@intel.com> To: "Roper, Matthew D" <matthew.d.roper@intel.com>, "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org> Cc: "Ceraolo Spurio, Daniele" <daniele.ceraolospurio@intel.com>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org> Subject: Re: [PATCH 02/11] drm/i915/pvc: Add forcewake support Date: Mon, 2 May 2022 22:33:53 +0000 [thread overview] Message-ID: <f3996e1efb75450f95908d8f9a05a4e16dc16f37.camel@intel.com> (raw) In-Reply-To: <20220502163417.2635462-3-matthew.d.roper@intel.com> On Mon, 2022-05-02 at 09:34 -0700, Matt Roper wrote: > Add PVC's forcewake ranges. > > Bspec: 67609 > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> > Signed-off-by: Matt Roper <matthew.d.roper@intel.com> > --- > drivers/gpu/drm/i915/intel_uncore.c | 150 > +++++++++++++++++- > drivers/gpu/drm/i915/selftests/intel_uncore.c | 2 + > 2 files changed, 151 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_uncore.c > b/drivers/gpu/drm/i915/intel_uncore.c > index 83517a703eb6..3352065635e8 100644 > --- a/drivers/gpu/drm/i915/intel_uncore.c > +++ b/drivers/gpu/drm/i915/intel_uncore.c > @@ -1080,6 +1080,45 @@ static const struct i915_range > dg2_shadowed_regs[] = { > { .start = 0x1F8510, .end = 0x1F8550 }, > }; > > +static const struct i915_range pvc_shadowed_regs[] = { > + { .start = 0x2030, .end = 0x2030 }, > + { .start = 0x2510, .end = 0x2550 }, > + { .start = 0xA008, .end = 0xA00C }, > + { .start = 0xA188, .end = 0xA188 }, > + { .start = 0xA278, .end = 0xA278 }, > + { .start = 0xA540, .end = 0xA56C }, > + { .start = 0xC4C8, .end = 0xC4C8 }, > + { .start = 0xC4E0, .end = 0xC4E0 }, > + { .start = 0xC600, .end = 0xC600 }, > + { .start = 0xC658, .end = 0xC658 }, > + { .start = 0x22030, .end = 0x22030 }, > + { .start = 0x22510, .end = 0x22550 }, > + { .start = 0x1C0030, .end = 0x1C0030 }, > + { .start = 0x1C0510, .end = 0x1C0550 }, > + { .start = 0x1C4030, .end = 0x1C4030 }, > + { .start = 0x1C4510, .end = 0x1C4550 }, > + { .start = 0x1C8030, .end = 0x1C8030 }, > + { .start = 0x1C8510, .end = 0x1C8550 }, > + { .start = 0x1D0030, .end = 0x1D0030 }, > + { .start = 0x1D0510, .end = 0x1D0550 }, > + { .start = 0x1D4030, .end = 0x1D4030 }, > + { .start = 0x1D4510, .end = 0x1D4550 }, > + { .start = 0x1D8030, .end = 0x1D8030 }, > + { .start = 0x1D8510, .end = 0x1D8550 }, > + { .start = 0x1E0030, .end = 0x1E0030 }, > + { .start = 0x1E0510, .end = 0x1E0550 }, > + { .start = 0x1E4030, .end = 0x1E4030 }, > + { .start = 0x1E4510, .end = 0x1E4550 }, > + { .start = 0x1E8030, .end = 0x1E8030 }, > + { .start = 0x1E8510, .end = 0x1E8550 }, > + { .start = 0x1F0030, .end = 0x1F0030 }, > + { .start = 0x1F0510, .end = 0x1F0550 }, > + { .start = 0x1F4030, .end = 0x1F4030 }, > + { .start = 0x1F4510, .end = 0x1F4550 }, > + { .start = 0x1F8030, .end = 0x1F8030 }, > + { .start = 0x1F8510, .end = 0x1F8550 }, > +}; > + > static int mmio_range_cmp(u32 key, const struct i915_range *range) > { > if (key < range->start) > @@ -1490,6 +1529,111 @@ static const struct intel_forcewake_range > __dg2_fw_ranges[] = { > XEHP_FWRANGES(FORCEWAKE_RENDER) > }; > > +/* > + * *Must* be sorted by offset ranges! See intel_fw_table_check(). > + * > + * Note that the spec lists several reserved/unused ranges that > don't actually > + * contain any registers. In the table below we'll combine those > reserved > + * ranges with either the preceding or following range to keep the > table small Looks like not just the reserved ranges are being used here. Maybe add "combine all ranges with preceding or following range with similar FW unit" or something similar. Thanks, Stuart > + * and lookups fast. > + */ > +static const struct intel_forcewake_range __pvc_fw_ranges[] = { > + GEN_FW_RANGE(0x0, 0xaff, 0), > + GEN_FW_RANGE(0xb00, 0xbff, FORCEWAKE_GT), > + GEN_FW_RANGE(0xc00, 0xfff, 0), > + GEN_FW_RANGE(0x1000, 0x1fff, FORCEWAKE_GT), > + GEN_FW_RANGE(0x2000, 0x26ff, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x2700, 0x2fff, FORCEWAKE_GT), > + GEN_FW_RANGE(0x3000, 0x3fff, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x4000, 0x813f, FORCEWAKE_GT), /* > + 0x4000 - 0x4aff: gt > + 0x4b00 - 0x4fff: reserved > + 0x5000 - 0x51ff: gt > + 0x5200 - 0x52ff: reserved > + 0x5300 - 0x53ff: gt > + 0x5400 - 0x7fff: reserved > + 0x8000 - 0x813f: gt */ > + GEN_FW_RANGE(0x8140, 0x817f, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x8180, 0x81ff, 0), > + GEN_FW_RANGE(0x8200, 0x94cf, FORCEWAKE_GT), /* > + 0x8200 - 0x82ff: gt > + 0x8300 - 0x84ff: reserved > + 0x8500 - 0x887f: gt > + 0x8880 - 0x8a7f: reserved > + 0x8a80 - 0x8aff: gt > + 0x8b00 - 0x8fff: reserved > + 0x9000 - 0x947f: gt > + 0x9480 - 0x94cf: reserved */ > + GEN_FW_RANGE(0x94d0, 0x955f, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x9560, 0x967f, 0), /* > + 0x9560 - 0x95ff: always on > + 0x9600 - 0x967f: reserved */ > + GEN_FW_RANGE(0x9680, 0x97ff, FORCEWAKE_RENDER), /* > + 0x9680 - 0x96ff: render > + 0x9700 - 0x97ff: reserved */ > + GEN_FW_RANGE(0x9800, 0xcfff, FORCEWAKE_GT), /* > + 0x9800 - 0xb4ff: gt > + 0xb500 - 0xbfff: reserved > + 0xc000 - 0xcfff: gt */ > + GEN_FW_RANGE(0xd000, 0xd3ff, 0), > + GEN_FW_RANGE(0xd400, 0xdbff, FORCEWAKE_GT), > + GEN_FW_RANGE(0xdc00, 0xdcff, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0xdd00, 0xde7f, FORCEWAKE_GT), /* > + 0xdd00 - 0xddff: gt > + 0xde00 - 0xde7f: reserved */ > + GEN_FW_RANGE(0xde80, 0xe8ff, FORCEWAKE_RENDER), /* > + 0xde80 - 0xdeff: render > + 0xdf00 - 0xe1ff: reserved > + 0xe200 - 0xe7ff: render > + 0xe800 - 0xe8ff: reserved */ > + GEN_FW_RANGE(0xe900, 0x11fff, FORCEWAKE_GT), /* > + 0xe900 - 0xe9ff: gt > + 0xea00 - 0xebff: reserved > + 0xec00 - 0xffff: gt > + 0x10000 - 0x11fff: reserved */ > + GEN_FW_RANGE(0x12000, 0x12fff, 0), /* > + 0x12000 - 0x127ff: always on > + 0x12800 - 0x12fff: reserved */ > + GEN_FW_RANGE(0x13000, 0x23fff, FORCEWAKE_GT), /* > + 0x13000 - 0x135ff: gt > + 0x13600 - 0x147ff: reserved > + 0x14800 - 0x153ff: gt > + 0x15400 - 0x19fff: reserved > + 0x1a000 - 0x1ffff: gt > + 0x20000 - 0x21fff: reserved > + 0x22000 - 0x23fff: gt */ > + GEN_FW_RANGE(0x24000, 0x2417f, 0), /* > + 24000 - 0x2407f: always on > + 24080 - 0x2417f: reserved */ > + GEN_FW_RANGE(0x24180, 0x3ffff, FORCEWAKE_GT), /* > + 0x24180 - 0x241ff: gt > + 0x24200 - 0x251ff: reserved > + 0x25200 - 0x252ff: gt > + 0x25300 - 0x25fff: reserved > + 0x26000 - 0x27fff: gt > + 0x28000 - 0x2ffff: reserved > + 0x30000 - 0x3ffff: gt */ > + GEN_FW_RANGE(0x40000, 0x1bffff, 0), > + GEN_FW_RANGE(0x1c0000, 0x1c3fff, FORCEWAKE_MEDIA_VDBOX0), /* > + 0x1c0000 - 0x1c2bff: VD0 > + 0x1c2c00 - 0x1c2cff: reserved > + 0x1c2d00 - 0x1c2dff: VD0 > + 0x1c2e00 - 0x1c3eff: reserved > + 0x1c3f00 - 0x1c3fff: VD0 */ > + GEN_FW_RANGE(0x1c4000, 0x1cffff, FORCEWAKE_MEDIA_VDBOX1), /* > + 0x1c4000 - 0x1c6aff: VD1 > + 0x1c6b00 - 0x1c7eff: reserved > + 0x1c7f00 - 0x1c7fff: VD1 > + 0x1c8000 - 0x1cffff: reserved */ > + GEN_FW_RANGE(0x1d0000, 0x23ffff, FORCEWAKE_MEDIA_VDBOX2), /* > + 0x1d0000 - 0x1d2aff: VD2 > + 0x1d2b00 - 0x1d3eff: reserved > + 0x1d3f00 - 0x1d3fff: VD2 > + 0x1d4000 - 0x23ffff: reserved */ > + GEN_FW_RANGE(0x240000, 0x3dffff, 0), > + GEN_FW_RANGE(0x3e0000, 0x3effff, FORCEWAKE_GT), > +}; > + > static void > ilk_dummy_write(struct intel_uncore *uncore) > { > @@ -2125,7 +2269,11 @@ static int uncore_forcewake_init(struct > intel_uncore *uncore) > > ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable); > > - if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55)) { > + if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 60)) { > + ASSIGN_FW_DOMAINS_TABLE(uncore, __pvc_fw_ranges); > + ASSIGN_SHADOW_TABLE(uncore, pvc_shadowed_regs); > + ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable); > + } else if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55)) { > ASSIGN_FW_DOMAINS_TABLE(uncore, __dg2_fw_ranges); > ASSIGN_SHADOW_TABLE(uncore, dg2_shadowed_regs); > ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable); > diff --git a/drivers/gpu/drm/i915/selftests/intel_uncore.c > b/drivers/gpu/drm/i915/selftests/intel_uncore.c > index cdd196783535..fda9bb79c049 100644 > --- a/drivers/gpu/drm/i915/selftests/intel_uncore.c > +++ b/drivers/gpu/drm/i915/selftests/intel_uncore.c > @@ -69,6 +69,7 @@ static int intel_shadow_table_check(void) > { gen11_shadowed_regs, ARRAY_SIZE(gen11_shadowed_regs) > }, > { gen12_shadowed_regs, ARRAY_SIZE(gen12_shadowed_regs) > }, > { dg2_shadowed_regs, ARRAY_SIZE(dg2_shadowed_regs) }, > + { pvc_shadowed_regs, ARRAY_SIZE(pvc_shadowed_regs) }, > }; > const struct i915_range *range; > unsigned int i, j; > @@ -115,6 +116,7 @@ int intel_uncore_mock_selftests(void) > { __gen11_fw_ranges, ARRAY_SIZE(__gen11_fw_ranges), > true }, > { __gen12_fw_ranges, ARRAY_SIZE(__gen12_fw_ranges), > true }, > { __xehp_fw_ranges, ARRAY_SIZE(__xehp_fw_ranges), true > }, > + { __pvc_fw_ranges, ARRAY_SIZE(__pvc_fw_ranges), true }, > }; > int err, i; >
WARNING: multiple messages have this Message-ID (diff)
From: "Summers, Stuart" <stuart.summers@intel.com> To: "Roper, Matthew D" <matthew.d.roper@intel.com>, "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org> Cc: "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org> Subject: Re: [Intel-gfx] [PATCH 02/11] drm/i915/pvc: Add forcewake support Date: Mon, 2 May 2022 22:33:53 +0000 [thread overview] Message-ID: <f3996e1efb75450f95908d8f9a05a4e16dc16f37.camel@intel.com> (raw) In-Reply-To: <20220502163417.2635462-3-matthew.d.roper@intel.com> On Mon, 2022-05-02 at 09:34 -0700, Matt Roper wrote: > Add PVC's forcewake ranges. > > Bspec: 67609 > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> > Signed-off-by: Matt Roper <matthew.d.roper@intel.com> > --- > drivers/gpu/drm/i915/intel_uncore.c | 150 > +++++++++++++++++- > drivers/gpu/drm/i915/selftests/intel_uncore.c | 2 + > 2 files changed, 151 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_uncore.c > b/drivers/gpu/drm/i915/intel_uncore.c > index 83517a703eb6..3352065635e8 100644 > --- a/drivers/gpu/drm/i915/intel_uncore.c > +++ b/drivers/gpu/drm/i915/intel_uncore.c > @@ -1080,6 +1080,45 @@ static const struct i915_range > dg2_shadowed_regs[] = { > { .start = 0x1F8510, .end = 0x1F8550 }, > }; > > +static const struct i915_range pvc_shadowed_regs[] = { > + { .start = 0x2030, .end = 0x2030 }, > + { .start = 0x2510, .end = 0x2550 }, > + { .start = 0xA008, .end = 0xA00C }, > + { .start = 0xA188, .end = 0xA188 }, > + { .start = 0xA278, .end = 0xA278 }, > + { .start = 0xA540, .end = 0xA56C }, > + { .start = 0xC4C8, .end = 0xC4C8 }, > + { .start = 0xC4E0, .end = 0xC4E0 }, > + { .start = 0xC600, .end = 0xC600 }, > + { .start = 0xC658, .end = 0xC658 }, > + { .start = 0x22030, .end = 0x22030 }, > + { .start = 0x22510, .end = 0x22550 }, > + { .start = 0x1C0030, .end = 0x1C0030 }, > + { .start = 0x1C0510, .end = 0x1C0550 }, > + { .start = 0x1C4030, .end = 0x1C4030 }, > + { .start = 0x1C4510, .end = 0x1C4550 }, > + { .start = 0x1C8030, .end = 0x1C8030 }, > + { .start = 0x1C8510, .end = 0x1C8550 }, > + { .start = 0x1D0030, .end = 0x1D0030 }, > + { .start = 0x1D0510, .end = 0x1D0550 }, > + { .start = 0x1D4030, .end = 0x1D4030 }, > + { .start = 0x1D4510, .end = 0x1D4550 }, > + { .start = 0x1D8030, .end = 0x1D8030 }, > + { .start = 0x1D8510, .end = 0x1D8550 }, > + { .start = 0x1E0030, .end = 0x1E0030 }, > + { .start = 0x1E0510, .end = 0x1E0550 }, > + { .start = 0x1E4030, .end = 0x1E4030 }, > + { .start = 0x1E4510, .end = 0x1E4550 }, > + { .start = 0x1E8030, .end = 0x1E8030 }, > + { .start = 0x1E8510, .end = 0x1E8550 }, > + { .start = 0x1F0030, .end = 0x1F0030 }, > + { .start = 0x1F0510, .end = 0x1F0550 }, > + { .start = 0x1F4030, .end = 0x1F4030 }, > + { .start = 0x1F4510, .end = 0x1F4550 }, > + { .start = 0x1F8030, .end = 0x1F8030 }, > + { .start = 0x1F8510, .end = 0x1F8550 }, > +}; > + > static int mmio_range_cmp(u32 key, const struct i915_range *range) > { > if (key < range->start) > @@ -1490,6 +1529,111 @@ static const struct intel_forcewake_range > __dg2_fw_ranges[] = { > XEHP_FWRANGES(FORCEWAKE_RENDER) > }; > > +/* > + * *Must* be sorted by offset ranges! See intel_fw_table_check(). > + * > + * Note that the spec lists several reserved/unused ranges that > don't actually > + * contain any registers. In the table below we'll combine those > reserved > + * ranges with either the preceding or following range to keep the > table small Looks like not just the reserved ranges are being used here. Maybe add "combine all ranges with preceding or following range with similar FW unit" or something similar. Thanks, Stuart > + * and lookups fast. > + */ > +static const struct intel_forcewake_range __pvc_fw_ranges[] = { > + GEN_FW_RANGE(0x0, 0xaff, 0), > + GEN_FW_RANGE(0xb00, 0xbff, FORCEWAKE_GT), > + GEN_FW_RANGE(0xc00, 0xfff, 0), > + GEN_FW_RANGE(0x1000, 0x1fff, FORCEWAKE_GT), > + GEN_FW_RANGE(0x2000, 0x26ff, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x2700, 0x2fff, FORCEWAKE_GT), > + GEN_FW_RANGE(0x3000, 0x3fff, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x4000, 0x813f, FORCEWAKE_GT), /* > + 0x4000 - 0x4aff: gt > + 0x4b00 - 0x4fff: reserved > + 0x5000 - 0x51ff: gt > + 0x5200 - 0x52ff: reserved > + 0x5300 - 0x53ff: gt > + 0x5400 - 0x7fff: reserved > + 0x8000 - 0x813f: gt */ > + GEN_FW_RANGE(0x8140, 0x817f, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x8180, 0x81ff, 0), > + GEN_FW_RANGE(0x8200, 0x94cf, FORCEWAKE_GT), /* > + 0x8200 - 0x82ff: gt > + 0x8300 - 0x84ff: reserved > + 0x8500 - 0x887f: gt > + 0x8880 - 0x8a7f: reserved > + 0x8a80 - 0x8aff: gt > + 0x8b00 - 0x8fff: reserved > + 0x9000 - 0x947f: gt > + 0x9480 - 0x94cf: reserved */ > + GEN_FW_RANGE(0x94d0, 0x955f, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0x9560, 0x967f, 0), /* > + 0x9560 - 0x95ff: always on > + 0x9600 - 0x967f: reserved */ > + GEN_FW_RANGE(0x9680, 0x97ff, FORCEWAKE_RENDER), /* > + 0x9680 - 0x96ff: render > + 0x9700 - 0x97ff: reserved */ > + GEN_FW_RANGE(0x9800, 0xcfff, FORCEWAKE_GT), /* > + 0x9800 - 0xb4ff: gt > + 0xb500 - 0xbfff: reserved > + 0xc000 - 0xcfff: gt */ > + GEN_FW_RANGE(0xd000, 0xd3ff, 0), > + GEN_FW_RANGE(0xd400, 0xdbff, FORCEWAKE_GT), > + GEN_FW_RANGE(0xdc00, 0xdcff, FORCEWAKE_RENDER), > + GEN_FW_RANGE(0xdd00, 0xde7f, FORCEWAKE_GT), /* > + 0xdd00 - 0xddff: gt > + 0xde00 - 0xde7f: reserved */ > + GEN_FW_RANGE(0xde80, 0xe8ff, FORCEWAKE_RENDER), /* > + 0xde80 - 0xdeff: render > + 0xdf00 - 0xe1ff: reserved > + 0xe200 - 0xe7ff: render > + 0xe800 - 0xe8ff: reserved */ > + GEN_FW_RANGE(0xe900, 0x11fff, FORCEWAKE_GT), /* > + 0xe900 - 0xe9ff: gt > + 0xea00 - 0xebff: reserved > + 0xec00 - 0xffff: gt > + 0x10000 - 0x11fff: reserved */ > + GEN_FW_RANGE(0x12000, 0x12fff, 0), /* > + 0x12000 - 0x127ff: always on > + 0x12800 - 0x12fff: reserved */ > + GEN_FW_RANGE(0x13000, 0x23fff, FORCEWAKE_GT), /* > + 0x13000 - 0x135ff: gt > + 0x13600 - 0x147ff: reserved > + 0x14800 - 0x153ff: gt > + 0x15400 - 0x19fff: reserved > + 0x1a000 - 0x1ffff: gt > + 0x20000 - 0x21fff: reserved > + 0x22000 - 0x23fff: gt */ > + GEN_FW_RANGE(0x24000, 0x2417f, 0), /* > + 24000 - 0x2407f: always on > + 24080 - 0x2417f: reserved */ > + GEN_FW_RANGE(0x24180, 0x3ffff, FORCEWAKE_GT), /* > + 0x24180 - 0x241ff: gt > + 0x24200 - 0x251ff: reserved > + 0x25200 - 0x252ff: gt > + 0x25300 - 0x25fff: reserved > + 0x26000 - 0x27fff: gt > + 0x28000 - 0x2ffff: reserved > + 0x30000 - 0x3ffff: gt */ > + GEN_FW_RANGE(0x40000, 0x1bffff, 0), > + GEN_FW_RANGE(0x1c0000, 0x1c3fff, FORCEWAKE_MEDIA_VDBOX0), /* > + 0x1c0000 - 0x1c2bff: VD0 > + 0x1c2c00 - 0x1c2cff: reserved > + 0x1c2d00 - 0x1c2dff: VD0 > + 0x1c2e00 - 0x1c3eff: reserved > + 0x1c3f00 - 0x1c3fff: VD0 */ > + GEN_FW_RANGE(0x1c4000, 0x1cffff, FORCEWAKE_MEDIA_VDBOX1), /* > + 0x1c4000 - 0x1c6aff: VD1 > + 0x1c6b00 - 0x1c7eff: reserved > + 0x1c7f00 - 0x1c7fff: VD1 > + 0x1c8000 - 0x1cffff: reserved */ > + GEN_FW_RANGE(0x1d0000, 0x23ffff, FORCEWAKE_MEDIA_VDBOX2), /* > + 0x1d0000 - 0x1d2aff: VD2 > + 0x1d2b00 - 0x1d3eff: reserved > + 0x1d3f00 - 0x1d3fff: VD2 > + 0x1d4000 - 0x23ffff: reserved */ > + GEN_FW_RANGE(0x240000, 0x3dffff, 0), > + GEN_FW_RANGE(0x3e0000, 0x3effff, FORCEWAKE_GT), > +}; > + > static void > ilk_dummy_write(struct intel_uncore *uncore) > { > @@ -2125,7 +2269,11 @@ static int uncore_forcewake_init(struct > intel_uncore *uncore) > > ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable); > > - if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55)) { > + if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 60)) { > + ASSIGN_FW_DOMAINS_TABLE(uncore, __pvc_fw_ranges); > + ASSIGN_SHADOW_TABLE(uncore, pvc_shadowed_regs); > + ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable); > + } else if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55)) { > ASSIGN_FW_DOMAINS_TABLE(uncore, __dg2_fw_ranges); > ASSIGN_SHADOW_TABLE(uncore, dg2_shadowed_regs); > ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable); > diff --git a/drivers/gpu/drm/i915/selftests/intel_uncore.c > b/drivers/gpu/drm/i915/selftests/intel_uncore.c > index cdd196783535..fda9bb79c049 100644 > --- a/drivers/gpu/drm/i915/selftests/intel_uncore.c > +++ b/drivers/gpu/drm/i915/selftests/intel_uncore.c > @@ -69,6 +69,7 @@ static int intel_shadow_table_check(void) > { gen11_shadowed_regs, ARRAY_SIZE(gen11_shadowed_regs) > }, > { gen12_shadowed_regs, ARRAY_SIZE(gen12_shadowed_regs) > }, > { dg2_shadowed_regs, ARRAY_SIZE(dg2_shadowed_regs) }, > + { pvc_shadowed_regs, ARRAY_SIZE(pvc_shadowed_regs) }, > }; > const struct i915_range *range; > unsigned int i, j; > @@ -115,6 +116,7 @@ int intel_uncore_mock_selftests(void) > { __gen11_fw_ranges, ARRAY_SIZE(__gen11_fw_ranges), > true }, > { __gen12_fw_ranges, ARRAY_SIZE(__gen12_fw_ranges), > true }, > { __xehp_fw_ranges, ARRAY_SIZE(__xehp_fw_ranges), true > }, > + { __pvc_fw_ranges, ARRAY_SIZE(__pvc_fw_ranges), true }, > }; > int err, i; >
next prev parent reply other threads:[~2022-05-02 22:34 UTC|newest] Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-05-02 16:34 [PATCH 00/11] i915: Introduce Ponte Vecchio Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 16:34 ` [PATCH 01/11] drm/i915/pvc: add initial Ponte Vecchio definitions Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 20:44 ` Lucas De Marchi 2022-05-02 20:44 ` [Intel-gfx] " Lucas De Marchi 2022-05-02 16:34 ` [PATCH 02/11] drm/i915/pvc: Add forcewake support Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 22:33 ` Summers, Stuart [this message] 2022-05-02 22:33 ` Summers, Stuart 2022-05-05 0:34 ` Matt Roper 2022-05-05 0:34 ` Matt Roper 2022-05-02 16:34 ` [PATCH 03/11] drm/i915/pvc: Define MOCS table for PVC Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 16:50 ` Matt Roper 2022-05-02 16:50 ` [Intel-gfx] " Matt Roper 2022-05-02 18:39 ` Lucas De Marchi 2022-05-02 18:50 ` Matt Roper 2022-05-02 19:27 ` Lucas De Marchi 2022-05-02 19:42 ` Matt Roper 2022-05-02 21:03 ` Lucas De Marchi 2022-05-02 21:03 ` [Intel-gfx] " Lucas De Marchi 2022-05-02 21:14 ` Matt Roper 2022-05-02 21:14 ` [Intel-gfx] " Matt Roper 2022-05-03 6:22 ` Lucas De Marchi 2022-05-03 6:22 ` [Intel-gfx] " Lucas De Marchi 2022-05-02 16:34 ` [PATCH 04/11] drm/i915/pvc: Read correct RP_STATE_CAP register Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 16:55 ` Rodrigo Vivi 2022-05-02 16:55 ` [Intel-gfx] " Rodrigo Vivi 2022-05-02 16:34 ` [PATCH 05/11] drm/i915/pvc: Remove additional 3D flags from PIPE_CONTROL Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 16:34 ` [PATCH 06/11] drm/i915/pvc: Reduce stack usage in reset selftest with extra blitter engine Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 18:46 ` Souza, Jose 2022-05-02 18:46 ` [Intel-gfx] " Souza, Jose 2022-05-03 8:25 ` Tvrtko Ursulin 2022-05-02 16:34 ` [PATCH 07/11] drm/i915/pvc: Engines definitions for new copy engines Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 18:45 ` Souza, Jose 2022-05-03 8:05 ` Tvrtko Ursulin 2022-05-03 8:05 ` Tvrtko Ursulin 2022-05-05 20:59 ` Matt Roper 2022-05-05 20:59 ` Matt Roper 2022-05-06 7:21 ` Tvrtko Ursulin 2022-05-06 7:21 ` Tvrtko Ursulin 2022-05-06 14:29 ` Matt Roper 2022-05-06 14:29 ` Matt Roper 2022-05-02 16:34 ` [PATCH 08/11] drm/i915/pvc: Interrupt support " Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 22:23 ` Summers, Stuart 2022-05-02 22:23 ` [Intel-gfx] " Summers, Stuart 2022-05-02 16:34 ` [PATCH 09/11] drm/i915/pvc: Reset " Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 18:44 ` Souza, Jose 2022-05-02 22:23 ` Summers, Stuart 2022-05-02 16:34 ` [PATCH 10/11] drm/i915/pvc: skip all copy engines from aux table invalidate Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 18:40 ` Souza, Jose 2022-05-02 22:58 ` Kumar Valsan, Prathap 2022-05-02 22:58 ` Kumar Valsan, Prathap 2022-05-02 16:34 ` [PATCH 11/11] drm/i915/pvc: read fuses for link copy engines Matt Roper 2022-05-02 16:34 ` [Intel-gfx] " Matt Roper 2022-05-02 18:48 ` Souza, Jose 2022-05-02 18:48 ` [Intel-gfx] " Souza, Jose 2022-05-03 8:19 ` Tvrtko Ursulin 2022-05-02 16:58 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for i915: Introduce Ponte Vecchio Patchwork 2022-05-02 16:58 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork 2022-05-02 17:22 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2022-05-02 22:58 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 2022-05-03 17:32 ` Matt Roper 2022-05-04 17:03 ` Vudum, Lakshminarayana 2022-05-03 8:21 ` [Intel-gfx] [PATCH 00/11] " Tvrtko Ursulin 2022-05-03 14:56 ` Matt Roper 2022-05-03 15:01 ` Tvrtko Ursulin 2022-05-04 16:22 ` [Intel-gfx] ✓ Fi.CI.IGT: success for " Patchwork 2022-05-04 16:43 ` Patchwork
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=f3996e1efb75450f95908d8f9a05a4e16dc16f37.camel@intel.com \ --to=stuart.summers@intel.com \ --cc=daniele.ceraolospurio@intel.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=intel-gfx@lists.freedesktop.org \ --cc=matthew.d.roper@intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.