Merge lp:~vanvugt/mir/android-prefer-truecolor into lp:mir
Status: | Work in progress |
---|---|
Proposed branch: | lp:~vanvugt/mir/android-prefer-truecolor |
Merge into: | lp:mir |
Diff against target: |
26 lines (+6/-0) 2 files modified
src/platforms/android/server/gl_context.cpp (+3/-0) src/platforms/android/server/hwc_blanking_control.cpp (+3/-0) |
To merge this branch: | bzr merge lp:~vanvugt/mir/android-prefer-truecolor |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel van Vugt | Abstain | ||
Mir CI Bot | continuous-integration | Needs Fixing | |
Review via email: mp+296172@code.launchpad.net |
Commit message
Android: Prefer the highest colour depth (>= 15 bit) available instead
of defaulting the lowest depth.
If the Android driver follows the EGL spec it will now choose the highest
colour depth. Previously we defaulted to zeros which meant we were asking
for the lowest colour depth. It was only by chance that we supported
devices with buggy EGL implementations before now that didn't follow the
spec and they gave us true colour erroneously.
https:/
https:/
Description of the change
UNTESTED proof of concept for morphis. Theoretically this can't make the problem any worse at least.
I know it's counter-intuitive but asking for 5 bits is the best way to get 8 bits. Because EGL says that any value greater than zero will give you the maximum colour depth available. And asking for 5 means your code will still work if only high colour is available.
Unmerged revisions
- 3526. By Daniel van Vugt
-
Proof of concept
FAILED: Continuous integration, rev:3526 /mir-jenkins. ubuntu. com/job/ mir-ci/ 1062/ /mir-jenkins. ubuntu. com/job/ build-mir/ 1171/console /mir-jenkins. ubuntu. com/job/ build-0- fetch/1219 /mir-jenkins. ubuntu. com/job/ build-1- sourcepkg/ release= vivid+overlay/ 1210 /mir-jenkins. ubuntu. com/job/ build-1- sourcepkg/ release= xenial/ 1210 /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= amd64,compiler= clang,platform= mesa,release= vivid+overlay/ 1181 /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= amd64,compiler= clang,platform= mesa,release= vivid+overlay/ 1181/artifact/ output/ *zip*/output. zip /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= amd64,compiler= gcc,platform= mesa,release= xenial/ 1181 /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= amd64,compiler= gcc,platform= mesa,release= xenial/ 1181/artifact/ output/ *zip*/output. zip /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= cross-armhf, compiler= gcc,platform= android, release= vivid+overlay/ 1181/console /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= cross-armhf, compiler= gcc,platform= android, release= vivid+overlay/ 1181/artifact/ output/ *zip*/output. zip /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= i386,compiler= gcc,platform= android, release= vivid+overlay/ 1181/console /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= i386,compiler= gcc,platform= mesa,release= xenial/ 1181 /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= i386,compiler= gcc,platform= mesa,release= xenial/ 1181/artifact/ output/ *zip*/output. zip
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
FAILURE: https:/
deb: https:/
FAILURE: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild: /mir-jenkins. ubuntu. com/job/ mir-ci/ 1062/rebuild
https:/