[Intel-gfx] include/drm/i915_drm.h:96: possible bad bitmask ?
Daniel Vetter
daniel at ffwll.ch
Mon Aug 8 09:40:27 UTC 2016
On Mon, Aug 08, 2016 at 10:31:32AM +0100, David Binderman wrote:
> Hello there,
>
> Recent versions of gcc say this:
>
> include/drm/i915_drm.h:96:34: warning: result of β65535 << 20β
> requires 37 bits to represent, but βintβ only has 32 bits
> [-Wshift-overflow=]
>
> Source code is
>
> #define INTEL_BSM_MASK (0xFFFF << 20)
>
> Maybe something like
>
> #define INTEL_BSM_MASK (0xFFFFUL<< 20)
>
> might be better.
Yup. Care to bake this into a patch (with s-o-b and everything per
Documentation/SubmittingPatches) so I can apply it?
-Daniel
>
>
> Regards
>
> David Binderman
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the Intel-gfx
mailing list