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 7B626C43334 for ; Fri, 1 Jul 2022 14:48:40 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=V5K5j2PMI6f9Dtmr9rdqh5WbrnZYJ5SF/HT2ZGEtD6Q=; b=IVuSzFndv+GP4v EXjtkSNPjRxdLaACowQFlUjg6oNzyWboOOA4GC2fB3oX/RzqDjl8Y+6XD5SPC7K2RjTEvq20M+O6w RwjOWIOImfbNdIrMytki3HcP8RNU1uwYsXn6jI11h6QpaxqthtFPp9gxUDXDpvL/B8xPQdG4OfLVp Kh0MiG40kC9j8iXRBNjzB1AxW7QH1NyA7nuIgpkL460xr4Wg8J3RO8nhpMM7vwMPYy+Rnut6Ec5e2 tr3OgYgZlfubSrLXIcxDzkN5RPrHOladzvPKfbnltgzpBFtSippXuz8pZYOoqHXVch+Qtu4EaGjxJ zycC4+zxJJcV2KRaUkYg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o7Hvp-005QlC-W4; Fri, 01 Jul 2022 14:47:42 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o7Hve-005QfD-K8 for linux-arm-kernel@lists.infradead.org; Fri, 01 Jul 2022 14:47:32 +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 EA541143D; Fri, 1 Jul 2022 07:47:24 -0700 (PDT) Received: from e120937-lin (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D082B3F792; Fri, 1 Jul 2022 07:47:22 -0700 (PDT) Date: Fri, 1 Jul 2022 15:47:20 +0100 From: Cristian Marussi To: Sudeep Holla Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, james.quinlan@broadcom.com, Jonathan.Cameron@huawei.com, f.fainelli@gmail.com, etienne.carriere@linaro.org, vincent.guittot@linaro.org, daniel.lezcano@linaro.org, tarek.el-sherbiny@arm.com, adrian.slatineanu@arm.com, souvik.chakravarty@arm.com, wleavitt@marvell.com, wbartczak@marvell.com Subject: Re: [PATCH v3 5/9] firmware: arm_scmi: Make use of FastChannels configurable Message-ID: References: <20220627123038.1427067-1-cristian.marussi@arm.com> <20220627123038.1427067-6-cristian.marussi@arm.com> <20220701140307.upgfn4qpxhl63syg@bogus> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220701140307.upgfn4qpxhl63syg@bogus> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220701_074730_756556_5A631880 X-CRM114-Status: GOOD ( 25.19 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Jul 01, 2022 at 03:03:07PM +0100, Sudeep Holla wrote: > On Mon, Jun 27, 2022 at 01:30:34PM +0100, Cristian Marussi wrote: > > Add a Kernel configuration entry used to optionally disable, globally, the > > usage of SCMI FastChannels even on platforms where they are available. > > > > Make such option default-no to preserve the original SCMI system behaviour > > of using any available FC. > > > > Signed-off-by: Cristian Marussi > > --- > > v2 --> v3 > > - fixed wording in Kconfig > > - reverted Kconfig logic _USE_ -> _AVOID_ > > --- > > drivers/firmware/arm_scmi/Kconfig | 13 +++++++++++++ > > drivers/firmware/arm_scmi/driver.c | 6 ++++++ > > 2 files changed, 19 insertions(+) > > > > diff --git a/drivers/firmware/arm_scmi/Kconfig b/drivers/firmware/arm_scmi/Kconfig > > index 1e7b7fec97d9..3fb34db01014 100644 > > --- a/drivers/firmware/arm_scmi/Kconfig > > +++ b/drivers/firmware/arm_scmi/Kconfig > > @@ -42,6 +42,19 @@ config ARM_SCMI_HAVE_MSG > > This declares whether a message passing based transport for SCMI is > > available. > > > > +config ARM_SCMI_AVOID_FASTCHANNELS > > + bool "Avoid using SCMI FastChannels even when available" > > Without default, won't this prompt user now ? I would have explicit default n > if we are adding this. But why do we need this is my question ? This would > be a quirk IMO on systems where FC is broken. I don't want people to enable > this during testing and ship f/w with FC broken(or not developed yet). > > We can add this if some platforms really need this as a quirk in the future. > Ah ok an explicit default no is better indeed to avoid prompting if you do not defconfig (I missed this difference between default implicit NO and explicit NO) ... ... regarding why, I am using personally this indeed for testing with or without FCs without having to change the installed FW blob, BUT the reason for upstreaming such an option is that FC support is indeed optional by the spec so I thought it would have been acceptabel that you could want to configure a platform NOT to use them even though the FW implementation you are using, maybe across multiple platforms, supports it. Thanks, Cristian _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel