[Intel-gfx] [PATCH i-g-t 2/2] lib/intel_chipset: Removing PCI IDs that are no longer listed as Kabylake.
Pandiyan, Dhinakaran
dhinakaran.pandiyan at intel.com
Tue Jun 28 18:33:44 UTC 2016
On Mon, 2016-06-27 at 17:11 -0700, Rodrigo Vivi wrote:
> This is unusual. Usually IDs listed on early stages of platform
> definition are kept there as reserved for later use.
>
> However these IDs here are not listed anymore in any of steppings
> and devices IDs tables for Kabylake on configurations overview
> section of BSpec.
>
> So it is better removing them before they become used in any
> other future platform.
>
> v2: Rebase.
>
> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> ---
> lib/intel_chipset.h | 16 +++-------------
> 1 file changed, 3 insertions(+), 13 deletions(-)
>
> diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
> index 23d0919..e1e552c 100644
> --- a/lib/intel_chipset.h
> +++ b/lib/intel_chipset.h
> @@ -223,18 +223,13 @@ void intel_check_pch(void);
> #define PCI_CHIP_KABYLAKE_DT_GT2 0x5912
> #define PCI_CHIP_KABYLAKE_DT_GT1_5 0x5917
> #define PCI_CHIP_KABYLAKE_DT_GT1 0x5902
> -#define PCI_CHIP_KABYLAKE_DT_GT4 0x5932
> #define PCI_CHIP_KABYLAKE_HALO_GT2 0x591B
> -#define PCI_CHIP_KABYLAKE_HALO_GT3 0x592B
> #define PCI_CHIP_KABYLAKE_HALO_GT1_0 0x5908
> #define PCI_CHIP_KABYLAKE_HALO_GT1_1 0x590B
> #define PCI_CHIP_KABYLAKE_HALO_GT4 0x593B
> #define PCI_CHIP_KABYLAKE_SRV_GT2 0x591A
> -#define PCI_CHIP_KABYLAKE_SRV_GT3 0x592A
> -#define PCI_CHIP_KABYLAKE_SRV_GT4 0x593A
> #define PCI_CHIP_KABYLAKE_SRV_GT1 0x590A
> #define PCI_CHIP_KABYLAKE_WKS_GT2 0x591D
> -#define PCI_CHIP_KABYLAKE_WKS_GT4 0x593D
>
> #define PCI_CHIP_BROXTON_0 0x0A84
> #define PCI_CHIP_BROXTON_1 0x1A84
> @@ -447,14 +442,9 @@ void intel_check_pch(void);
>
> #define IS_KBL_GT3(devid) ((devid) == PCI_CHIP_KABYLAKE_ULT_GT3_0|| \
> (devid) == PCI_CHIP_KABYLAKE_ULT_GT3_1|| \
> - (devid) == PCI_CHIP_KABYLAKE_ULT_GT3_2|| \
> - (devid) == PCI_CHIP_KABYLAKE_HALO_GT3|| \
> - (devid) == PCI_CHIP_KABYLAKE_SRV_GT3)
> -
> -#define IS_KBL_GT4(devid) ((devid) == PCI_CHIP_KABYLAKE_DT_GT4|| \
> - (devid) == PCI_CHIP_KABYLAKE_HALO_GT4|| \
> - (devid) == PCI_CHIP_KABYLAKE_SRV_GT4|| \
> - (devid) == PCI_CHIP_KABYLAKE_WKS_GT4)
> + (devid) == PCI_CHIP_KABYLAKE_ULT_GT3_2)
> +
> +#define IS_KBL_GT4(devid) ((devid) == PCI_CHIP_KABYLAKE_HALO_GT4)
>
> #define IS_KABYLAKE(devid) (IS_KBL_GT1(devid) || \
> IS_KBL_GT2(devid) || \
Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
More information about the Intel-gfx
mailing list