+Chris On Tue, Aug 28, 2018 at 10:41 AM Lucas De Marchi <lucas.demarchi@xxxxxxxxx> wrote: > > subvendor and subdevice are unsigned, so fix their initialization in > INTEL_VGA_DEVICE. > > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx> > --- > 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@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Lucas De Marchi _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx