[Intel-gfx] [RFC v2 1/2] drm/i915: Add sub platform for 480MHz CDCLK step

Borah, Chaitanya Kumar chaitanya.kumar.borah at intel.com
Fri Jan 13 04:38:36 UTC 2023


Hello Jani and Matt,

> -----Original Message-----
> From: Roper, Matthew D <matthew.d.roper at intel.com>
> Sent: Thursday, January 12, 2023 11:03 PM
> To: Jani Nikula <jani.nikula at linux.intel.com>
> Cc: Borah, Chaitanya Kumar <chaitanya.kumar.borah at intel.com>; intel-
> gfx at lists.freedesktop.org; Shankar, Uma <uma.shankar at intel.com>; Syrjala,
> Ville <ville.syrjala at intel.com>; Srivatsa, Anusha
> <anusha.srivatsa at intel.com>; Atwood, Matthew S
> <matthew.s.atwood at intel.com>
> Subject: Re: [RFC v2 1/2] drm/i915: Add sub platform for 480MHz CDCLK step
> 
> On Thu, Jan 12, 2023 at 01:03:44PM +0200, Jani Nikula wrote:
> > On Thu, 12 Jan 2023, Chaitanya Kumar Borah
> <chaitanya.kumar.borah at intel.com> wrote:
> > > Add a new sub platform INTEL_SUBPLATFORM_480CDCLK to identify
> device
> > > ids that need the 480MHz CDCLK step. Separate out RPLU device ids
> > > and add them to both RPL and 480CDCLK subplatforms.
> > >
> > > v2: (Matt)
> > >   - Sort PCI-IDs numerically
> > >   - Name the sub-platform to accurately depict what it is for
> >
> > Did Matt actually ask you to call it INTEL_SUBPLATFORM_480CDCLK?
> >
> > Personally, I don't like it, because RPL-U has its own PCI IDs and
> > that's what it's called in the specs. I'd prefer what it was.
> >
> > (The one outlier is INTEL_SUBPLATFORM_PORTF and I don't like that one
> > either.)
> 
> I think initially the plan was to not match RPL-U, but rather to only match a
> specific subset of RPL-U that had this special cdclk (which I think was
> determined by CPU vendor string or something).  If that's no longer
> something we care about, then it probably isn't as important to have the
> more precise subplatform name anymore.
> 

I will go back to the RPLU subplatform implementation. What do you think of adding RPL-U ids to RPL subplatform as well? Shall we retain that?

Regards

Chaitanya

