Commit 2f77d82e authored by Adam Jackson's avatar Adam Jackson Committed by Ville Syrjälä

drm/fourcc: Fix undefined left shift in DRM_FORMAT_BIG_ENDIAN macros

1<<31 is undefined because it's a signed int and C is terrible.
Reviewed-by: default avatarEric Engestrom <eric.engestrom@intel.com>
Signed-off-by: default avatarAdam Jackson <ajax@redhat.com>
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20191018175041.613780-1-ajax@redhat.com
parent 2093dea3
...@@ -69,7 +69,7 @@ extern "C" { ...@@ -69,7 +69,7 @@ extern "C" {
#define fourcc_code(a, b, c, d) ((__u32)(a) | ((__u32)(b) << 8) | \ #define fourcc_code(a, b, c, d) ((__u32)(a) | ((__u32)(b) << 8) | \
((__u32)(c) << 16) | ((__u32)(d) << 24)) ((__u32)(c) << 16) | ((__u32)(d) << 24))
#define DRM_FORMAT_BIG_ENDIAN (1<<31) /* format is big endian instead of little endian */ #define DRM_FORMAT_BIG_ENDIAN (1U<<31) /* format is big endian instead of little endian */
/* Reserve 0 for the invalid format specifier */ /* Reserve 0 for the invalid format specifier */
#define DRM_FORMAT_INVALID 0 #define DRM_FORMAT_INVALID 0
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment