Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmc: Make no_stepping_info an array

[Vivi, Rodrigo]

Reviewed-by: Rodrigo Vivi 

On Fri, 2016-10-14 at 16:56 -0700, Anusha Srivatsa wrote:
> Make no_stepping_info an array of structs so that
> on plaforms that have only one binary of DMC, we can
> iterate through this array by having the same logic
> for firmware loads
> 
> Cc: Rodrigo Vivi 
> Signed-off-by: Anusha Srivatsa 
> ---
>  drivers/gpu/drm/i915/intel_csr.c | 6 --
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_csr.c 
> b/drivers/gpu/drm/i915/intel_csr.c
> index 1ea0e1f..cf57167 100644
> --- a/drivers/gpu/drm/i915/intel_csr.c
> +++ b/drivers/gpu/drm/i915/intel_csr.c
> @@ -186,7 +186,9 @@ static const struct stepping_info bxt_stepping_info[] = {
>   {'B', '0'}, {'B', '1'}, {'B', '2'}
>  };
>  
> -static const struct stepping_info no_stepping_info = { '*', '*' };
> +static const struct stepping_info no_stepping_info[] = {
> + { '*', '*' }
> +};
>  
>  static const struct stepping_info *
>  intel_get_stepping_info(struct drm_i915_private *dev_priv)
> @@ -210,7 +212,7 @@ intel_get_stepping_info(struct drm_i915_private *dev_priv)
>   if (INTEL_REVID(dev_priv) < size)
>   return si + INTEL_REVID(dev_priv);
>  
> - return _stepping_info;
> + return no_stepping_info;
>  }
>  
>  static void gen9_set_dc_state_debugmask(struct drm_i915_private *dev_priv)

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

[Intel-gfx] [PATCH 1/2] drm/i915/dmc: Make no_stepping_info an array

[Anusha Srivatsa]

Make no_stepping_info an array of structs so that
on plaforms that have only one binary of DMC, we can
iterate through this array by having the same logic
for firmware loads

Cc: Rodrigo Vivi 
Signed-off-by: Anusha Srivatsa 
---
 drivers/gpu/drm/i915/intel_csr.c | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
index 1ea0e1f..cf57167 100644
--- a/drivers/gpu/drm/i915/intel_csr.c
+++ b/drivers/gpu/drm/i915/intel_csr.c
@@ -186,7 +186,9 @@ static const struct stepping_info bxt_stepping_info[] = {
{'B', '0'}, {'B', '1'}, {'B', '2'}
 };
 
-static const struct stepping_info no_stepping_info = { '*', '*' };
+static const struct stepping_info no_stepping_info[] = {
+   { '*', '*' }
+};
 
 static const struct stepping_info *
 intel_get_stepping_info(struct drm_i915_private *dev_priv)
@@ -210,7 +212,7 @@ intel_get_stepping_info(struct drm_i915_private *dev_priv)
if (INTEL_REVID(dev_priv) < size)
return si + INTEL_REVID(dev_priv);
 
-   return _stepping_info;
+   return no_stepping_info;
 }
 
 static void gen9_set_dc_state_debugmask(struct drm_i915_private *dev_priv)
-- 
2.7.4

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