summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChih-Wei Huang <cwhuang@linux.org.tw>2016-07-11 19:37:18 +0800
committerChih-Wei Huang <cwhuang@linux.org.tw>2016-07-11 19:56:50 +0800
commitc6a6d36255eb6f835e7d6e1c2e2ab4c8ffa9d143 (patch)
tree726e985fd4282505d57fdcc095eac6ad9b811134
parenta9d0cbf474511070c13990b1a1ba7a22bba62b8e (diff)
downloadexternal_drm_gralloc-c6a6d36255eb6f835e7d6e1c2e2ab4c8ffa9d143.zip
external_drm_gralloc-c6a6d36255eb6f835e7d6e1c2e2ab4c8ffa9d143.tar.gz
external_drm_gralloc-c6a6d36255eb6f835e7d6e1c2e2ab4c8ffa9d143.tar.bz2
gralloc_drm: show the unsupported error if drv is null
The unsupported driver message was suppressed by a subtle bug of commit cbf96c0d which treats the latest checking as an else case. Remove the unnecessary else. Besides, logically on entering ENABLE_PIPE snippet, drv is always null. No need to check it.
-rw-r--r--gralloc_drm.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/gralloc_drm.c b/gralloc_drm.c
index 40342cb..49ae7d7 100644
--- a/gralloc_drm.c
+++ b/gralloc_drm.c
@@ -93,10 +93,10 @@ init_drv_from_fd(int fd)
} else
#endif
#ifdef ENABLE_PIPE
- if (!drv) {
+ {
drv = gralloc_drm_drv_create_for_pipe(fd, version->name);
ALOGI_IF(drv, "create pipe for driver %s", version->name);
- } else
+ }
#endif
if (!drv) {
ALOGE("unsupported driver: %s", (version->name) ?