All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] lib/cfl: Introduce Coffeelake platform definition.
@ 2017-06-29 17:18 Rodrigo Vivi
  2017-06-29 17:35 ` Clint Taylor
  0 siblings, 1 reply; 3+ messages in thread
From: Rodrigo Vivi @ 2017-06-29 17:18 UTC (permalink / raw)
  To: intel-gfx; +Cc: Rodrigo Vivi

Coffeelake is a Intel® Processor containing Intel® HD Graphics
following Kabylake.

It is Gen9 graphics based platform on top of CNP PCH.

On following patches we will start adding PCI IDs and the
platform specific changes.

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 lib/intel_chipset.h     | 2 ++
 lib/intel_device_info.c | 6 ++++++
 2 files changed, 8 insertions(+)

diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
index 259b45f..2468890 100644
--- a/lib/intel_chipset.h
+++ b/lib/intel_chipset.h
@@ -64,6 +64,7 @@ struct intel_device_info {
 	bool is_broxton : 1;
 	bool is_kabylake : 1;
 	bool is_geminilake : 1;
+	bool is_coffeelake : 1;
 	const char *codename;
 };
 
@@ -158,6 +159,7 @@ void intel_check_pch(void);
 #define IS_SKYLAKE(devid)	(intel_get_device_info(devid)->is_skylake)
 #define IS_BROXTON(devid)	(intel_get_device_info(devid)->is_broxton)
 #define IS_GEMINILAKE(devid)	(intel_get_device_info(devid)->is_geminilake)
+#define IS_COFFEELAKE(devid)	(intel_get_device_info(devid)->is_coffeelake)
 
 #define IS_GEN(devid, x)	(intel_get_device_info(devid)->gen & (1u << ((x)-1)))
 #define AT_LEAST_GEN(devid, x)	(intel_get_device_info(devid)->gen & -(1u << ((x)-1)))
diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
index 2b0658c..199fa2d 100644
--- a/lib/intel_device_info.c
+++ b/lib/intel_device_info.c
@@ -187,6 +187,12 @@ static const struct intel_device_info intel_geminilake_info = {
 	.codename = "geminilake"
 };
 
+static const struct intel_device_info intel_coffeelake_info = {
+	.gen = BIT(8),
+	.is_coffeelake = true,
+	.codename = "coffeelake"
+};
+
 static const struct pci_id_match intel_device_match[] = {
 	INTEL_I810_IDS(&intel_i810_info),
 	INTEL_I815_IDS(&intel_i815_info),
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH i-g-t] lib/cfl: Introduce Coffeelake platform definition.
  2017-06-29 17:18 [PATCH i-g-t] lib/cfl: Introduce Coffeelake platform definition Rodrigo Vivi
@ 2017-06-29 17:35 ` Clint Taylor
  2017-06-29 17:42   ` Vivi, Rodrigo
  0 siblings, 1 reply; 3+ messages in thread
From: Clint Taylor @ 2017-06-29 17:35 UTC (permalink / raw)
  To: Rodrigo Vivi, intel-gfx

Identical to other platforms.

Reviewed-by: Clinton Taylor <clinton.a.taylor@intel.com>

On 06/29/2017 10:18 AM, Rodrigo Vivi wrote:
> Coffeelake is a Intel® Processor containing Intel® HD Graphics
> following Kabylake.
>
> It is Gen9 graphics based platform on top of CNP PCH.
>
> On following patches we will start adding PCI IDs and the
> platform specific changes.
>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>   lib/intel_chipset.h     | 2 ++
>   lib/intel_device_info.c | 6 ++++++
>   2 files changed, 8 insertions(+)
>
> diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
> index 259b45f..2468890 100644
> --- a/lib/intel_chipset.h
> +++ b/lib/intel_chipset.h
> @@ -64,6 +64,7 @@ struct intel_device_info {
>   	bool is_broxton : 1;
>   	bool is_kabylake : 1;
>   	bool is_geminilake : 1;
> +	bool is_coffeelake : 1;
>   	const char *codename;
>   };
>   
> @@ -158,6 +159,7 @@ void intel_check_pch(void);
>   #define IS_SKYLAKE(devid)	(intel_get_device_info(devid)->is_skylake)
>   #define IS_BROXTON(devid)	(intel_get_device_info(devid)->is_broxton)
>   #define IS_GEMINILAKE(devid)	(intel_get_device_info(devid)->is_geminilake)
> +#define IS_COFFEELAKE(devid)	(intel_get_device_info(devid)->is_coffeelake)
>   
>   #define IS_GEN(devid, x)	(intel_get_device_info(devid)->gen & (1u << ((x)-1)))
>   #define AT_LEAST_GEN(devid, x)	(intel_get_device_info(devid)->gen & -(1u << ((x)-1)))
> diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
> index 2b0658c..199fa2d 100644
> --- a/lib/intel_device_info.c
> +++ b/lib/intel_device_info.c
> @@ -187,6 +187,12 @@ static const struct intel_device_info intel_geminilake_info = {
>   	.codename = "geminilake"
>   };
>   
> +static const struct intel_device_info intel_coffeelake_info = {
> +	.gen = BIT(8),
> +	.is_coffeelake = true,
> +	.codename = "coffeelake"
> +};
> +
>   static const struct pci_id_match intel_device_match[] = {
>   	INTEL_I810_IDS(&intel_i810_info),
>   	INTEL_I815_IDS(&intel_i815_info),

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH i-g-t] lib/cfl: Introduce Coffeelake platform definition.
  2017-06-29 17:35 ` Clint Taylor
