[Intel-gfx] [PATCH i-g-t] intel: Adding missing Broxton PCI IDs.

[Arun Siluvery]

On 17/03/2016 21:39, Rodrigo Vivi wrote:
> These IDs were already part of the kernel since:
>
> kernel commit 985dd4360fdf2533fe48a33a4a2094f2e4718dc0
> Author: Imre Deak <imre.deak at intel.com>
> Date:   Thu Jan 28 16:04:12 2016 +0200
>
>       drm/i915/bxt: update list of PCIIDs
>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> ---
>   lib/intel_chipset.h | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
> index f3bd04f..4b40612 100644
> --- a/lib/intel_chipset.h
> +++ b/lib/intel_chipset.h
> @@ -236,6 +236,8 @@ void intel_check_pch(void);
>   #define PCI_CHIP_BROXTON_0		0x0A84
>   #define PCI_CHIP_BROXTON_1		0x1A84
>   #define PCI_CHIP_BROXTON_2		0x5A84
> +#define PCI_CHIP_BROXTON_3		0x1A85
> +#define PCI_CHIP_BROXTON_4		0x5A85
>
>   #endif /* __GTK_DOC_IGNORE__ */
>
> @@ -460,7 +462,9 @@ void intel_check_pch(void);
>
>   #define IS_BROXTON(devid)	((devid) == PCI_CHIP_BROXTON_0 || \
>   				 (devid) == PCI_CHIP_BROXTON_1 || \
> -				 (devid) == PCI_CHIP_BROXTON_2)
> +				 (devid) == PCI_CHIP_BROXTON_2 || \
> +				 (devid) == PCI_CHIP_BROXTON_3 || \
> +				 (devid) == PCI_CHIP_BROXTON_4)
>
>   #define IS_GEN9(devid)		(IS_KABYLAKE(devid) || \
>   				 IS_SKYLAKE(devid) || \
>
I have recently added them to libdrm in android, would you also be 
adding them to libdrm?

looks good to me,
Reviewed-by: Arun Siluvery <arun.siluvery at linux.intel.com>

regards
Arun