Re: [Intel-gfx] [PATCH] drm/i915/hwconfig: Future-proof platform checks

2022-05-25 Thread Lucas De Marchi 

On Tue, May 24, 2022 at 04:59:06PM -0700, Matt Roper wrote:

PVC also has a hwconfig table.  Actually the current expectation is that
all future platforms will have hwconfig, so let's just change the
condition to an IP version check so that we don't need to keep updating
this for each new platform that shows up.

Cc: John Harrison 
Cc: Radhakrishna Sripada 
Signed-off-by: Matt Roper 
---
drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c 
b/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
index 5aaa3948de74..4781fccc2687 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
@@ -96,7 +96,7 @@ static bool has_table(struct drm_i915_private *i915)
{
if (IS_ALDERLAKE_P(i915) && !IS_ADLP_N(i915))
return true;
-   if (IS_DG2(i915))
+   if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55))
return true;


probably a matter of taste, but given gcc's history on how to decide
when to inline[1], could be better as

return GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55) ||
(IS_ALDERLAKE_P(i915) && !IS_ADLP_N(i915));

Lucas De Marchi

[1] https://lwn.net/Articles/767884/



return false;
--
2.35.3

Re: [Intel-gfx] [PATCH] drm/i915/hwconfig: Future-proof platform checks

2022-05-25 Thread John Harrison 

On 5/24/2022 16:59, Matt Roper wrote:

PVC also has a hwconfig table.  Actually the current expectation is that
all future platforms will have hwconfig, so let's just change the
condition to an IP version check so that we don't need to keep updating
this for each new platform that shows up.

Cc: John Harrison 
Cc: Radhakrishna Sripada 
Signed-off-by: Matt Roper 

Reviewed-by: John Harrison 


---
  drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c 
b/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
index 5aaa3948de74..4781fccc2687 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
@@ -96,7 +96,7 @@ static bool has_table(struct drm_i915_private *i915)
  {
if (IS_ALDERLAKE_P(i915) && !IS_ADLP_N(i915))
return true;
-   if (IS_DG2(i915))
+   if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55))
return true;
  
  	return false;

[Intel-gfx] [PATCH] drm/i915/hwconfig: Future-proof platform checks

2022-05-24 Thread Matt Roper 
PVC also has a hwconfig table.  Actually the current expectation is that
all future platforms will have hwconfig, so let's just change the
condition to an IP version check so that we don't need to keep updating
this for each new platform that shows up.

Cc: John Harrison 
Cc: Radhakrishna Sripada 
Signed-off-by: Matt Roper 
---
 drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c 
b/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
index 5aaa3948de74..4781fccc2687 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
@@ -96,7 +96,7 @@ static bool has_table(struct drm_i915_private *i915)
 {
if (IS_ALDERLAKE_P(i915) && !IS_ADLP_N(i915))
return true;
-   if (IS_DG2(i915))
+   if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 55))
return true;
 
return false;
-- 
2.35.3