+Chris
On Tue, Aug 28, 2018 at 10:41 AM Lucas De Marchi
<lucas.demarchi at intel.com> wrote:
>
> subvendor and subdevice are unsigned, so fix their initialization in
> INTEL_VGA_DEVICE.
>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
> ---
> include/drm/i915_pciids.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h
> index fd965ffbb92e..754ce4b10129 100644
> --- a/include/drm/i915_pciids.h
> +++ b/include/drm/i915_pciids.h
> @@ -37,7 +37,7 @@
> */
> #define INTEL_VGA_DEVICE(id, info) { \
> 0x8086, id, \
> - ~0, ~0, \
> + ~0u, ~0u, \
This came from your suggestion in
https://patchwork.freedesktop.org/patch/245943/
"And then while you are there, add the missing 'u' to ~0u"
Should I resend this simple patch alone (since the second got nacked)?
thanks
Lucas De Marchi
> 0x030000, 0xff0000, \
> (unsigned long) info }
>
> --
> 2.17.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Lucas De Marchi