diff options
| author | Lang Dai <langx.dai@intel.com> | 2017-05-04 16:18:11 +0800 |
|---|---|---|
| committer | Daniel Cardenas <danielcar@google.com> | 2017-05-08 20:10:10 -0700 |
| commit | cc0ec6239a14453c8e9983cef2104d22c73cc892 (patch) | |
| tree | 1850853c26aeaf4a194cab09cab18f5fbc06919c | |
| parent | b4a718c0d8f97a70f1b3fd08a33258df98b837ed (diff) | |
| download | hwcomposer-cc0ec6239a14453c8e9983cef2104d22c73cc892.tar.gz | |
Fixed Klocwork warnings.
BZ: IMINAN-51481
Fixes: 38135184
Test: Netflix, play movies, youtube
Change-Id: I67c5f1353f008035afa2492937a12818c1bff1a1
Signed-off-by: Austin Hu <austin.hu@intel.com>
(cherry picked from commit fe3bb5afb0875ca02b6232ec860ae2e33b0b4d17)
| -rw-r--r-- | moorefield_hdmi/ips/common/WsbmWrapper.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/moorefield_hdmi/ips/common/WsbmWrapper.c b/moorefield_hdmi/ips/common/WsbmWrapper.c index 8d7c275..cdbf951 100644 --- a/moorefield_hdmi/ips/common/WsbmWrapper.c +++ b/moorefield_hdmi/ips/common/WsbmWrapper.c @@ -136,7 +136,8 @@ int psbWsbmInitialize(int drmFD) goto out; } - VLOGTRACE("ioctl offset %#x", arg.rep.driver_ioctl_offset); + unsigned int ioctl_offset = arg.rep.driver_ioctl_offset; + ILOGTRACE("ioctl offset %#x", ioctl_offset); mainPool = wsbmTTMPoolInit(drmFD, arg.rep.driver_ioctl_offset); if(!mainPool) { @@ -356,8 +357,9 @@ void * psbWsbmGetCPUAddress(void * buf) return NULL; } + unsigned long buf_size = wsbmBOSize((struct _WsbmBufferObject *)buf); VLOGTRACE("mapped successfully. %p, size %ld", - address, wsbmBOSize((struct _WsbmBufferObject *)buf)); + address, buf_size); return address; } @@ -374,7 +376,8 @@ uint32_t psbWsbmGetGttOffset(void * buf) uint32_t offset = wsbmBOOffsetHint((struct _WsbmBufferObject *)buf) - 0x10000000; - VLOGTRACE("offset %#x", offset >> 12); + uint32_t offset_tmp = offset >> 12; + VLOGTRACE("offset %#x", offset_tmp); return offset >> 12; } |