Chris Wilson <ch...@chris-wilson.co.uk> writes:

> Be consistent and define the device's GEN as part of the GENx_FEATURE.
> It will be overridden by the next gen upon inheriting, as per usual.
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>

Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com>

> ---
>  drivers/gpu/drm/i915/i915_pci.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index 3b4516d7f9a4..364d3b41f816 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -383,6 +383,7 @@ static const struct intel_device_info 
> intel_haswell_gt3_info = {
>  
>  #define GEN8_FEATURES \
>       G75_FEATURES, \
> +     GEN(8), \
>       BDW_COLORS, \
>       .page_sizes = I915_GTT_PAGE_SIZE_4K | \
>                     I915_GTT_PAGE_SIZE_2M, \
> @@ -393,7 +394,6 @@ static const struct intel_device_info 
> intel_haswell_gt3_info = {
>  
>  #define BDW_PLATFORM \
>       GEN8_FEATURES, \
> -     GEN(8), \
>       .platform = INTEL_BROADWELL
>  
>  static const struct intel_device_info intel_broadwell_gt1_info = {
> @@ -452,6 +452,7 @@ static const struct intel_device_info 
> intel_cherryview_info = {
>  
>  #define GEN9_FEATURES \
>       GEN8_FEATURES, \
> +     GEN(9), \
>       GEN9_DEFAULT_PAGE_SIZES, \
>       .has_logical_ring_preemption = 1, \
>       .has_csr = 1, \
> @@ -461,7 +462,6 @@ static const struct intel_device_info 
> intel_cherryview_info = {
>  
>  #define SKL_PLATFORM \
>       GEN9_FEATURES, \
> -     GEN(9), \
>       .platform = INTEL_SKYLAKE
>  
>  static const struct intel_device_info intel_skylake_gt1_info = {
> @@ -535,7 +535,6 @@ static const struct intel_device_info 
> intel_geminilake_info = {
>  
>  #define KBL_PLATFORM \
>       GEN9_FEATURES, \
> -     GEN(9),  \
>       .platform = INTEL_KABYLAKE
>  
>  static const struct intel_device_info intel_kabylake_gt1_info = {
> @@ -556,7 +555,6 @@ static const struct intel_device_info 
> intel_kabylake_gt3_info = {
>  
>  #define CFL_PLATFORM \
>       GEN9_FEATURES, \
> -     GEN(9), \
>       .platform = INTEL_COFFEELAKE
>  
>  static const struct intel_device_info intel_coffeelake_gt1_info = {
> @@ -577,12 +575,12 @@ static const struct intel_device_info 
> intel_coffeelake_gt3_info = {
>  
>  #define GEN10_FEATURES \
>       GEN9_FEATURES, \
> +     GEN(10), \
>       .ddb_size = 1024, \
>       GLK_COLORS
>  
>  static const struct intel_device_info intel_cannonlake_info = {
>       GEN10_FEATURES,
> -     GEN(10),
>       .is_alpha_support = 1,
>       .platform = INTEL_CANNONLAKE,
>       .gt = 2,
> -- 
> 2.16.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to