From: Ye Li <ye...@nxp.com>

If a pad is not owned by current partition we should not set its
pinmux.

Signed-off-by: Ye Li <ye...@nxp.com>
Signed-off-by: Peng Fan <peng....@nxp.com>
---
 drivers/pinctrl/nxp/pinctrl-scu.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/drivers/pinctrl/nxp/pinctrl-scu.c 
b/drivers/pinctrl/nxp/pinctrl-scu.c
index aa11075e0a..c032be782a 100644
--- a/drivers/pinctrl/nxp/pinctrl-scu.c
+++ b/drivers/pinctrl/nxp/pinctrl-scu.c
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0+
 /*
- * Copyright 2018 NXP
+ * Copyright 2018-2019 NXP
  */
 
 #include <common.h>
@@ -29,6 +29,11 @@ static int imx_pinconf_scu_set(struct imx_pinctrl_soc_info 
*info, u32 pad,
         * to handle that in scfw, so config it in pad conf func
         */
 
+       if (!sc_rm_is_pad_owned(-1, pad)) {
+               debug("Pad[%u] is not owned by curr partition\n", pad);
+               return -EPERM;
+       }
+
        val |= PADRING_IFMUX_EN_MASK;
        val |= PADRING_GP_EN_MASK;
        val |= (mux << PADRING_IFMUX_SHIFT) & PADRING_IFMUX_MASK;
@@ -57,7 +62,7 @@ int imx_pinctrl_scu_conf_pins(struct imx_pinctrl_soc_info 
*info, u32 *pin_data,
                config_val = pin_data[j++];
 
                ret = imx_pinconf_scu_set(info, pin_id, mux, config_val);
-               if (ret)
+               if (ret && ret != -EPERM)
                        printf("Set pin %d, mux %d, val %d, error\n", pin_id,
                               mux, config_val);
        }
-- 
2.16.4

Reply via email to