> 
> Matt
> 
> >
> > BR,
> > Jani.
> >
> >
> > >   - Make RPL-U part of RPL subplatform
> > >
> > > Signed-off-by: Chaitanya Kumar Borah
> > > <chaitanya.kumar.borah at intel.com>
> > > ---
> > >  drivers/gpu/drm/i915/i915_drv.h          |  2 ++
> > >  drivers/gpu/drm/i915/i915_pci.c          |  1 +
> > >  drivers/gpu/drm/i915/intel_device_info.c |  8 ++++++++
> > > drivers/gpu/drm/i915/intel_device_info.h |  2 ++
> > >  include/drm/i915_pciids.h                | 11 +++++++----
> > >  5 files changed, 20 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/i915_drv.h
> > > b/drivers/gpu/drm/i915/i915_drv.h index 48fd82722f12..fc5518314598
> > > 100644
> > > --- a/drivers/gpu/drm/i915/i915_drv.h
> > > +++ b/drivers/gpu/drm/i915/i915_drv.h
> > > @@ -619,6 +619,8 @@ IS_SUBPLATFORM(const struct drm_i915_private
> *i915,
> > >  	IS_SUBPLATFORM(dev_priv, INTEL_ALDERLAKE_P,
> INTEL_SUBPLATFORM_N)
> > > #define IS_ADLP_RPLP(dev_priv) \
> > >  	IS_SUBPLATFORM(dev_priv, INTEL_ALDERLAKE_P,
> INTEL_SUBPLATFORM_RPL)
> > > +#define IS_ADLP_WITH_480CDCLK(dev_priv) \
> > > +	IS_SUBPLATFORM(dev_priv, INTEL_ALDERLAKE_P,
> > > +INTEL_SUBPLATFORM_480CDCLK)
> > >  #define IS_HSW_EARLY_SDV(dev_priv) (IS_HASWELL(dev_priv) && \
> > >  				    (INTEL_DEVID(dev_priv) & 0xFF00) ==
> 0x0C00)  #define
> > > IS_BDW_ULT(dev_priv) \ diff --git a/drivers/gpu/drm/i915/i915_pci.c
> > > b/drivers/gpu/drm/i915/i915_pci.c index 6cc65079b18d..e9f3b99b3e00
> > > 100644
> > > --- a/drivers/gpu/drm/i915/i915_pci.c
> > > +++ b/drivers/gpu/drm/i915/i915_pci.c
> > > @@ -1234,6 +1234,7 @@ static const struct pci_device_id pciidlist[] = {
> > >  	INTEL_DG1_IDS(&dg1_info),
> > >  	INTEL_RPLS_IDS(&adl_s_info),
> > >  	INTEL_RPLP_IDS(&adl_p_info),
> > > +	INTEL_RPLU_IDS(&adl_p_info),
> > >  	INTEL_DG2_IDS(&dg2_info),
> > >  	INTEL_ATS_M_IDS(&ats_m_info),
> > >  	INTEL_MTL_IDS(&mtl_info),
> > > diff --git a/drivers/gpu/drm/i915/intel_device_info.c
> > > b/drivers/gpu/drm/i915/intel_device_info.c
> > > index 849baf6c3b3c..a1732ad519fb 100644
> > > --- a/drivers/gpu/drm/i915/intel_device_info.c
> > > +++ b/drivers/gpu/drm/i915/intel_device_info.c
> > > @@ -199,6 +199,11 @@ static const u16 subplatform_n_ids[] = {
> > > static const u16 subplatform_rpl_ids[] = {
> > >  	INTEL_RPLS_IDS(0),
> > >  	INTEL_RPLP_IDS(0),
> > > +	INTEL_RPLU_IDS(0)
> > > +};
> > > +
> > > +static const u16 subplatform_480cdclk_ids[] = {
> > > +	INTEL_RPLU_IDS(0),
> > >  };
> > >
> > >  static const u16 subplatform_g10_ids[] = { @@ -268,6 +273,9 @@
> > > static void intel_device_info_subplatform_init(struct drm_i915_private
> *i915)
> > >  	} else if (find_devid(devid, subplatform_rpl_ids,
> > >  			      ARRAY_SIZE(subplatform_rpl_ids))) {
> > >  		mask = BIT(INTEL_SUBPLATFORM_RPL);
> > > +		if (find_devid(devid, subplatform_480cdclk_ids,
> > > +			       ARRAY_SIZE(subplatform_480cdclk_ids)))
> > > +			mask |= BIT(INTEL_SUBPLATFORM_480CDCLK);
> > >  	} else if (find_devid(devid, subplatform_g10_ids,
> > >  			      ARRAY_SIZE(subplatform_g10_ids))) {
> > >  		mask = BIT(INTEL_SUBPLATFORM_G10); diff --git
> > > a/drivers/gpu/drm/i915/intel_device_info.h
> > > b/drivers/gpu/drm/i915/intel_device_info.h
> > > index d588e5fd2eea..4f488c900273 100644
> > > --- a/drivers/gpu/drm/i915/intel_device_info.h
> > > +++ b/drivers/gpu/drm/i915/intel_device_info.h
> > > @@ -127,6 +127,8 @@ enum intel_platform {
> > >   * bit set
> > >   */
> > >  #define INTEL_SUBPLATFORM_N    1
> > > +/* Sub Platform with 480MHz CDCLK step */ #define
> > > +INTEL_SUBPLATFORM_480CDCLK  2
> > >
> > >  /* MTL */
> > >  #define INTEL_SUBPLATFORM_M	0
> > > diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h
> > > index 4a4c190f7698..758be5fb09a2 100644
> > > --- a/include/drm/i915_pciids.h
> > > +++ b/include/drm/i915_pciids.h
> > > @@ -684,14 +684,17 @@
> > >  	INTEL_VGA_DEVICE(0xA78A, info), \
> > >  	INTEL_VGA_DEVICE(0xA78B, info)
> > >
> > > +/* RPL-U */
> > > +#define INTEL_RPLU_IDS(info) \
> > > +	INTEL_VGA_DEVICE(0xA721, info), \
> > > +	INTEL_VGA_DEVICE(0xA7A1, info), \
> > > +	INTEL_VGA_DEVICE(0xA7A9, info)
> > > +
> > >  /* RPL-P */
> > >  #define INTEL_RPLP_IDS(info) \
> > >  	INTEL_VGA_DEVICE(0xA720, info), \
> > > -	INTEL_VGA_DEVICE(0xA721, info), \
> > >  	INTEL_VGA_DEVICE(0xA7A0, info), \
> > > -	INTEL_VGA_DEVICE(0xA7A1, info), \
> > > -	INTEL_VGA_DEVICE(0xA7A8, info), \
> > > -	INTEL_VGA_DEVICE(0xA7A9, info)
> > > +	INTEL_VGA_DEVICE(0xA7A8, info)
> > >
> > >  /* DG2 */
> > >  #define INTEL_DG2_G10_IDS(info) \
> >
> > --
> > Jani Nikula, Intel Open Source Graphics Center
> 
> --
> Matt Roper
> Graphics Software Engineer
> Linux GPU Platform Enablement
> Intel Corporation


More information about the Intel-gfx mailing list