From mboxrd@z Thu Jan 1 00:00:00 1970 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 2EE1EC2BD09 for ; Mon, 1 Jul 2024 15:33:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+/ENE7GJfihicpgZMwWVT60Vvyfwl1vTkYitMu5qmEk=; b=JdnRIbINrpE4NhRx49o8q1muvd KszMrQ9xFzP1gheeOHJx6jD4qXK+1YFZWb0ME0p7u/b4QJjaNisa1GhnCVMy8+Ec5G4MRWUisP/ZK /E540urvvKAhnnES2D7ZBFl9Ca9KOtGqLKIwrHSP+V2Cvd/aaabHA8UvlaxyJEEmWhoyPEfUBQUZX 055MtXWKZNRL3/9LtS3hNB5apmyT4cntf6S/cZJUudkEM9pT4fFNrYSwoouBQwIFahINZptXFYG8c bRvoGB345GfSpwTggfjADc61a6OHBIWML7Xj/tlf0R3qLbYwmK8Cf1EDXEh0L5gRk77eR7AO+fIMX 1UHPjVlg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOJ1W-00000003xVh-0V5U; Mon, 01 Jul 2024 15:32:58 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOJ1H-00000003xNc-2SMD for linux-arm-kernel@lists.infradead.org; Mon, 01 Jul 2024 15:32:44 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A5AED339; Mon, 1 Jul 2024 08:33:07 -0700 (PDT) Received: from pluto (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id CC6433F73B; Mon, 1 Jul 2024 08:32:41 -0700 (PDT) Date: Mon, 1 Jul 2024 16:32:39 +0100 From: Cristian Marussi To: Luke Parkin Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, sudeep.holla@arm.com, cristian.marussi@arm.com Subject: Re: [PATCH 1/3] Add Kconfig option for scmi debug statistics Message-ID: References: <20240701142851.1448515-1-luke.parkin@arm.com> <20240701142851.1448515-2-luke.parkin@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240701142851.1448515-2-luke.parkin@arm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240701_083243_689605_A4FF812F X-CRM114-Status: GOOD ( 18.48 ) 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 On Mon, Jul 01, 2024 at 03:28:49PM +0100, Luke Parkin wrote: > Add a new config option for statistic tracking in SCMI subsystem. > > Signed-off-by: Luke Parkin > --- > drivers/firmware/arm_scmi/Kconfig | 10 ++++++++++ > 1 file changed, 10 insertions(+) > Hi, > diff --git a/drivers/firmware/arm_scmi/Kconfig b/drivers/firmware/arm_scmi/Kconfig > index aa5842be19b2..fac50fd0be72 100644 > --- a/drivers/firmware/arm_scmi/Kconfig > +++ b/drivers/firmware/arm_scmi/Kconfig > @@ -55,6 +55,16 @@ config ARM_SCMI_RAW_MODE_SUPPORT_COEX > operate normally, thing which could make an SCMI test suite using the > SCMI Raw mode support unreliable. If unsure, say N. > > +config ARM_SCMI_DEBUG_STATISTICS > + bool "Enable SCMI Raw mode statistic tracking" I would add also a depends on DEBUG_FS like in RAW...and maybe in the future move such all of such depends on ARM_SCMI_NEED_DEBUGFS instead...but for now it is easier and less potentially disruptive to the build to just add a depends here. > + select ARM_SCMI_NEED_DEBUGFS > + help > + Enables statistic tracking for the SCMI subsystem. > + > + Enable this option to create a new debugfs directory which contains > + several useful statistics on various SCMI features. This can be useful > + for debugging and SCMI monitoring. If unsure, say N. > + Moreover you should move this Kconfig patch after the patches in the series that includes the logic underneath this...here really you are defining some option that, if enabled, at this point it really still does NOT enable anything. Thanks, Cristian