From patchwork Mon Apr 7 08:51:25 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Johan Hovold X-Patchwork-Id: 880352 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4EE742288D6; Mon, 7 Apr 2025 08:51:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744015905; cv=none; b=cpZD0fmyB8nw70nMpiOnoxMqdmSgVB++wjEjDK3gP8/nh8BQi1HfRV5y5sFpBJIC8K+4D4zKhRcBlkvTSQA9J3GpgctHJCn0KUUirFWyrGeRhKeYQJ4Ghjs3RiGPfAg20I7/FIRMVqt4izJuv6YK+QsucybG8j4MAtaXyFYI5qg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744015905; c=relaxed/simple; bh=/cZ4bLwYN3F5IlIV369ScToRFFo5Y2ZC1Fw5hv9absI=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=d4no9Oot/sNaUe44auBC76GivwNcQOGCZ4qTk/B5O8ZHhtTIKyJiideSM2DY9voBdqaTby69hjLsdNmc7y9AYA3x/6QjxJgfW1WxA5BmhbRf+KWJuDjk3TD2CGdm+eN/HWMXI9abKFZglmiBG1s9STxTaZu6GNwm61ibsD3SIZg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=TqCUj+ND; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="TqCUj+ND" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D4F07C4CEDD; Mon, 7 Apr 2025 08:51:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1744015904; bh=/cZ4bLwYN3F5IlIV369ScToRFFo5Y2ZC1Fw5hv9absI=; h=From:To:Cc:Subject:Date:From; b=TqCUj+NDK/sbQgrGvHdijroyFHd81/aqWTTu5eeUBozVH4GcbdydFZQJuS5Ia0DEw uPONKsVeV68lNlbzcmJCNkPTgnjqVfIpSqrQxNgrGqAE0FRlFGaK/UUC5FkQ56bOjF 0TB64KilrZMePFrtgHf0l5xFZaYxPUHXxpcuq56/fRH9/3/mHJawMp0zx/Q6ahD7pk A+kXwAcLFUGsbapMDA5etFa7tS9FRsZiUa5cPwbXXnMsk+sOe5yCGo+7Q6UGLx640C DulE/aHTL6f8pivUIKxhX4Ds5hSXNx6k44KfIlDRkywB3UaO/VtdrYLNyCKEYhCPAU akFF+SoEduXvg== Received: from johan by xi.lan with local (Exim 4.97.1) (envelope-from ) id 1u1iCs-000000005YX-2yRE; Mon, 07 Apr 2025 10:51:50 +0200 From: Johan Hovold To: Robert Foss , Todor Tomov , Bryan ODonoghue , Mauro Carvalho Chehab Cc: Hans Verkuil , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Johan Hovold , Depeng Shao Subject: [PATCH] media: qcom: camss: csid: suppress CSID log spam Date: Mon, 7 Apr 2025 10:51:25 +0200 Message-ID: <20250407085125.21325-1-johan+linaro@kernel.org> X-Mailer: git-send-email 2.49.0 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 A recent commit refactored the printing of the CSID hardware version, but (without it being mentioned) also changed the log level from debug to info. This results in repeated log spam during use, for example, on the Lenovo ThinkPad X13s: qcom-camss ac5a000.camss: CSID:0 HW Version = 1.0.0 qcom-camss ac5a000.camss: CSID:0 HW Version = 1.0.0 qcom-camss ac5a000.camss: CSID:0 HW Version = 1.0.0 qcom-camss ac5a000.camss: CSID:0 HW Version = 1.0.0 qcom-camss ac5a000.camss: CSID:0 HW Version = 1.0.0 Suppress the version logging by demoting to debug level again. Fixes: f759b8fd3086 ("media: qcom: camss: csid: Move common code into csid core") Cc: Depeng Shao Signed-off-by: Johan Hovold --- drivers/media/platform/qcom/camss/camss-csid.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/qcom/camss/camss-csid.c b/drivers/media/platform/qcom/camss/camss-csid.c index d08117f46f3b..5284b5857368 100644 --- a/drivers/media/platform/qcom/camss/camss-csid.c +++ b/drivers/media/platform/qcom/camss/camss-csid.c @@ -613,8 +613,8 @@ u32 csid_hw_version(struct csid_device *csid) hw_gen = (hw_version >> HW_VERSION_GENERATION) & 0xF; hw_rev = (hw_version >> HW_VERSION_REVISION) & 0xFFF; hw_step = (hw_version >> HW_VERSION_STEPPING) & 0xFFFF; - dev_info(csid->camss->dev, "CSID:%d HW Version = %u.%u.%u\n", - csid->id, hw_gen, hw_rev, hw_step); + dev_dbg(csid->camss->dev, "CSID:%d HW Version = %u.%u.%u\n", + csid->id, hw_gen, hw_rev, hw_step); return hw_version; }