On Thu, Jun 1, 2017 at 3:27 PM, Srivatsa, Anusha <anusha.srivatsa@xxxxxxxxx> wrote: > > >>-----Original Message----- >>From: Intel-gfx [mailto:intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of >>Rodrigo Vivi >>Sent: Tuesday, May 30, 2017 3:43 PM >>To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx >>Cc: Vivi, Rodrigo <rodrigo.vivi@xxxxxxxxx> >>Subject: [PATCH 07/13] drm/i915/cfl: Introduce Coffee Lake platform >>definition. >> >>Coffee Lake is a Intel® Processor containing Intel® HD Graphics following >>Kabylake. >> >>It is Gen9 graphics based platform on top of CNP PCH. >> >>Let's start by adding the platform definition based on previous platforms but yet >>as preliminary_hw_support. >> >>On following patches we will start adding PCI IDs and the platform specific >>changes. >> >>Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> >>--- >> drivers/gpu/drm/i915/i915_drv.h | 2 ++ >> drivers/gpu/drm/i915/i915_pci.c | 8 ++++++++ >> drivers/gpu/drm/i915/intel_device_info.c | 1 + >> 3 files changed, 11 insertions(+) >> >>diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h >>index c31c0cf..2f20e87 100644 >>--- a/drivers/gpu/drm/i915/i915_drv.h >>+++ b/drivers/gpu/drm/i915/i915_drv.h >>@@ -826,6 +826,7 @@ enum intel_platform { >> INTEL_BROXTON, >> INTEL_KABYLAKE, >> INTEL_GEMINILAKE, >>+ INTEL_COFFEELAKE, >> INTEL_MAX_PLATFORMS >> }; >> >>@@ -2768,6 +2769,7 @@ static inline struct scatterlist *__sg_next(struct >>scatterlist *sg) >> #define IS_BROXTON(dev_priv) ((dev_priv)->info.platform == INTEL_BROXTON) >> #define IS_KABYLAKE(dev_priv) ((dev_priv)->info.platform == INTEL_KABYLAKE) >> #define IS_GEMINILAKE(dev_priv) ((dev_priv)->info.platform == >>INTEL_GEMINILAKE) >>+#define IS_COFFEELAKE(dev_priv) ((dev_priv)->info.platform == >>INTEL_COFFEELAKE) >> #define IS_MOBILE(dev_priv) ((dev_priv)->info.is_mobile) >> #define IS_HSW_EARLY_SDV(dev_priv) (IS_HASWELL(dev_priv) && \ >> (INTEL_DEVID(dev_priv) & 0xFF00) == 0x0C00) >>diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c >>index cf43dc1..31ea988 100644 >>--- a/drivers/gpu/drm/i915/i915_pci.c >>+++ b/drivers/gpu/drm/i915/i915_pci.c >>@@ -425,6 +425,14 @@ >> .ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING | >>BSD2_RING, }; >> >>+static const struct intel_device_info intel_coffeelake_info = { >>+ BDW_FEATURES, >>+ .is_alpha_support = 1, >>+ .platform = INTEL_COFFEELAKE, >>+ .gen = 9, >>+ .ddb_size = 896, >>+}; > > Rodrigo, what about properties like has_guc, has_csr, color etc? I believe "has_guc" should be part of the GuC patch that you are about to send ;) as "has_csr" is part of "[PATCH 13/13] drm/i915/cfl: Coffe Lake reuses Kabylake DMC." while "color" one is not needed, as it was not in Kabylake and Skylake. > > BR > Anusha >> /* >> * Make sure any device matches here are from most specific to most >> * general. For example, since the Quanta match is based on the subsystem diff - >>-git a/drivers/gpu/drm/i915/intel_device_info.c >>b/drivers/gpu/drm/i915/intel_device_info.c >>index 3718341..acc746f 100644 >>--- a/drivers/gpu/drm/i915/intel_device_info.c >>+++ b/drivers/gpu/drm/i915/intel_device_info.c >>@@ -51,6 +51,7 @@ >> PLATFORM_NAME(BROXTON), >> PLATFORM_NAME(KABYLAKE), >> PLATFORM_NAME(GEMINILAKE), >>+ PLATFORM_NAME(COFFEELAKE), >> }; >> #undef PLATFORM_NAME >> >>-- >>1.9.1 >> >>_______________________________________________ >>Intel-gfx mailing list >>Intel-gfx@xxxxxxxxxxxxxxxxxxxxx >>https://lists.freedesktop.org/mailman/listinfo/intel-gfx > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Rodrigo Vivi Blog: http://blog.vivi.eng.br _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
