Skip to content

Commit 08783aa

Browse files
vsyrjalatursulin
authored andcommitted
drm/i915: s/JSP2/ICP2/ PCH
This JSP2 PCH actually seems to be some special Apple specific ICP variant rather than a JSP. Make it so. Or at least all the references to it seem to be some Apple ICL machines. Didn't manage to find these PCI IDs in any public chipset docs unfortunately. The only thing we're losing here with this JSP->ICP change is Wa_14011294188, but based on the HSD that isn't actually needed on any ICP based design (including JSP), only TGP based stuff (including MCC) really need it. The documented w/a just never made that distinction because Windows didn't want to differentiate between JSP and MCC (not sure how they handle hpd/ddc/etc. then though...). Cc: stable@vger.kernel.org Cc: Matt Roper <matthew.d.roper@intel.com> Cc: Vivek Kasireddy <vivek.kasireddy@intel.com> Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/4226 Fixes: 943682e ("drm/i915: Introduce Jasper Lake PCH") Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20220224132142.12927-1-ville.syrjala@linux.intel.com Acked-by: Vivek Kasireddy <vivek.kasireddy@intel.com> Tested-by: Tomas Bzatek <bugs@bzatek.net> (cherry picked from commit 5358150) Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
1 parent 1b279f6 commit 08783aa

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

drivers/gpu/drm/i915/intel_pch.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,7 @@ intel_pch_type(const struct drm_i915_private *dev_priv, unsigned short id)
108108
/* Comet Lake V PCH is based on KBP, which is SPT compatible */
109109
return PCH_SPT;
110110
case INTEL_PCH_ICP_DEVICE_ID_TYPE:
111+
case INTEL_PCH_ICP2_DEVICE_ID_TYPE:
111112
drm_dbg_kms(&dev_priv->drm, "Found Ice Lake PCH\n");
112113
drm_WARN_ON(&dev_priv->drm, !IS_ICELAKE(dev_priv));
113114
return PCH_ICP;
@@ -123,7 +124,6 @@ intel_pch_type(const struct drm_i915_private *dev_priv, unsigned short id)
123124
!IS_GEN9_BC(dev_priv));
124125
return PCH_TGP;
125126
case INTEL_PCH_JSP_DEVICE_ID_TYPE:
126-
case INTEL_PCH_JSP2_DEVICE_ID_TYPE:
127127
drm_dbg_kms(&dev_priv->drm, "Found Jasper Lake PCH\n");
128128
drm_WARN_ON(&dev_priv->drm, !IS_JSL_EHL(dev_priv));
129129
return PCH_JSP;

drivers/gpu/drm/i915/intel_pch.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,11 @@ enum intel_pch {
5050
#define INTEL_PCH_CMP2_DEVICE_ID_TYPE 0x0680
5151
#define INTEL_PCH_CMP_V_DEVICE_ID_TYPE 0xA380
5252
#define INTEL_PCH_ICP_DEVICE_ID_TYPE 0x3480
53+
#define INTEL_PCH_ICP2_DEVICE_ID_TYPE 0x3880
5354
#define INTEL_PCH_MCC_DEVICE_ID_TYPE 0x4B00
5455
#define INTEL_PCH_TGP_DEVICE_ID_TYPE 0xA080
5556
#define INTEL_PCH_TGP2_DEVICE_ID_TYPE 0x4380
5657
#define INTEL_PCH_JSP_DEVICE_ID_TYPE 0x4D80
57-
#define INTEL_PCH_JSP2_DEVICE_ID_TYPE 0x3880
5858
#define INTEL_PCH_ADP_DEVICE_ID_TYPE 0x7A80
5959
#define INTEL_PCH_ADP2_DEVICE_ID_TYPE 0x5180
6060
#define INTEL_PCH_ADP3_DEVICE_ID_TYPE 0x7A00

0 commit comments

Comments
 (0)