@ 2017-06-29 17:42   ` Vivi, Rodrigo
  0 siblings, 0 replies; 3+ messages in thread
From: Vivi, Rodrigo @ 2017-06-29 17:42 UTC (permalink / raw)
  To: Taylor, Clinton A; +Cc: intel-gfx

patch merged. thanks for the review. 

On Thu, 2017-06-29 at 10:35 -0700, Clint Taylor wrote:
> Identical to other platforms.
> 
> Reviewed-by: Clinton Taylor <clinton.a.taylor@intel.com>
> 
> On 06/29/2017 10:18 AM, Rodrigo Vivi wrote:
> > Coffeelake is a Intel® Processor containing Intel® HD Graphics
> > following Kabylake.
> >
> > It is Gen9 graphics based platform on top of CNP PCH.
> >
> > On following patches we will start adding PCI IDs and the
> > platform specific changes.
> >
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > ---
> >   lib/intel_chipset.h     | 2 ++
> >   lib/intel_device_info.c | 6 ++++++
> >   2 files changed, 8 insertions(+)
> >
> > diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
> > index 259b45f..2468890 100644
> > --- a/lib/intel_chipset.h
> > +++ b/lib/intel_chipset.h
> > @@ -64,6 +64,7 @@ struct intel_device_info {
> >   	bool is_broxton : 1;
> >   	bool is_kabylake : 1;
> >   	bool is_geminilake : 1;
> > +	bool is_coffeelake : 1;
> >   	const char *codename;
> >   };
> >   
> > @@ -158,6 +159,7 @@ void intel_check_pch(void);
> >   #define IS_SKYLAKE(devid)	(intel_get_device_info(devid)->is_skylake)
> >   #define IS_BROXTON(devid)	(intel_get_device_info(devid)->is_broxton)
> >   #define IS_GEMINILAKE(devid)	(intel_get_device_info(devid)->is_geminilake)
> > +#define IS_COFFEELAKE(devid)	(intel_get_device_info(devid)->is_coffeelake)
> >   
> >   #define IS_GEN(devid, x)	(intel_get_device_info(devid)->gen & (1u << ((x)-1)))
> >   #define AT_LEAST_GEN(devid, x)	(intel_get_device_info(devid)->gen & -(1u << ((x)-1)))
> > diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
> > index 2b0658c..199fa2d 100644
> > --- a/lib/intel_device_info.c
> > +++ b/lib/intel_device_info.c
> > @@ -187,6 +187,12 @@ static const struct intel_device_info intel_geminilake_info = {
> >   	.codename = "geminilake"
> >   };
> >   
> > +static const struct intel_device_info intel_coffeelake_info = {
> > +	.gen = BIT(8),
> > +	.is_coffeelake = true,
> > +	.codename = "coffeelake"
> > +};
> > +
> >   static const struct pci_id_match intel_device_match[] = {
> >   	INTEL_I810_IDS(&intel_i810_info),
> >   	INTEL_I815_IDS(&intel_i815_info),
> 

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-06-29 17:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-29 17:18 [PATCH i-g-t] lib/cfl: Introduce Coffeelake platform definition Rodrigo Vivi
2017-06-29 17:35 ` Clint Taylor
2017-06-29 17:42   ` Vivi, Rodrigo

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.