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 X-Spam-Level: X-Spam-Status: No, score=-5.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 41B5DC433DB for ; Tue, 9 Mar 2021 08:06:37 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B14CF652A4 for ; Tue, 9 Mar 2021 08:06:36 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B14CF652A4 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; 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=4Wuk7foS8+9FnUoJfq8cBc56PiaNDf9gZPiR7ku3pAU=; b=FZJO1RGvBCPYZ/0B1NUj/7HdQ enSDlDs51dvVzTIf2OQHFmFiFGqGnXgW+M8wRNTyj4e0rYCFhWVHoHINFBbWqHj5q5WhbWamy4Ug4 nv9tFI73aNi+PChAXP3uMQPp32XFx91kdb6J6ijAFVjW1+cnrmcPoEhF4C+Ybw0q3NHy/Uyd/7X4O E0TktEj8VlR3bhdTV7tDg0cy5iZBZ74i59ERhxlvl+HGPwc2ez4zCVvMt0JTxnlInwaFGDTpDmipY igPv2JvYvsjjbCkKT46F21fDl0HslHfTOsyxLO8ayt2TODOqhllx8KLn7O6UnJ+plxWkW2Y+z3bJb UnAtZdnAg==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lJXMc-0040Jd-NH; Tue, 09 Mar 2021 08:05:11 +0000 Received: from foss.arm.com ([217.140.110.172]) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lJXMV-0040J2-W3 for linux-arm-kernel@lists.infradead.org; Tue, 09 Mar 2021 08:05:08 +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 1AADBD6E; Tue, 9 Mar 2021 00:05:01 -0800 (PST) Received: from e120937-lin (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D9D803F71B; Tue, 9 Mar 2021 00:04:58 -0800 (PST) Date: Tue, 9 Mar 2021 08:04:56 +0000 From: Cristian Marussi To: Sudeep Holla Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, lukasz.luba@arm.com, james.quinlan@broadcom.com, Jonathan.Cameron@Huawei.com, f.fainelli@gmail.com, etienne.carriere@linaro.org, thara.gopinath@linaro.org, vincent.guittot@linaro.org, souvik.chakravarty@arm.com, cristian.marussi@arm.com Subject: Re: [PATCH v6 36/37] firmware: arm_scmi: add protocol modularization support Message-ID: <20210309080456.GB47884@e120937-lin> References: <20210202221555.41167-1-cristian.marussi@arm.com> <20210202221555.41167-37-cristian.marussi@arm.com> <20210308073425.g7zkpoamxu2m67tj@bogus> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210308073425.g7zkpoamxu2m67tj@bogus> User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210309_080505_210752_F24DBCE2 X-CRM114-Status: GOOD ( 19.95 ) 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 Hi Sudeep, On Mon, Mar 08, 2021 at 07:34:25AM +0000, Sudeep Holla wrote: > On Tue, Feb 02, 2021 at 10:15:54PM +0000, Cristian Marussi wrote: > > Extend SCMI protocols accounting mechanism to address possible module > > usage and add the support to possibly define new protocols as loadable > > modules. > > > > Keep Standard protocols built into the SCMI core. > > > > The changes look good, however without any users I am bit hesitant to add > this yet. However if you think it is hard to maintain it outside the tree > until first user gets merged, we can merge provided we test this every > release. Let me know your thoughts. > On one side this series aimed not only at simplifying the API exposed to the drivers/protocols but also to unify such API betweeen custom and standard protocols and drivers, so that whatever custom proto and driver you wrote, you are anyway going to exercise the same API and triggering the same internals as the standard protos, so to avoid the issue of having to expose/support an API not used by anyone (still).... ...on the other side, indeed, you are right that this specific thin patch WON'T be really tested/exercised indeed until a custon protocol module is upstreamed...if ever. For me it's fine to test it every cycle with my custom dummy protocol, and maybe it could be an incentive for partners to post finally upstream their protos if all the support is already there in the mainline (I know, I'm a dreamer :D), but better would be as a future alternative (not for this series though) to enable (in some well guarded/controlled form) the optional modularization of core standard protocols too (also to possibly comply with ACK extreme modularization policies...), so that this patch would be constantly exercised by our std protocol too. But I have not really a strong opinion on this, if you want to keep this out and merge only once a real custom proto appears or once we modularize the core standard protocols (in a controlled manner as said), it's fine for me too. Thanks Cristian > Also any comment from users requesting this would be useful. > > -- > Regards, > Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel