commit | 152bc62dc8ec4c38ec18ff917062f29d38f25744 | [log] [tgz] |
---|---|---|
author | Yongqin Liu <yongqin.liu@linaro.org> | Sun Jan 29 00:48:10 2023 +0800 |
committer | Yongqin Liu <yongqin.liu@linaro.org> | Sun Jan 29 00:48:10 2023 +0800 |
tree | d093cc45e102e5f962cba77b218bed3cdea6e4cc | |
parent | 22fe9617dab9c0568ce625143b1d064c95df8fa2 [diff] |
drm_hwcomposer: fix shift compiling error with AOSP which reported like this: 08:58:21 external/drm_hwcomposer/hwc2_device/HwcDisplay.cpp:76:26: error: constexpr variable 'kOne' must be initialized by a constant expression 08:58:21 constexpr uint64_t kOne = 1L << 32; /* 1.0 in s31.32 format */ 08:58:21 ^ ~~~~~~~~ 08:58:21 external/drm_hwcomposer/hwc2_device/HwcDisplay.cpp:76:36: note: shift count 32 >= width of type 'long' (32 bits) 08:58:21 constexpr uint64_t kOne = 1L << 32; /* 1.0 in s31.32 format */ 08:58:21 ^ 08:58:21 external/drm_hwcomposer/hwc2_device/HwcDisplay.cpp:76:36: error: shift count >= width of type [-Werror,-Wshift-count-overflow] 08:58:21 constexpr uint64_t kOne = 1L << 32; /* 1.0 in s31.32 format */ 08:58:21 ^ ~~ 08:58:21 external/drm_hwcomposer/hwc2_device/HwcDisplay.cpp:697:71: error: shift count >= width of type [-Werror,-Wshift-count-overflow] 08:58:21 auto value = uint64_t(matrix[i * kInCtmRows + j] * float(1L << 32)); 08:58:21 ^ ~~ 08:58:21 3 errors generated. Test: build passes and could boot to home screen Change-Id: Ia9dfcd842a186dfbe279da1793363c6a25df0e2f Signed-off-by: Yongqin Liu <yongqin.liu@linaro.org>
Patches to drm_hwcomposer are very much welcome, we really want this to be the universal HW composer implementation for Android and similar platforms. So please bring on porting patches, bugfixes, improvements for documentation and new features.
A short list of contribution guidelines:
Submit changes via gitlab merge requests on gitlab.freedesktop.org.
drm_hwcomposer is Apache 2.0 Licensed and we require contributions to follow the developer's certificate of origin: http://developercertificate.org/.
When submitting new code please follow the naming conventions documented in the generated documentation. Also please make full use of all the helpers and convenience macros provided by drm_hwcomposer. The below command can help you with formatting of your patches:
git diff | clang-format-diff-15 -p 1 -style=file
Hardware specific changes should be tested on relevant platforms before committing.
If you need inspiration, please checkout our TODO issues.
Happy hacking!