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 6affcd341b059a5770396ca360d3d66785803040..ced838ac6e2b66238170f397e44d1999e7adac0c 100644 (file)
--- 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 71f7b28a4828d0cfca2d8da94de93a6aa87e6a11..57cd8019c3051d91953cca2d74f9d269b507c1c8 100644 (file)
--- 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 f7fe78d5d5d206e31363674182bc47234bbb45b3..6110d0bd066c6dae8a445dfb6fa0841f106651ff 100644 (file)
--- 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...) \