remoteproc: imx_rproc: Add runtime ops copy to support dynamic behavior

Structure imx_rproc_dcfg contains a const pointer to imx_rproc_plat_ops,
which defines the start/stop/detect_mode operations for a remote processor.
To preserve the const correctness of the static configuration while
allowing runtime modification of ops behavior, introduce a new
imx_rproc_plat_ops member in struct imx_rproc named `ops`.

During initialization, dcfg->ops is assigned to priv->ops.
Enable the driver to safely override ops at runtime without affecting the
original const configuration.

Improve flexibility for platforms that require dynamic operation switching
(e.g. i.MX95 Logical Machine ops and CPU ops).

Reviewed-by: Frank Li <Frank.Li@nxp.com>
Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Link: https://lore.kernel.org/r/20260109-imx95-rproc-2026-1-8-v6-2-d2fefb36263d@nxp.com
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>

diff --git a/drivers/remoteproc/imx_rproc.c b/drivers/remoteproc/imx_rproc.c
index 33f21ab24c921135319728ead3d24f1833ea1d64..78659c8fb84d95f3d258c21af76cefea1c82850e 100644 (file)
--- a/drivers/remoteproc/imx_rproc.c
+++ b/drivers/remoteproc/imx_rproc.c
@@ -116,6 +116,7 @@ struct imx_rproc {
        u32                             entry;          /* cpu start address */
        u32                             core_index;
        struct dev_pm_domain_list       *pd_list;
+       const struct imx_rproc_plat_ops *ops;
 };
 
 static const struct imx_rproc_att imx_rproc_att_imx93[] = {
@@ -315,7 +316,6 @@ static int imx_rproc_scu_api_start(struct rproc *rproc)
 static int imx_rproc_start(struct rproc *rproc)
 {
        struct imx_rproc *priv = rproc->priv;
-       const struct imx_rproc_dcfg *dcfg = priv->dcfg;
        struct device *dev = priv->dev;
        int ret;
 
@@ -323,10 +323,10 @@ static int imx_rproc_start(struct rproc *rproc)
        if (ret)
                return ret;
 
-       if (!dcfg->ops || !dcfg->ops->start)
+       if (!priv->ops || !priv->ops->start)
                return -EOPNOTSUPP;
 
-       ret = dcfg->ops->start(rproc);
+       ret = priv->ops->start(rproc);
        if (ret)
                dev_err(dev, "Failed to enable remote core!\n");
 
@@ -372,14 +372,13 @@ static int imx_rproc_scu_api_stop(struct rproc *rproc)
 static int imx_rproc_stop(struct rproc *rproc)
 {
        struct imx_rproc *priv = rproc->priv;
-       const struct imx_rproc_dcfg *dcfg = priv->dcfg;
        struct device *dev = priv->dev;
        int ret;
 
-       if (!dcfg->ops || !dcfg->ops->stop)
+       if (!priv->ops || !priv->ops->stop)
                return -EOPNOTSUPP;
 
-       ret = dcfg->ops->stop(rproc);
+       ret = priv->ops->stop(rproc);
        if (ret)
                dev_err(dev, "Failed to stop remote core\n");
        else
@@ -584,12 +583,11 @@ static int imx_rproc_scu_api_detach(struct rproc *rproc)
 static int imx_rproc_detach(struct rproc *rproc)
 {
        struct imx_rproc *priv = rproc->priv;
-       const struct imx_rproc_dcfg *dcfg = priv->dcfg;
 
-       if (!dcfg->ops || !dcfg->ops->detach)
+       if (!priv->ops || !priv->ops->detach)
                return -EOPNOTSUPP;
 
-       return dcfg->ops->detach(rproc);
+       return priv->ops->detach(rproc);
 }
 
 static struct resource_table *imx_rproc_get_loaded_rsc_table(struct rproc *rproc, size_t *table_sz)
@@ -979,18 +977,16 @@ static int imx_rproc_scu_api_detect_mode(struct rproc *rproc)
 
 static int imx_rproc_detect_mode(struct imx_rproc *priv)
 {
-       const struct imx_rproc_dcfg *dcfg = priv->dcfg;
-
        /*
         * To i.MX{7,8} ULP, Linux is under control of RTOS, no need
-        * dcfg->ops or dcfg->ops->detect_mode, it is state RPROC_DETACHED.
+        * priv->ops or priv->ops->detect_mode, it is state RPROC_DETACHED.
         */
-       if (!dcfg->ops || !dcfg->ops->detect_mode) {
+       if (!priv->ops || !priv->ops->detect_mode) {
                priv->rproc->state = RPROC_DETACHED;
                return 0;
        }
 
-       return dcfg->ops->detect_mode(priv->rproc);
+       return priv->ops->detect_mode(priv->rproc);
 }
 
 static int imx_rproc_sys_off_handler(struct sys_off_data *data)
@@ -1040,6 +1036,9 @@ static int imx_rproc_probe(struct platform_device *pdev)
        priv->dcfg = dcfg;
        priv->dev = dev;
 
+       if (dcfg->ops)
+               priv->ops = dcfg->ops;
+
        dev_set_drvdata(dev, rproc);
        priv->workqueue = create_workqueue(dev_name(dev));
        if (!priv->workqueue) {