All of lore.kernel.org
 help / color / mirror / Atom feed
From: Carlos Santa <carlos.santa@intel.com>
To: intel-gfx@lists.freedesktop.org
Subject: [PATCH v4 10/22] drm/i915: Move HAS_RC6p definition to platform definition
Date: Wed, 17 Aug 2016 12:30:45 -0700	[thread overview]
Message-ID: <1471462257-24146-11-git-send-email-carlos.santa@intel.com> (raw)
In-Reply-To: <1471462257-24146-1-git-send-email-carlos.santa@intel.com>

Moving all GPU features to the platform struct definition allows for
	- standard place when adding new features from new platforms
	- possible to see supported features when dumping struct
	  definitions

Signed-off-by: Carlos Santa <carlos.santa@intel.com>
---
 drivers/gpu/drm/i915/i915_drv.h | 3 ++-
 drivers/gpu/drm/i915/i915_pci.c | 3 +++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 46fc71a..f20a3e3 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -651,6 +651,7 @@ struct intel_csr {
 	func(has_csr) sep \
 	func(has_resource_streamer) sep \
 	func(has_rc6) sep \
+	func(has_rc6p) sep \
 	func(has_pipe_cxsr) sep \
 	func(has_hotplug) sep \
 	func(cursor_needs_physical) sep \
@@ -2788,7 +2789,7 @@ struct drm_i915_cmd_table {
 #define HAS_PSR(dev)		(INTEL_INFO(dev)->has_psr)
 #define HAS_RUNTIME_PM(dev)	(INTEL_INFO(dev)->has_runtime_pm)
 #define HAS_RC6(dev)		(INTEL_INFO(dev)->has_rc6)
-#define HAS_RC6p(dev)		(IS_GEN6(dev) || IS_IVYBRIDGE(dev))
+#define HAS_RC6p(dev)		(INTEL_INFO(dev)->has_rc6p)
 
 #define HAS_CSR(dev)	(INTEL_INFO(dev)->has_csr)
 
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
index 42108dc..c6a5bd0 100644
--- a/drivers/gpu/drm/i915/i915_pci.c
+++ b/drivers/gpu/drm/i915/i915_pci.c
@@ -202,6 +202,7 @@ static const struct intel_device_info intel_ironlake_m_info = {
 	.ring_mask = RENDER_RING | BSD_RING | BLT_RING, \
 	.has_llc = 1, \
 	.has_rc6 = 1, \
+	.has_rc6p = 1, \
 	GEN_DEFAULT_PIPEOFFSETS, \
 	CURSOR_OFFSETS
 
@@ -221,6 +222,7 @@ static const struct intel_device_info intel_sandybridge_m_info = {
 	.ring_mask = RENDER_RING | BSD_RING | BLT_RING, \
 	.has_llc = 1, \
 	.has_rc6 = 1, \
+	.has_rc6p = 1, \
 	GEN_DEFAULT_PIPEOFFSETS, \
 	IVB_CURSOR_OFFSETS
 
@@ -264,6 +266,7 @@ static const struct intel_device_info intel_valleyview_info = {
 	.has_fpga_dbg = 1, \
 	.has_psr = 1, \
 	.has_resource_streamer = 1, \
+	.has_rc6p = 0 /* RC6p removed-by HSW */, \
 	.has_runtime_pm = 1
 
 static const struct intel_device_info intel_haswell_info = {
-- 
1.9.1

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

  parent reply	other threads:[~2016-08-17 19:30 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-17 19:30 [PATCH v4 00/22] drm/i915: Organize most GPU features by platform Carlos Santa
2016-08-17 19:30 ` [PATCH v4 01/22] drm/i915: Move HAS_PSR definition to platform struct definition Carlos Santa
2016-08-17 19:30 ` [PATCH v4 02/22] drm/i915: Remove .is_mobile field from platform struct Carlos Santa
2016-08-17 19:30 ` [PATCH v4 03/22] drm/i915: Introduce GEN6_FEATURES for device info Carlos Santa
2016-08-17 19:30 ` [PATCH v4 04/22] drm/i915: Move HAS_RUNTIME_PM definition to platform Carlos Santa
2016-09-07 22:55   ` Rodrigo Vivi
2016-08-17 19:30 ` [PATCH v4 05/22] drm/i915: Remove runtime PM for SNB Carlos Santa
2016-09-07 22:54   ` Rodrigo Vivi
2016-08-17 19:30 ` [PATCH v4 06/22] drm/i915: Get rid of HAS_CORE_RING_FREQ Carlos Santa
2016-09-07 22:50   ` Rodrigo Vivi
2016-08-17 19:30 ` [PATCH v4 07/22] drm/i915 Move HAS_CSR definition to platform definition Carlos Santa
2016-08-17 19:30 ` [PATCH v4 08/22] drm/i915: Move HAS_RESOURCE_STREAMER " Carlos Santa
2016-08-17 19:30 ` [PATCH v4 09/22] drm/i915: Move HAS_RC6 " Carlos Santa
2016-08-17 19:30 ` Carlos Santa [this message]
2016-09-07 22:52   ` [PATCH v4 10/22] drm/i915: Move HAS_RC6p " Rodrigo Vivi
2016-08-17 19:30 ` [PATCH v4 11/22] drm/i915: Move HAS_DP_MST " Carlos Santa
2016-08-17 19:30 ` [PATCH v4 12/22] drm/i915: Introduce GEN5_FEATURES for device info Carlos Santa
2016-08-17 19:30 ` [PATCH v4 13/22] drm/i915: Move HAS_GMBUS_IRQ definition to platform definition Carlos Santa
2016-08-17 19:30 ` [PATCH v4 14/22] drm/i915: Introduce GEN4_FEATURES for device info Carlos Santa
2016-08-17 19:30 ` [PATCH v4 15/22] drm/i915: Introduce GEN3_FEATURES " Carlos Santa
2016-08-17 19:30 ` [PATCH v4 16/22] drm/i915: Introduce GEN2_FEATURES " Carlos Santa
2016-08-17 19:30 ` [PATCH v4 17/22] drm/i915: Move HAS_HW_CONTEXTS definition to platform Carlos Santa
2016-08-17 19:30 ` [PATCH v4 18/22] drm/i915: Move HAS_LOGICAL_RING_CONTEXTS " Carlos Santa
2016-08-17 19:30 ` [PATCH v4 19/22] drm/i915: Move HAS_L3_DPF definition to platform definition Carlos Santa
2016-08-17 19:30 ` [PATCH v4 20/22] drm/i915: Move HAS_GMCH_DISPLAY definition to platform Carlos Santa
2016-08-17 19:30 ` [PATCH v4 21/22] drm/i915: Make HWS_NEEDS_PHYSICAL the exception Carlos Santa
2016-08-17 19:30 ` [PATCH v4 22/22] drm/i915: Move HAS_GUC definition to platform definition Carlos Santa
2016-08-18  6:22 ` ✗ Ro.CI.BAT: failure for drm/i915: Organize most GPU features by platform (rev4) Patchwork
2016-09-13 16:06 ` [PATCH v4 00/22] drm/i915: Organize most GPU features by platform Rodrigo Vivi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1471462257-24146-11-git-send-email-carlos.santa@intel.com \
    --to=carlos.santa@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.