On Friday, May 12, 2017 4:38:28 PM PDT Anuj Phogat wrote:
> Cherryview and Broxton are always gt1. So, remove the redundant checks.
> 
> Signed-off-by: Anuj Phogat <anuj.pho...@gmail.com>
> ---
>  src/intel/common/gen_l3_config.c | 10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/src/intel/common/gen_l3_config.c 
> b/src/intel/common/gen_l3_config.c
> index 4fe3503..0720079 100644
> --- a/src/intel/common/gen_l3_config.c
> +++ b/src/intel/common/gen_l3_config.c
> @@ -258,13 +258,11 @@ get_l3_way_size(const struct gen_device_info *devinfo)
>     if (devinfo->is_baytrail)
>        return 2;
>  
> -   else if (devinfo->gt == 1 ||
> -            devinfo->is_cherryview ||
> -            devinfo->is_broxton)
> -      return 4;
> +   /* Cherryview and Broxton are always gt1 */
> +   if (devinfo->gt == 1)
> +       return 4;
>  
> -   else
> -      return 8 * devinfo->num_slices;
> +   return 8 * devinfo->num_slices;
>  }
>  
>  /**
> 

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to