Re: [igt-dev] [PATCH i-g-t 1/6] tools/intel_watermark: Bump number of planes to 8 for icl

Souza, Jose
2019-01-28T22:16:53Z

[-- Attachment #1.1: Type: text/plain, Size: 1239 bytes --]

On Mon, 2019-01-28 at 22:01 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> icl has 8 planes per pipe. Bump the limits to match.

Reviewed-by: José Roberto de Souza <jose.souza@intel.com>

>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> tools/intel_watermark.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/tools/intel_watermark.c b/tools/intel_watermark.c
> index 3fd37441cba7..b61a26f1cd57 100644
> --- a/tools/intel_watermark.c
> +++ b/tools/intel_watermark.c
> @@ -137,7 +137,9 @@ static int is_hsw_plus(uint32_t d)
>
> static int skl_num_planes(uint32_t d, int pipe)
> {
> - if (IS_GEN10(d) || IS_GEMINILAKE(d))
> + if (IS_GEN11(d))
> + return 8;
> + else if (IS_GEN10(d) || IS_GEMINILAKE(d))
> return 5;
> else if (IS_BROXTON(d))
> return pipe == 2 ? 4 : 5;
> @@ -147,7 +149,9 @@ static int skl_num_planes(uint32_t d, int pipe)
>
> static int skl_max_planes(uint32_t d)
> {
> - if (IS_GEN10(d) || IS_GEMINILAKE(d) || IS_BROXTON(d))
> + if (IS_GEN11(d))
> + return 8;
> + else if (IS_GEN10(d) || IS_GEMINILAKE(d) || IS_BROXTON(d))
> return 5;
> else
> return 4;

[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 154 bytes --]