From 17f1383e33401f72e9e5df9f795725abeb770cc0 Mon Sep 17 00:00:00 2001 From: Jigar Agrawal Date: Thu, 23 Jul 2020 12:22:44 -0700 Subject: [PATCH] msm: camera: common: Improve IFE and CSIPhy logging Remove unnecessary logs from ife start, stop and release routines. Add secure mode to CSIPHY logging. CRs-Fixed: 2709537 Change-Id: Ib58d008cff5fef3d75f19c88aab76a85b5dc2e89 Signed-off-by: Jigar Agrawal Signed-off-by: Jeyaprakash Soundrapandian --- drivers/cam_isp/isp_hw_mgr/cam_ife_hw_mgr.c | 11 +---------- .../cam_sensor_module/cam_csiphy/cam_csiphy_core.c | 3 ++- 2 files changed, 3 insertions(+), 11 deletions(-) diff --git a/drivers/cam_isp/isp_hw_mgr/cam_ife_hw_mgr.c b/drivers/cam_isp/isp_hw_mgr/cam_ife_hw_mgr.c index 233a9392a6e1..b673368258ac 100644 --- a/drivers/cam_isp/isp_hw_mgr/cam_ife_hw_mgr.c +++ b/drivers/cam_isp/isp_hw_mgr/cam_ife_hw_mgr.c @@ -406,9 +406,7 @@ static int cam_ife_hw_mgr_start_hw_res( "Can not start HW:%d resources", hw_intf->hw_idx); goto err; - } else - CAM_INFO(CAM_ISP, "Started HW:%d", - hw_intf->hw_idx); + } } else { CAM_ERR(CAM_ISP, "function null"); goto err; @@ -442,8 +440,6 @@ static void cam_ife_hw_mgr_stop_hw_res( hw_intf->hw_ops.stop(hw_intf->hw_priv, isp_hw_res->hw_res[i], sizeof(struct cam_isp_resource_node)); - - CAM_INFO(CAM_ISP, "Stopped Hw:%d", hw_intf->hw_idx); } else CAM_ERR(CAM_ISP, "stop null"); @@ -652,11 +648,6 @@ static int cam_ife_hw_mgr_free_hw_res( CAM_ERR(CAM_ISP, "Release HW:%d resource id %d failed", hw_intf->hw_idx, isp_hw_res->res_id); - else - CAM_DBG(CAM_ISP, - "Released HW:%d resource id %d", - hw_intf->hw_idx, isp_hw_res->res_id); - isp_hw_res->hw_res[i] = NULL; } else CAM_ERR(CAM_ISP, "Release null"); diff --git a/drivers/cam_sensor_module/cam_csiphy/cam_csiphy_core.c b/drivers/cam_sensor_module/cam_csiphy/cam_csiphy_core.c index cd046c4eacc5..b3ff2435b08b 100644 --- a/drivers/cam_sensor_module/cam_csiphy/cam_csiphy_core.c +++ b/drivers/cam_sensor_module/cam_csiphy/cam_csiphy_core.c @@ -1264,8 +1264,9 @@ int32_t cam_csiphy_core_cfg(void *phy_dev, } CAM_INFO(CAM_CSIPHY, - "START_DEV: CSIPHY_IDX: %d, Device_slot: %d, Datarate: %llu, Settletime: %llu", + "CAM_START_PHYDEV: CSIPHY_IDX: %d, Device_slot: %d, cp_mode: %d, Datarate: %llu, Settletime: %llu", csiphy_dev->soc_info.index, offset, + csiphy_dev->csiphy_info[offset].secure_mode, csiphy_dev->csiphy_info[offset].data_rate, csiphy_dev->csiphy_info[offset].settle_time);