From patchwork Thu Oct 19 08:21:43 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Umang Jain X-Patchwork-Id: 13428444 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 94F2ECDB482 for ; Thu, 19 Oct 2023 08:22:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=HPKMJQo0EV+khCjA+cWQ5Dcd8Qrr5/U9BKu1ooqVEsQ=; b=UYg2WGaNLdxjo6 zqTEFmjBpZGNEc7k+pqZYHq8kDxRzq+BydVqG5eqSE2IdnhJnA3fQZh1LS0yTtdNkLM1TJGyK9uqI zP8kYPH5TPOFWKDNPewNo5/huOqDsTQODVwSXj2d95vw6OAwyBuXRFXrEBa9YjPKymvo5JsdnyqX6 nYAu1SsjgkEu+EBk++3vqP5V40n+dzkBVRyCUWjingomXEgtxkaBCw0QzRx/hx+ZSOnhniQi+DqNp ZZxE14unthKoyE8zX8J8gsDtdQEC5jh34kM+YRTdM7lBy1m4OEjVxSIVSeEL3KTbVIbDb442QFY6B KKug0Q+rbal8LQGykjKg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qtOID-00GhAT-38; Thu, 19 Oct 2023 08:22:09 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qtOIB-00Gh8n-1z; Thu, 19 Oct 2023 08:22:09 +0000 Received: from umang.jainideasonboard.com (unknown [103.251.226.1]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 016D625A; Thu, 19 Oct 2023 10:21:50 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1697703713; bh=LzjQ3czQq/jljPSeb2fPZh9bsWvb3v3VA3S4TgVmnlE=; h=From:To:Cc:Subject:Date:From; b=ZoPm3K6MYry9ic6T6mlw9gXX07pOXBLn9wH3bn1TwQq1gDbLisQBmx7olqh7ujyKT fpo0CVUXg7mhQJs01ysai4/oIfDt1AC32PDvjNHZhc4bB+7XyFShUyh11HnOjm60Pl 1CpF4MHL69ZCOnOTSVnOWCw77s5UYm3IRyL/YKls= From: Umang Jain To: linux-staging@lists.linux.dev, linux-rpi-kernel@lists.infradead.org, linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: Stefan Wahren , Greg Kroah-Hartman , Dave Stevenson , Dan Carpenter , Kieran Bingham , Laurent Pinchart , Umang Jain Subject: [PATCH v1 0/4] staging: vc04: Drop custom logging based on printk Date: Thu, 19 Oct 2023 13:51:43 +0530 Message-Id: <20231019082147.384723-1-umang.jain@ideasonboard.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231019_012207_804013_B205E1B4 X-CRM114-Status: UNSURE ( 7.74 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello, This series address the discussion on [1], i.e. removing the printk based logging mechanism and migrate it use dynamic debug. Patch 1/4 targets migration of vchiq_log_error(). Patch 2/4 targets migration of vchiq_log_warning(). ... and so on. This series does not touch debugfs entries which are: DEBUG_INITIALISE, DEBUG_TRACE, DEBUG_VALUE and DEBUG_COUNT. As far as I understand from the previous discussions, those too should be moved to use dynamic debug as well (but is not part of this series yet), as I found there are more moving parts involved. I will build a series on top of this one, to move the debugfs entries to dynamic debug and remove all custom logging remanants (like VCHIQ_LOG_* macros, *_log_levels externs) because they do still tie into vchiq_debugfs.c. [1] https://lore.kernel.org/linux-media/20230913185528.770634-1-umang.jain@ideasonboard.com/ Umang Jain (4): staging: vc04: Convert vchiq_log_error() to use dynamic debug staging: vc04: Convert vchiq_log_warning() to use dynamic debug staging: vc04: Convert(and rename) vchiq_log_info() to use dynamic debug staging: vc04: Convert vchiq_log_trace() to use dynamic debug .../interface/vchiq_arm/vchiq_arm.c | 143 ++++---- .../interface/vchiq_arm/vchiq_connected.c | 2 +- .../interface/vchiq_arm/vchiq_core.c | 329 +++++++++--------- .../interface/vchiq_arm/vchiq_core.h | 53 ++- .../interface/vchiq_arm/vchiq_dev.c | 59 ++-- 5 files changed, 307 insertions(+), 279 deletions(-)