From: Mathieu Poirier <mathieu.poirier@linaro.org>
To: "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
Cc: Bjorn Andersson <andersson@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Shawn Guo <shawnguo@kernel.org>,
Sascha Hauer <s.hauer@pengutronix.de>,
Pengutronix Kernel Team <kernel@pengutronix.de>,
Fabio Estevam <festevam@gmail.com>,
Daniel Baluta <daniel.baluta@nxp.com>,
Frank Li <Frank.Li@nxp.com>,
linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org,
imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Peng Fan <peng.fan@nxp.com>,
Krzysztof Kozlowski <krzk@kernel.org>
Subject: Re: [PATCH v6 0/6] remoteproc: imx_rproc: Support i.MX95
Date: Tue, 13 Jan 2026 10:07:15 -0700 [thread overview]
Message-ID: <aWZ7w5g7mYMDoffv@p14s> (raw)
In-Reply-To: <20260109-imx95-rproc-2026-1-8-v6-0-d2fefb36263d@nxp.com>
On Fri, Jan 09, 2026 at 08:08:00PM +0800, Peng Fan (OSS) wrote:
> This patchset is pick up a previous patchset [1] with rebased on
> next-20251030, and some changes applied.
> - Add runtime ops to separate cpu ops and lmm ops
> - added more comments
> - moved some check imx_sm_rproc_detect_mode() from imx_rproc_sm_prepare().
>
> No changes to the dt-binding patch, so R-b/A-b are kept.
> More info could be found in commit message of each patch and below.
>
> [1]https://lore.kernel.org/linux-remoteproc/20250821-imx95-rproc-1-v5-0-e93191dfac51@nxp.com/
>
> i.MX95 features a Cortex-M33 core, six Cortex-A55 cores, and
> one Cortex-M7 core. The System Control Management Interface(SCMI)
> firmware runs on the M33 core. The i.MX95 SCMI firmware named System
> Manager(SM) includes vendor extension protocols, Logical Machine
> Management(LMM) protocol and CPU protocol and etc.
>
> There are three cases for M7:
> (1) M7 in a separate Logical Machine(LM) that Linux couldn't control it.
> (2) M7 in a separate Logical Machine that Linux could control it using
> LMM protocol
> (3) M7 runs in same Logical Machine as A55, so Linux could control it
> using CPU protocol
>
> In patch 3, Use LMM and CPU protocol to manage M7. More info could be
> found in the patch commit log
> Current setup relies on pre-Linux software(U-Boot) to do
> M7 TCM ECC initialization. In future, we could add the support in Linux
>
> Patchset was tested with below boot images when the patchset based on next-20251030:
> imx-boot-variant-rpmsg-imx95-19x19-lpddr5-evk-sd.bin-flash_lpboot_sm_a55 (Use LMM protocol)
> imx-boot-variant-alt-imx95-19x19-lpddr5-evk-sd.bin-flash_alt (Use CPU protocol)
> imx-boot-imx95-19x19-lpddr5-evk-sd.bin-flash_all (M7 not under A55 control)
>
> Also tested i.MX8MP/8ULP-EVK.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> Changes in v6:
> Added R-b from Frank for patch 4
> Following Mathieu's suggestions for patch 4
> - Renamed IMX_RPROC_FLAGS_SM_LMM_AVAIL to IMX_RPROC_FLAGS_SM_LMM_CTRL
> - Added comment in imx_rproc_sm_lmm_start and add comment for
> imx_rproc_sm_lmm_check, and update comment to use "M7"
> - Following the suggested code snippets, update imx_rproc_sm_lmm_prepare
> and imx_rproc_sm_lmm_check
> - With above done, separate the CPU API support into a separate patch,
> no other changes.
>
> Changes in v5:
> - Add R-b from Daniel for patch 4
> - Patch 4, per Mathieu's comments:
> Move imx_rproc_ops_sm_lmm and imx_rproc_ops_sm_cpu to bottom of the file.
> Set IMX_RPROC_FLAGS_SM_LMM_AVAIL when scmi_imx_lmm_operation(shutdown)
> pass or m7 is started before Linux.
> - Link to v4: https://lore.kernel.org/r/20251216-imx95-rproc-2025-12-15-v4-0-bf1064ea8daf@nxp.com
>
> Changes in V4:
> - Rebased to 6.19-rc1 to resolve conflicts.
> - Address Frank's comment patch 4 regarding "if (ret != 0)" -> "if (ret)"
> - Link to v3: https://lore.kernel.org/linux-remoteproc/20251122-imx95-rproc-2025-11-20-v3-0-9ae6ee619a78@nxp.com/#t
>
> Changes in v3:
> - Add R-b for patch 2,3,5
> - Patch 4:
> Rename imx_rproc_ops_sm to imx_rproc_ops_sm_lmm.
> Add more comments in code
> Simplify if check in imx_rproc_sm_lmm_prepare()
> Update commit log
> Use dev_err for failure path
> - Link to v2: https://lore.kernel.org/linux-remoteproc/20251031-imx95-rproc-2025-10-31-v2-0-758b2e547a55@nxp.com/T/#t
>
> Changes in v2:
> - Patch 2: Change to const pointer for ops, update commit log and add R-b from Frank
> - Patch 3: New patch, introduce prepare ops
> - Patch 4: Rename imx_rproc_sm_prepare to imx_rproc_sm_lmm_prepare
> drop IMX_RPROC_FLAGS_SM_CPU_OP and IMX_RPROC_FLAGS_SM_LMM_OP
> Add a bool for dcfg->lmid and info->lmid check.
> Provide separate plat ops for lmm and cpu setup.
> Move LM permission check to a separate function.
> - Patch 5: Drop macro IMX95_M7_CPUID and IMX95_M7_LMID
> - Link to v1: https://lore.kernel.org/r/20251028-imx95-rproc-2025-10-28-v1-0-ce9e7db9edcb@nxp.com
>
> ---
> Peng Fan (6):
> dt-bindings: remoteproc: fsl,imx-rproc: Add support for i.MX95
> remoteproc: imx_rproc: Add runtime ops copy to support dynamic behavior
> remoteproc: imx_rproc: Introduce prepare ops for imx_rproc_dcfg
> remoteproc: imx_rproc: Add support for System Manager LMM API
> remoteproc: imx_rproc: Add support for System Manager CPU API
> remoteproc: imx_rproc: Add support for i.MX95
>
> .../bindings/remoteproc/fsl,imx-rproc.yaml | 1 +
> drivers/remoteproc/Kconfig | 2 +
> drivers/remoteproc/imx_rproc.c | 249 +++++++++++++++++++--
> drivers/remoteproc/imx_rproc.h | 4 +
> 4 files changed, 243 insertions(+), 13 deletions(-)
I have applied this set.
Thanks,
Mathieu
> ---
> base-commit: fc4e91c639c0af93d63c3d5bc0ee45515dd7504a
> change-id: 20260108-imx95-rproc-2026-1-8-04f759890742
>
> Best regards,
> --
> Peng Fan <peng.fan@nxp.com>
>
prev parent reply other threads:[~2026-01-13 17:07 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-09 12:08 [PATCH v6 0/6] remoteproc: imx_rproc: Support i.MX95 Peng Fan (OSS)
2026-01-09 12:08 ` [PATCH v6 1/6] dt-bindings: remoteproc: fsl,imx-rproc: Add support for i.MX95 Peng Fan (OSS)
2026-01-09 12:08 ` [PATCH v6 2/6] remoteproc: imx_rproc: Add runtime ops copy to support dynamic behavior Peng Fan (OSS)
2026-01-09 12:08 ` [PATCH v6 3/6] remoteproc: imx_rproc: Introduce prepare ops for imx_rproc_dcfg Peng Fan (OSS)
2026-01-09 12:08 ` [PATCH v6 4/6] remoteproc: imx_rproc: Add support for System Manager LMM API Peng Fan (OSS)
2026-01-13 10:35 ` Daniel Baluta
2026-01-09 12:08 ` [PATCH v6 5/6] remoteproc: imx_rproc: Add support for System Manager CPU API Peng Fan (OSS)
2026-01-13 10:29 ` Daniel Baluta
2026-01-09 12:08 ` [PATCH v6 6/6] remoteproc: imx_rproc: Add support for i.MX95 Peng Fan (OSS)
2026-01-13 17:07 ` Mathieu Poirier [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aWZ7w5g7mYMDoffv@p14s \
--to=mathieu.poirier@linaro.org \
--cc=Frank.Li@nxp.com \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=daniel.baluta@nxp.com \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-remoteproc@vger.kernel.org \
--cc=peng.fan@nxp.com \
--cc=peng.fan@oss.nxp.com \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.