From: Hou Zhiqiang <zhiqiang....@nxp.com>

The implementation of workaround A-010315 is wrong, it updated
other IP's CSU registers.

Signed-off-by: Hou Zhiqiang <zhiqiang....@nxp.com>
---
 board/freescale/common/ns_access.c | 20 ++++++++++----------
 include/fsl_csu.h                  |  2 +-
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/board/freescale/common/ns_access.c 
b/board/freescale/common/ns_access.c
index 1c2287d..0c3a54c 100644
--- a/board/freescale/common/ns_access.c
+++ b/board/freescale/common/ns_access.c
@@ -10,15 +10,15 @@
 #include <asm/arch/ns_access.h>
 #include <asm/arch/fsl_serdes.h>
 
-void set_devices_ns_access(struct csu_ns_dev *ns_dev, u16 val)
+void set_devices_ns_access(unsigned long index, u16 val)
 {
        u32 *base = (u32 *)CONFIG_SYS_FSL_CSU_ADDR;
        u32 *reg;
        uint32_t tmp;
 
-       reg = base + ns_dev->ind / 2;
+       reg = base + index / 2;
        tmp = in_be32(reg);
-       if (ns_dev->ind % 2 == 0) {
+       if (index % 2 == 0) {
                tmp &= 0x0000ffff;
                tmp |= val << 16;
        } else {
@@ -34,7 +34,7 @@ static void enable_devices_ns_access(struct csu_ns_dev 
*ns_dev, uint32_t num)
        int i;
 
        for (i = 0; i < num; i++)
-               set_devices_ns_access(ns_dev + i, ns_dev[i].val);
+               set_devices_ns_access(ns_dev[i].ind, ns_dev[i].val);
 }
 
 void enable_layerscape_ns_access(void)
@@ -50,20 +50,20 @@ void set_pcie_ns_access(int pcie, u16 val)
        switch (pcie) {
 #ifdef CONFIG_PCIE1
        case PCIE1:
-               set_devices_ns_access(&ns_dev[CSU_CSLX_PCIE1], val);
-               set_devices_ns_access(&ns_dev[CSU_CSLX_PCIE1_IO], val);
+               set_devices_ns_access(CSU_CSLX_PCIE1, val);
+               set_devices_ns_access(CSU_CSLX_PCIE1_IO, val);
                return;
 #endif
 #ifdef CONFIG_PCIE2
        case PCIE2:
-               set_devices_ns_access(&ns_dev[CSU_CSLX_PCIE2], val);
-               set_devices_ns_access(&ns_dev[CSU_CSLX_PCIE2_IO], val);
+               set_devices_ns_access(CSU_CSLX_PCIE2, val);
+               set_devices_ns_access(CSU_CSLX_PCIE2_IO, val);
                return;
 #endif
 #ifdef CONFIG_PCIE3
        case PCIE3:
-               set_devices_ns_access(&ns_dev[CSU_CSLX_PCIE3], val);
-               set_devices_ns_access(&ns_dev[CSU_CSLX_PCIE3_IO], val);
+               set_devices_ns_access(CSU_CSLX_PCIE3, val);
+               set_devices_ns_access(CSU_CSLX_PCIE3_IO, val);
                return;
 #endif
        default:
diff --git a/include/fsl_csu.h b/include/fsl_csu.h
index 8582ac0..027a811 100644
--- a/include/fsl_csu.h
+++ b/include/fsl_csu.h
@@ -30,7 +30,7 @@ struct csu_ns_dev {
 };
 
 void enable_layerscape_ns_access(void);
-void set_devices_ns_access(struct csu_ns_dev *ns_dev, u16 val);
+void set_devices_ns_access(unsigned long, u16 val);
 void set_pcie_ns_access(int pcie, u16 val);
 
 #endif
-- 
2.1.0.27.g96db324

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to