From fe6842e2747845aca6b74191c8f7717f156d6127 Mon Sep 17 00:00:00 2001 From: Govindaraj Rajagopal Date: Fri, 6 Nov 2020 11:15:36 +0530 Subject: [PATCH] msm: vidc: fix format string mismatch Some debug prints doesn't contain proper args. So added proper args. Change-Id: I36fe3e967b3bf600b3a948cb88f556a4d93c02d1 Signed-off-by: Govindaraj Rajagopal --- msm/vidc/msm_venc.c | 8 ++++---- msm/vidc/msm_vidc_clocks.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/msm/vidc/msm_venc.c b/msm/vidc/msm_venc.c index 55e05e41e5a6..fe2da7816dac 100644 --- a/msm/vidc/msm_venc.c +++ b/msm/vidc/msm_venc.c @@ -1555,7 +1555,7 @@ static int msm_venc_update_bitrate(struct msm_vidc_inst *inst) u32 cabac_max_bitrate = 0; if (!inst) { - d_vpr_e("%s: invalid params %pK\n", __func__); + d_vpr_e("%s: invalid params\n", __func__); return -EINVAL; } @@ -1908,17 +1908,17 @@ int msm_venc_s_ctrl(struct msm_vidc_inst *inst, struct v4l2_ctrl *ctrl) } } else if (inst->state == MSM_VIDC_START_DONE) { if (!inst->external_blur) { - s_vpr_e(sid, "external blur not enabled"); + s_vpr_e(sid, "%s: external blur not enabled", __func__); break; } if (ctrl->val == MSM_VIDC_BLUR_EXTERNAL_DYNAMIC) { s_vpr_h(sid, - "external blur setting already enabled\n", + "%s: external blur setting already enabled\n", __func__); break; } else if (ctrl->val == MSM_VIDC_BLUR_INTERNAL) { s_vpr_e(sid, - "cannot change to internal blur config dynamically\n", + "%s: cannot change to internal blur config dynamically\n", __func__); break; } else { diff --git a/msm/vidc/msm_vidc_clocks.c b/msm/vidc/msm_vidc_clocks.c index 4185500e5bdd..aa8825caa007 100644 --- a/msm/vidc/msm_vidc_clocks.c +++ b/msm/vidc/msm_vidc_clocks.c @@ -1444,7 +1444,7 @@ static inline int msm_vidc_power_save_mode_enable(struct msm_vidc_inst *inst, sizeof(hfi_perf_mode)); if (rc) { s_vpr_e(inst->sid, "%s: Failed to set power save mode\n", - __func__, inst); + __func__); return rc; } inst->flags = enable ?