[Mesa-dev] [PATCH 24/24] i965: Simplify get_l3_way_size() function

[Kenneth Graunke]

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.phogat at 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 <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170513/51e163f9/attachment.sig>