On Thu, Mar 07, 2019 at 12:56:55PM -0800, José Roberto de Souza wrote:
> Lets keep it sorted to make easy to spot missing PCI IDs.

Hmm... In my opinion leaving it as the identical order
of the spec is the way to make it easier to spot if we missed
something...

Otherwise when reviewing I have to stop and sort one or another.

> 
> Cc: Rodrigo Vivi <[email protected]>
> Signed-off-by: José Roberto de Souza <[email protected]>
> ---
>  include/drm/i915_pciids.h | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h
> index d2fad7b0fcf6..be735811591d 100644
> --- a/include/drm/i915_pciids.h
> +++ b/include/drm/i915_pciids.h
> @@ -459,16 +459,16 @@
>  #define INTEL_ICL_11_IDS(info) \
>       INTEL_VGA_DEVICE(0x8A50, info), \
>       INTEL_VGA_DEVICE(0x8A51, info), \
> -     INTEL_VGA_DEVICE(0x8A5C, info), \
> -     INTEL_VGA_DEVICE(0x8A5D, info), \
> -     INTEL_VGA_DEVICE(0x8A59, info), \
> -     INTEL_VGA_DEVICE(0x8A58, info), \
>       INTEL_VGA_DEVICE(0x8A52, info), \
> +     INTEL_VGA_DEVICE(0x8A56, info), \
> +     INTEL_VGA_DEVICE(0x8A57, info), \
> +     INTEL_VGA_DEVICE(0x8A58, info), \
> +     INTEL_VGA_DEVICE(0x8A59, info), \
>       INTEL_VGA_DEVICE(0x8A5A, info), \
>       INTEL_VGA_DEVICE(0x8A5B, info), \
> -     INTEL_VGA_DEVICE(0x8A57, info), \
> -     INTEL_VGA_DEVICE(0x8A56, info), \
> -     INTEL_VGA_DEVICE(0x8A71, info), \
> -     INTEL_VGA_DEVICE(0x8A70, info)
> +     INTEL_VGA_DEVICE(0x8A5C, info), \
> +     INTEL_VGA_DEVICE(0x8A5D, info), \
> +     INTEL_VGA_DEVICE(0x8A70, info), \
> +     INTEL_VGA_DEVICE(0x8A71, info)
>  
>  #endif /* _I915_PCIIDS_H */
> -- 
> 2.21.0
> 
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to