From: Dima Zavin <dima@android.com>
Date: Fri, 19 Nov 2010 21:20:15 +0000 (-0800)
Subject: media: video: tegra: avp: turn off chatty avp/rpc debug logging
X-Git-Tag: firefly_0821_release~9833^2~99^2~6
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=888f2866312dcbce0f0d1ebaefa3bce895a916a4;p=firefly-linux-kernel-4.4.55.git

media: video: tegra: avp: turn off chatty avp/rpc debug logging

Change-Id: I3f9025285fc5a002dbd436ee76bb6ca757c425ea
Signed-off-by: Dima Zavin <dima@android.com>
---

diff --git a/drivers/media/video/tegra/avp/avp.c b/drivers/media/video/tegra/avp/avp.c
index 6affcd341b05..ced838ac6e2b 100644
--- a/drivers/media/video/tegra/avp/avp.c
+++ b/drivers/media/video/tegra/avp/avp.c
@@ -59,9 +59,7 @@ enum {
 	AVP_DBG_TRACE_LIB	= 1U << 6,
 };
 
-static u32 avp_debug_mask = (AVP_DBG_TRACE_TRPC_CONN |
-			     AVP_DBG_TRACE_XPC_CONN |
-			     AVP_DBG_TRACE_LIB);
+static u32 avp_debug_mask = 0;
 module_param_named(debug_mask, avp_debug_mask, uint, S_IWUSR | S_IRUGO);
 
 #define DBG(flag, args...) \
@@ -1036,6 +1034,7 @@ static void avp_uninit(struct avp_info *avp)
 
 	avp->shutdown = false;
 	smp_wmb();
+	pr_info("%s: avp teardown done\n", __func__);
 }
 
 /* returns the remote lib handle in lib->handle */
@@ -1050,7 +1049,7 @@ static int _load_lib(struct avp_info *avp, struct tegra_avp_lib *lib)
 	unsigned long lib_phys;
 	int ret;
 
-	pr_info("avp_lib: loading library %s\n", lib->name);
+	DBG(AVP_DBG_TRACE_LIB, "avp_lib: loading library '%s'\n", lib->name);
 
 	args = kmalloc(lib->args_len, GFP_KERNEL);
 	if (!args) {
@@ -1122,8 +1121,9 @@ static int _load_lib(struct avp_info *avp, struct tegra_avp_lib *lib)
 	}
 	lib->handle = resp.lib_id;
 	ret = 0;
-	pr_info("avp_lib: Successfully loaded library %s (lib_id=%x)\n",
-		lib->name, resp.lib_id);
+	DBG(AVP_DBG_TRACE_LIB,
+	    "avp_lib: Successfully loaded library %s (lib_id=%x)\n",
+	    lib->name, resp.lib_id);
 
 	/* We free the memory here because by this point the AVP has already
 	 * requested memory for the library for all the sections since it does
diff --git a/drivers/media/video/tegra/avp/avp_svc.c b/drivers/media/video/tegra/avp/avp_svc.c
index 71f7b28a4828..57cd8019c305 100644
--- a/drivers/media/video/tegra/avp/avp_svc.c
+++ b/drivers/media/video/tegra/avp/avp_svc.c
@@ -318,7 +318,6 @@ static void do_svc_module_reset(struct avp_svc_info *avp_svc,
 		resp.err = 0;
 		goto send_response;
 	}
-	pr_info("avp_svc: module reset: %s\n", mod->name);
 
 	aclk = &avp_svc->clks[mod->clk_req];
 	tegra_periph_reset_assert(aclk->clk);
@@ -348,8 +347,6 @@ static void do_svc_module_clock(struct avp_svc_info *avp_svc,
 		resp.err = AVP_ERR_EINVAL;
 		goto send_response;
 	}
-	pr_info("avp_svc: module clock: %s %s\n", mod->name,
-		msg->enable ? "on" : "off");
 
 	mutex_lock(&avp_svc->clk_lock);
 	aclk = &avp_svc->clks[mod->clk_req];
diff --git a/drivers/media/video/tegra/avp/tegra_rpc.c b/drivers/media/video/tegra/avp/tegra_rpc.c
index f7fe78d5d5d2..6110d0bd066c 100644
--- a/drivers/media/video/tegra/avp/tegra_rpc.c
+++ b/drivers/media/video/tegra/avp/tegra_rpc.c
@@ -70,7 +70,7 @@ enum {
 	TRPC_TRACE_PORT	= 1U << 2,
 };
 
-static u32 trpc_debug_mask = TRPC_TRACE_CONN | TRPC_TRACE_PORT;
+static u32 trpc_debug_mask = 0;
 module_param_named(debug_mask, trpc_debug_mask, uint, S_IWUSR | S_IRUGO);
 
 #define DBG(flag, args...) \