Fix format warnings in hardware/qcom/display
system/core/include is included in the global include path using -isystem, which hides log format warnings when built with gcc. Fix the warnings in preparation for switching from -isystem to -I. Test: lunch aosp_angler-userdebug && m -j Bug: 31492149 Change-Id: Ie75967aa30bae2e9bebbd65efe0ff570a80203fd
This commit is contained in:
parent
f881bb5247
commit
abe67dd75c
|
@ -136,7 +136,7 @@ static int gralloc_unmap(gralloc_module_t const* module,
|
|||
if(hnd->base) {
|
||||
err = memalloc->unmap_buffer((void*)hnd->base, hnd->size, hnd->offset);
|
||||
if (err) {
|
||||
ALOGE("Could not unmap memory at address %p, %s", hnd->base,
|
||||
ALOGE("Could not unmap memory at address %p, %s", (void*) hnd->base,
|
||||
strerror(errno));
|
||||
return -errno;
|
||||
}
|
||||
|
@ -149,7 +149,7 @@ static int gralloc_unmap(gralloc_module_t const* module,
|
|||
size, hnd->offset_metadata);
|
||||
if (err) {
|
||||
ALOGE("Could not unmap memory at address %p, %s",
|
||||
hnd->base_metadata, strerror(errno));
|
||||
(void*) hnd->base_metadata, strerror(errno));
|
||||
return -errno;
|
||||
}
|
||||
hnd->base_metadata = 0;
|
||||
|
|
|
@ -783,7 +783,7 @@ int HDMIDisplay::getAttrForConfig(int config, uint32_t& xres,
|
|||
int HDMIDisplay::getDisplayConfigs(uint32_t* configs,
|
||||
size_t* numConfigs) const {
|
||||
if (*numConfigs <= 0) {
|
||||
ALOGE("%s Invalid number of configs (%d)", __FUNCTION__, *numConfigs);
|
||||
ALOGE("%s Invalid number of configs (%zu)", __FUNCTION__, *numConfigs);
|
||||
return -EINVAL;
|
||||
}
|
||||
*numConfigs = mModeCount;
|
||||
|
|
|
@ -794,7 +794,7 @@ int hwc_getDisplayConfigs(struct hwc_composer_device_1* dev, int disp,
|
|||
}
|
||||
|
||||
if (*numConfigs <= 0) {
|
||||
ALOGE("%s Invalid number of configs (%d)", __FUNCTION__, *numConfigs);
|
||||
ALOGE("%s Invalid number of configs (%zu)", __FUNCTION__, *numConfigs);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
|
|
|
@ -1401,7 +1401,7 @@ bool MDPComp::mdpOnlyLayersComp(hwc_context_t *ctx,
|
|||
/* Bail out if we are processing only secured video/ui layers
|
||||
* and we dont have any */
|
||||
if(secureOnly) {
|
||||
ALOGD_IF(isDebug(), "%s: No secure video/ui layers");
|
||||
ALOGD_IF(isDebug(), "%s: No secure video/ui layers", __FUNCTION__);
|
||||
return false;
|
||||
}
|
||||
/* No Idle fall back for secure video/ui layers and if there is only
|
||||
|
|
Loading…
Reference in New Issue