[U-Boot] [PATCH] arm: ls102xa: Enable snooping for CCI-400 S0 slave interface

CAAM is connected to CCI-400 S0 slave interface. Disable snooping for S0 will cause CAAM self test failure. This patch is to enable snooping for S0 slave interface.
Signed-off-by: Alison Wang alison.wang@freescale.com --- board/freescale/ls1021aqds/ls1021aqds.c | 3 +-- board/freescale/ls1021atwr/ls1021atwr.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/board/freescale/ls1021aqds/ls1021aqds.c b/board/freescale/ls1021aqds/ls1021aqds.c index 4dda5e3..e5dea5d 100644 --- a/board/freescale/ls1021aqds/ls1021aqds.c +++ b/board/freescale/ls1021aqds/ls1021aqds.c @@ -202,10 +202,9 @@ int board_early_init_f(void) CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);
/* - * Set CCI-400 Slave interface S0, S1, S2 Shareable Override Register + * Set CCI-400 Slave interface S1, S2 Shareable Override Register * All transactions are treated as non-shareable */ - out_le32(&cci->slave[0].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[1].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[2].sha_ord, CCI400_SHAORD_NON_SHAREABLE);
diff --git a/board/freescale/ls1021atwr/ls1021atwr.c b/board/freescale/ls1021atwr/ls1021atwr.c index 0629d1c..2c8afc0 100644 --- a/board/freescale/ls1021atwr/ls1021atwr.c +++ b/board/freescale/ls1021atwr/ls1021atwr.c @@ -419,10 +419,9 @@ int board_init(void) struct ccsr_cci400 *cci = (struct ccsr_cci400 *)CONFIG_SYS_CCI400_ADDR;
/* - * Set CCI-400 Slave interface S0, S1, S2 Shareable Override Register + * Set CCI-400 Slave interface S1, S2 Shareable Override Register * All transactions are treated as non-shareable */ - out_le32(&cci->slave[0].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[1].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[2].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(0x1185000, 0x3);

Alison,
On 11/13/2014 06:12 AM, Alison Wang wrote:
CAAM is connected to CCI-400 S0 slave interface. Disable snooping for S0 will cause CAAM self test failure. This patch is to enable snooping for S0 slave interface.
Signed-off-by: Alison Wang alison.wang@freescale.com
board/freescale/ls1021aqds/ls1021aqds.c | 3 +-- board/freescale/ls1021atwr/ls1021atwr.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/board/freescale/ls1021aqds/ls1021aqds.c b/board/freescale/ls1021aqds/ls1021aqds.c index 4dda5e3..e5dea5d 100644 --- a/board/freescale/ls1021aqds/ls1021aqds.c +++ b/board/freescale/ls1021aqds/ls1021aqds.c @@ -202,10 +202,9 @@ int board_early_init_f(void) CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);
/*
* Set CCI-400 Slave interface S0, S1, S2 Shareable Override Register
* Set CCI-400 Slave interface S1, S2 Shareable Override Register
*/
- All transactions are treated as non-shareable
- out_le32(&cci->slave[0].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[1].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[2].sha_ord, CCI400_SHAORD_NON_SHAREABLE);
diff --git a/board/freescale/ls1021atwr/ls1021atwr.c b/board/freescale/ls1021atwr/ls1021atwr.c index 0629d1c..2c8afc0 100644 --- a/board/freescale/ls1021atwr/ls1021atwr.c +++ b/board/freescale/ls1021atwr/ls1021atwr.c @@ -419,10 +419,9 @@ int board_init(void) struct ccsr_cci400 *cci = (struct ccsr_cci400 *)CONFIG_SYS_CCI400_ADDR;
/*
* Set CCI-400 Slave interface S0, S1, S2 Shareable Override Register
* Set CCI-400 Slave interface S1, S2 Shareable Override Register
*/
- All transactions are treated as non-shareable
- out_le32(&cci->slave[0].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[1].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[2].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(0x1185000, 0x3);
Which patch does this one depend on? The code you remove doesn't exist. Please check and rebase to latest code if necessary.
York

Hi, York,
On 11/13/2014 06:12 AM, Alison Wang wrote:
CAAM is connected to CCI-400 S0 slave interface. Disable snooping for S0 will cause CAAM self test failure. This patch is to enable
snooping
for S0 slave interface.
Signed-off-by: Alison Wang alison.wang@freescale.com
board/freescale/ls1021aqds/ls1021aqds.c | 3 +-- board/freescale/ls1021atwr/ls1021atwr.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/board/freescale/ls1021aqds/ls1021aqds.c b/board/freescale/ls1021aqds/ls1021aqds.c index 4dda5e3..e5dea5d 100644 --- a/board/freescale/ls1021aqds/ls1021aqds.c +++ b/board/freescale/ls1021aqds/ls1021aqds.c @@ -202,10 +202,9 @@ int board_early_init_f(void) CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);
/*
* Set CCI-400 Slave interface S0, S1, S2 Shareable Override
Register
* Set CCI-400 Slave interface S1, S2 Shareable Override Register
*/
- All transactions are treated as non-shareable
- out_le32(&cci->slave[0].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[1].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[2].sha_ord, CCI400_SHAORD_NON_SHAREABLE);
diff --git a/board/freescale/ls1021atwr/ls1021atwr.c b/board/freescale/ls1021atwr/ls1021atwr.c index 0629d1c..2c8afc0 100644 --- a/board/freescale/ls1021atwr/ls1021atwr.c +++ b/board/freescale/ls1021atwr/ls1021atwr.c @@ -419,10 +419,9 @@ int board_init(void) struct ccsr_cci400 *cci = (struct ccsr_cci400 *)CONFIG_SYS_CCI400_ADDR;
/*
* Set CCI-400 Slave interface S0, S1, S2 Shareable Override
Register
* Set CCI-400 Slave interface S1, S2 Shareable Override Register
*/
- All transactions are treated as non-shareable
- out_le32(&cci->slave[0].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[1].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(&cci->slave[2].sha_ord, CCI400_SHAORD_NON_SHAREABLE); out_le32(0x1185000, 0x3);
Which patch does this one depend on? The code you remove doesn't exist. Please check and rebase to latest code if necessary.
[Alison Wang] This patch was based on other patch about deep sleep. I will rebase and integrate it into another patch. I will abandon this patch.
Thanks.
Best Regards, Alison Wang
participants (3)
-
Alison Wang
-
Huan Wang
-
York Sun