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 872B2C83F03 for ; Fri, 4 Jul 2025 11:53:07 +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=1+82/1t755uTfXW3ST8sS2iDG+BfPnPQBew3rfjsvk0=; b=Esa67LwAUBceXu8/tfwl7Wc/QQ hDr4WSMSflaJ/EGzjBBK260m59HA0YAqIv5xQ6Sdm73ByeNIg2HHKyGGIb2wXhDbx3QNxDyhbBgSg Sri7wmZRZh2iU8vLG5QSSz4aO6cm0lEgsIhb+coeEKTDlRR18C5wnVWEHAFGc6ZEccKk4geaz3/3r mTgyTvCudDHqKOC9fjw2NnpkGkMbLqujllYzFcKixHyoGXUbnvhMkvoArMNP2Av0OFi+DT7WKD9W9 ubtKULi7P8yN50s/GT8WLmIOZquQOJhm8aEqC6tIxN5tm51PJUaZczOwQQFm/hN5+LZpAgwqveLsV 8Np3xF9Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uXeyQ-0000000EIC9-1pQ1; Fri, 04 Jul 2025 11:52:58 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uXe16-0000000E7lO-3QCK for linux-arm-kernel@lists.infradead.org; Fri, 04 Jul 2025 10:51:41 +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 318F12661; Fri, 4 Jul 2025 03:51:26 -0700 (PDT) Received: from bogus (e133711.arm.com [10.1.196.55]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id EF38A3F6A8; Fri, 4 Jul 2025 03:51:37 -0700 (PDT) Date: Fri, 4 Jul 2025 11:51:35 +0100 From: Sudeep Holla To: Peng Fan Cc: Chuck Cannon , "Peng Fan (OSS)" , Sudeep Holla , Cristian Marussi , Shawn Guo , Sascha Hauer , "Pengutronix Kernel Team" , Fabio Estevam , "arm-scmi@vger.kernel.org" , "imx@lists.linux.dev" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 3/7] firmware: arm_scmi: imx: Support getting cfg info of MISC protocol Message-ID: References: <20250627-sm-misc-api-v1-v1-0-2b99481fe825@nxp.com> <20250627-sm-misc-api-v1-v1-3-2b99481fe825@nxp.com> <20250702-arrogant-jackdaw-of-skill-bb9fd3@sudeepholla> <20250704100741.GC4525@nxa18884-linux> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250704_035140_947034_E25FD001 X-CRM114-Status: GOOD ( 28.61 ) 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 Fri, Jul 04, 2025 at 10:39:53AM +0000, Peng Fan wrote: > > Subject: Re: [PATCH 3/7] firmware: arm_scmi: imx: Support getting cfg > > info of MISC protocol > > > > On Fri, Jul 04, 2025 at 06:07:41PM +0800, Peng Fan wrote: > > > On Wed, Jul 02, 2025 at 04:21:58PM +0100, Sudeep Holla wrote: > > > >On Fri, Jun 27, 2025 at 02:03:46PM +0800, Peng Fan wrote: > > > >> MISC protocol supports getting the System Manager(SM) mode > > > >> selection and configuration name. Add the API for user to retrieve > > > >> the information from SM. > > > >> > > > >> Signed-off-by: Peng Fan > > > >> --- > > > >> .../firmware/arm_scmi/vendors/imx/imx-sm-misc.c | 30 > > ++++++++++++++++++++++ > > > >> include/linux/scmi_imx_protocol.h | 5 ++++ > > > >> 2 files changed, 35 insertions(+) > > > >> > > > >> diff --git a/drivers/firmware/arm_scmi/vendors/imx/imx-sm-misc.c > > > >> b/drivers/firmware/arm_scmi/vendors/imx/imx-sm-misc.c > > > >> index > > > >> > > 1b24d070c6f4856b92f515fcdba5836fd6498ce6..8ce4bf92e6535af2f3 > > 0d72a34 > > > >> 717678613b35049 100644 > > > >> --- a/drivers/firmware/arm_scmi/vendors/imx/imx-sm-misc.c > > > >> +++ b/drivers/firmware/arm_scmi/vendors/imx/imx-sm-misc.c > > > >> @@ -26,6 +26,7 @@ enum scmi_imx_misc_protocol_cmd { > > > >> SCMI_IMX_MISC_CTRL_SET = 0x3, > > > >> SCMI_IMX_MISC_CTRL_GET = 0x4, > > > >> SCMI_IMX_MISC_DISCOVER_BUILDINFO = 0x6, > > > >> + SCMI_IMX_MISC_CFG_INFO = 0xC, > > > >> SCMI_IMX_MISC_CTRL_NOTIFY = 0x8, > > > >> }; > > > >> > > > >> @@ -73,6 +74,11 @@ struct scmi_imx_misc_buildinfo_out { > > > >> u8 buildtime[MISC_MAX_BUILDTIME]; }; > > > >> > > > >> +struct scmi_imx_misc_cfg_info_out { > > > >> + __le32 msel; > > > > > > > >Now, I realise, this mode select is not properly defined in the > > document. > > > >Just 32-bit word. What are those values ? Any fixed list of values > > > >with well defined modes or configurations ? If so, please add to the > > document. > > > > > > The current used value are 0,1,2. It is used to decide the logic > > > machine boot order, such as 0 means booting LM0, LM1, LM2, LM3... > > > Regarding 0,1,2 using which LM order, it could be defined by user in > > > i.MX9 System Manger cfg file. That means 1 could mean LM0, LM2, > > LM3, > > > LM1 or LM0, LM3, LM1, LM2. > > > > > > > This sounds like this is not well defined and could change on a different > > platform ? If so, how will you manage this extension across i.MX > > platforms ? > > > > Or the above order is actually fixed and will remain same across the > > platforms ? > > + Chuck who is the best person to answer this. I may not explain it clear. > Sorry I missed the mention of i.MX9 System Manger cfg file. So the exact order is specified in that file and this seems like an index to that entry. It is better to mention that in the document at the least if nothing else can be fixed and you need flexibility to keep that variable. -- Regards, Sudeep