Re: [Intel-gfx] [PATCH v5] drm/i915/rkl: new rkl ddc map for different PCH
Hi, Progress here seems to have stalled, or did I miss something? On 20.11.2020 4.21, Lee, Shawn C wrote: On Thu, Nov. 19, 2020 at 11:51 PM, Matt Roper wrote: On Tue, Nov 17, 2020 at 10:26:29PM +0800, Lee Shawn C wrote: After boot into kernel. Driver configured ddc pin mapping based on predefined table in parse_ddi_port(). Now driver configure rkl ddc pin mapping depends on icp_ddc_pin_map[]. Then this table will give incorrect gmbus port number to cause HDMI can't work. Refer to commit cd0a89527d06 ("drm/i915/rkl: Add DDC pin mapping"). Create two ddc pin table for rkl TGP and CMP pch. Then HDMI can works properly on rkl. v2: update patch based on latest dinq branch. v3: update ddc table for RKL+TGP sku. RKL+CNP sku will load cnp_ddc_pin_map[] setting. v4: modify the if/else judgment to avoid nesting. v5: fix typo in v4. Cc: Matt Roper Cc: Aditya Swarup Cc: Anusha Srivatsa Cc: Jani Nikula Cc: Cooper Chiou Cc: Khaled Almahallawy Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/2577 Signed-off-by: Lee Shawn C Reviewed-by: Matt Roper Do you plan to follow up with a separate patch to fix the CMP handling in rkl_port_to_ddc_pin that I mentioned previously? I want to make sure that part doesn't fall through the cracks. Do you mean the modification like this in rkl_port_to_ddc_pin()? If so, I will commit a separate patch to fix it later. return GMBUS_PIN_1_BXT + phy - 1; Best regards, Shawn Matt --- drivers/gpu/drm/i915/display/intel_bios.c | 10 ++ drivers/gpu/drm/i915/display/intel_vbt_defs.h | 2 ++ 2 files changed, 12 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c index 4cc949b228f2..cf2fba490b7b 100644 --- a/drivers/gpu/drm/i915/display/intel_bios.c +++ b/drivers/gpu/drm/i915/display/intel_bios.c @@ -1623,6 +1623,13 @@ static const u8 icp_ddc_pin_map[] = { [TGL_DDC_BUS_PORT_6] = GMBUS_PIN_14_TC6_TGP, }; +static const u8 rkl_pch_tgp_ddc_pin_map[] = { + [ICL_DDC_BUS_DDI_A] = GMBUS_PIN_1_BXT, + [ICL_DDC_BUS_DDI_B] = GMBUS_PIN_2_BXT, + [RKL_DDC_BUS_DDI_D] = GMBUS_PIN_9_TC1_ICP, + [RKL_DDC_BUS_DDI_E] = GMBUS_PIN_10_TC2_ICP, }; + static u8 map_ddc_pin(struct drm_i915_private *dev_priv, u8 vbt_pin) { const u8 *ddc_pin_map; @@ -1630,6 +1637,9 @@ static u8 map_ddc_pin(struct drm_i915_private *dev_priv, u8 vbt_pin) if (INTEL_PCH_TYPE(dev_priv) >= PCH_DG1) { return vbt_pin; + } else if (IS_ROCKETLAKE(dev_priv) && INTEL_PCH_TYPE(dev_priv) == PCH_TGP) { + ddc_pin_map = rkl_pch_tgp_ddc_pin_map; + n_entries = ARRAY_SIZE(rkl_pch_tgp_ddc_pin_map); } else if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP) { ddc_pin_map = icp_ddc_pin_map; n_entries = ARRAY_SIZE(icp_ddc_pin_map); diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h b/drivers/gpu/drm/i915/display/intel_vbt_defs.h index 49b4b5fca941..187ec573de59 100644 --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h @@ -319,6 +319,8 @@ enum vbt_gmbus_ddi { ICL_DDC_BUS_DDI_A = 0x1, ICL_DDC_BUS_DDI_B, TGL_DDC_BUS_DDI_C, + RKL_DDC_BUS_DDI_D = 0x3, + RKL_DDC_BUS_DDI_E, ICL_DDC_BUS_PORT_1 = 0x4, ICL_DDC_BUS_PORT_2, ICL_DDC_BUS_PORT_3, -- 2.17.1 -- 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 -- t ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v5] drm/i915/rkl: new rkl ddc map for different PCH
On Thu, Nov. 19, 2020 at 11:51 PM, Matt Roper wrote: >On Tue, Nov 17, 2020 at 10:26:29PM +0800, Lee Shawn C wrote: >> After boot into kernel. Driver configured ddc pin mapping based on >> predefined table in parse_ddi_port(). Now driver configure rkl ddc pin >> mapping depends on icp_ddc_pin_map[]. Then this table will give >> incorrect gmbus port number to cause HDMI can't work. >> >> Refer to commit cd0a89527d06 ("drm/i915/rkl: Add DDC pin mapping"). >> Create two ddc pin table for rkl TGP and CMP pch. Then HDMI can works >> properly on rkl. >> >> v2: update patch based on latest dinq branch. >> v3: update ddc table for RKL+TGP sku. >> RKL+CNP sku will load cnp_ddc_pin_map[] setting. >> v4: modify the if/else judgment to avoid nesting. >> v5: fix typo in v4. >> >> Cc: Matt Roper >> Cc: Aditya Swarup >> Cc: Anusha Srivatsa >> Cc: Jani Nikula >> Cc: Cooper Chiou >> Cc: Khaled Almahallawy >> Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/2577 >> Signed-off-by: Lee Shawn C > >Reviewed-by: Matt Roper > >Do you plan to follow up with a separate patch to fix the CMP handling in >rkl_port_to_ddc_pin that I mentioned previously? I want to make sure that >part doesn't fall through the cracks. > Do you mean the modification like this in rkl_port_to_ddc_pin()? If so, I will commit a separate patch to fix it later. return GMBUS_PIN_1_BXT + phy - 1; Best regards, Shawn > >Matt > >> --- >> drivers/gpu/drm/i915/display/intel_bios.c | 10 ++ >> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 2 ++ >> 2 files changed, 12 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_bios.c >> b/drivers/gpu/drm/i915/display/intel_bios.c >> index 4cc949b228f2..cf2fba490b7b 100644 >> --- a/drivers/gpu/drm/i915/display/intel_bios.c >> +++ b/drivers/gpu/drm/i915/display/intel_bios.c >> @@ -1623,6 +1623,13 @@ static const u8 icp_ddc_pin_map[] = { >> [TGL_DDC_BUS_PORT_6] = GMBUS_PIN_14_TC6_TGP, }; >> >> +static const u8 rkl_pch_tgp_ddc_pin_map[] = { >> +[ICL_DDC_BUS_DDI_A] = GMBUS_PIN_1_BXT, >> +[ICL_DDC_BUS_DDI_B] = GMBUS_PIN_2_BXT, >> +[RKL_DDC_BUS_DDI_D] = GMBUS_PIN_9_TC1_ICP, >> +[RKL_DDC_BUS_DDI_E] = GMBUS_PIN_10_TC2_ICP, }; >> + >> static u8 map_ddc_pin(struct drm_i915_private *dev_priv, u8 vbt_pin) >> { >> const u8 *ddc_pin_map; >> @@ -1630,6 +1637,9 @@ static u8 map_ddc_pin(struct drm_i915_private >> *dev_priv, u8 vbt_pin) >> >> if (INTEL_PCH_TYPE(dev_priv) >= PCH_DG1) { >> return vbt_pin; >> +} else if (IS_ROCKETLAKE(dev_priv) && INTEL_PCH_TYPE(dev_priv) == >> PCH_TGP) { >> +ddc_pin_map = rkl_pch_tgp_ddc_pin_map; >> +n_entries = ARRAY_SIZE(rkl_pch_tgp_ddc_pin_map); >> } else if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP) { >> ddc_pin_map = icp_ddc_pin_map; >> n_entries = ARRAY_SIZE(icp_ddc_pin_map); diff --git >> a/drivers/gpu/drm/i915/display/intel_vbt_defs.h >> b/drivers/gpu/drm/i915/display/intel_vbt_defs.h >> index 49b4b5fca941..187ec573de59 100644 >> --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h >> +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h >> @@ -319,6 +319,8 @@ enum vbt_gmbus_ddi { >> ICL_DDC_BUS_DDI_A = 0x1, >> ICL_DDC_BUS_DDI_B, >> TGL_DDC_BUS_DDI_C, >> +RKL_DDC_BUS_DDI_D = 0x3, >> +RKL_DDC_BUS_DDI_E, >> ICL_DDC_BUS_PORT_1 = 0x4, >> ICL_DDC_BUS_PORT_2, >> ICL_DDC_BUS_PORT_3, >> -- >> 2.17.1 >> > >-- >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
Re: [Intel-gfx] [PATCH v5] drm/i915/rkl: new rkl ddc map for different PCH
On Tue, Nov 17, 2020 at 10:26:29PM +0800, Lee Shawn C wrote: > After boot into kernel. Driver configured ddc pin mapping based on > predefined table in parse_ddi_port(). Now driver configure rkl > ddc pin mapping depends on icp_ddc_pin_map[]. Then this table will > give incorrect gmbus port number to cause HDMI can't work. > > Refer to commit cd0a89527d06 ("drm/i915/rkl: Add DDC pin mapping"). > Create two ddc pin table for rkl TGP and CMP pch. Then HDMI can > works properly on rkl. > > v2: update patch based on latest dinq branch. > v3: update ddc table for RKL+TGP sku. > RKL+CNP sku will load cnp_ddc_pin_map[] setting. > v4: modify the if/else judgment to avoid nesting. > v5: fix typo in v4. > > Cc: Matt Roper > Cc: Aditya Swarup > Cc: Anusha Srivatsa > Cc: Jani Nikula > Cc: Cooper Chiou > Cc: Khaled Almahallawy > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/2577 > Signed-off-by: Lee Shawn C Reviewed-by: Matt Roper Do you plan to follow up with a separate patch to fix the CMP handling in rkl_port_to_ddc_pin that I mentioned previously? I want to make sure that part doesn't fall through the cracks. Matt > --- > drivers/gpu/drm/i915/display/intel_bios.c | 10 ++ > drivers/gpu/drm/i915/display/intel_vbt_defs.h | 2 ++ > 2 files changed, 12 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_bios.c > b/drivers/gpu/drm/i915/display/intel_bios.c > index 4cc949b228f2..cf2fba490b7b 100644 > --- a/drivers/gpu/drm/i915/display/intel_bios.c > +++ b/drivers/gpu/drm/i915/display/intel_bios.c > @@ -1623,6 +1623,13 @@ static const u8 icp_ddc_pin_map[] = { > [TGL_DDC_BUS_PORT_6] = GMBUS_PIN_14_TC6_TGP, > }; > > +static const u8 rkl_pch_tgp_ddc_pin_map[] = { > + [ICL_DDC_BUS_DDI_A] = GMBUS_PIN_1_BXT, > + [ICL_DDC_BUS_DDI_B] = GMBUS_PIN_2_BXT, > + [RKL_DDC_BUS_DDI_D] = GMBUS_PIN_9_TC1_ICP, > + [RKL_DDC_BUS_DDI_E] = GMBUS_PIN_10_TC2_ICP, > +}; > + > static u8 map_ddc_pin(struct drm_i915_private *dev_priv, u8 vbt_pin) > { > const u8 *ddc_pin_map; > @@ -1630,6 +1637,9 @@ static u8 map_ddc_pin(struct drm_i915_private > *dev_priv, u8 vbt_pin) > > if (INTEL_PCH_TYPE(dev_priv) >= PCH_DG1) { > return vbt_pin; > + } else if (IS_ROCKETLAKE(dev_priv) && INTEL_PCH_TYPE(dev_priv) == > PCH_TGP) { > + ddc_pin_map = rkl_pch_tgp_ddc_pin_map; > + n_entries = ARRAY_SIZE(rkl_pch_tgp_ddc_pin_map); > } else if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP) { > ddc_pin_map = icp_ddc_pin_map; > n_entries = ARRAY_SIZE(icp_ddc_pin_map); > diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h > b/drivers/gpu/drm/i915/display/intel_vbt_defs.h > index 49b4b5fca941..187ec573de59 100644 > --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h > +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h > @@ -319,6 +319,8 @@ enum vbt_gmbus_ddi { > ICL_DDC_BUS_DDI_A = 0x1, > ICL_DDC_BUS_DDI_B, > TGL_DDC_BUS_DDI_C, > + RKL_DDC_BUS_DDI_D = 0x3, > + RKL_DDC_BUS_DDI_E, > ICL_DDC_BUS_PORT_1 = 0x4, > ICL_DDC_BUS_PORT_2, > ICL_DDC_BUS_PORT_3, > -- > 2.17.1 > -- 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
[Intel-gfx] [PATCH v5] drm/i915/rkl: new rkl ddc map for different PCH
After boot into kernel. Driver configured ddc pin mapping based on predefined table in parse_ddi_port(). Now driver configure rkl ddc pin mapping depends on icp_ddc_pin_map[]. Then this table will give incorrect gmbus port number to cause HDMI can't work. Refer to commit cd0a89527d06 ("drm/i915/rkl: Add DDC pin mapping"). Create two ddc pin table for rkl TGP and CMP pch. Then HDMI can works properly on rkl. v2: update patch based on latest dinq branch. v3: update ddc table for RKL+TGP sku. RKL+CNP sku will load cnp_ddc_pin_map[] setting. v4: modify the if/else judgment to avoid nesting. v5: fix typo in v4. Cc: Matt Roper Cc: Aditya Swarup Cc: Anusha Srivatsa Cc: Jani Nikula Cc: Cooper Chiou Cc: Khaled Almahallawy Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/2577 Signed-off-by: Lee Shawn C --- drivers/gpu/drm/i915/display/intel_bios.c | 10 ++ drivers/gpu/drm/i915/display/intel_vbt_defs.h | 2 ++ 2 files changed, 12 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c index 4cc949b228f2..cf2fba490b7b 100644 --- a/drivers/gpu/drm/i915/display/intel_bios.c +++ b/drivers/gpu/drm/i915/display/intel_bios.c @@ -1623,6 +1623,13 @@ static const u8 icp_ddc_pin_map[] = { [TGL_DDC_BUS_PORT_6] = GMBUS_PIN_14_TC6_TGP, }; +static const u8 rkl_pch_tgp_ddc_pin_map[] = { + [ICL_DDC_BUS_DDI_A] = GMBUS_PIN_1_BXT, + [ICL_DDC_BUS_DDI_B] = GMBUS_PIN_2_BXT, + [RKL_DDC_BUS_DDI_D] = GMBUS_PIN_9_TC1_ICP, + [RKL_DDC_BUS_DDI_E] = GMBUS_PIN_10_TC2_ICP, +}; + static u8 map_ddc_pin(struct drm_i915_private *dev_priv, u8 vbt_pin) { const u8 *ddc_pin_map; @@ -1630,6 +1637,9 @@ static u8 map_ddc_pin(struct drm_i915_private *dev_priv, u8 vbt_pin) if (INTEL_PCH_TYPE(dev_priv) >= PCH_DG1) { return vbt_pin; + } else if (IS_ROCKETLAKE(dev_priv) && INTEL_PCH_TYPE(dev_priv) == PCH_TGP) { + ddc_pin_map = rkl_pch_tgp_ddc_pin_map; + n_entries = ARRAY_SIZE(rkl_pch_tgp_ddc_pin_map); } else if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP) { ddc_pin_map = icp_ddc_pin_map; n_entries = ARRAY_SIZE(icp_ddc_pin_map); diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h b/drivers/gpu/drm/i915/display/intel_vbt_defs.h index 49b4b5fca941..187ec573de59 100644 --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h @@ -319,6 +319,8 @@ enum vbt_gmbus_ddi { ICL_DDC_BUS_DDI_A = 0x1, ICL_DDC_BUS_DDI_B, TGL_DDC_BUS_DDI_C, + RKL_DDC_BUS_DDI_D = 0x3, + RKL_DDC_BUS_DDI_E, ICL_DDC_BUS_PORT_1 = 0x4, ICL_DDC_BUS_PORT_2, ICL_DDC_BUS_PORT_3, -- 2.17.1 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx