
Hi York,
I'm sorry I'm late replies.
Thanks for reminding me. We have some investigates and then depend on the document we think both snoop request and DVM are supported on those port. So we think the setting should be ok in here.
Best Regards, Yuan Yao
-----Original Message----- From: York Sun [mailto:yorksun@freescale.com] Sent: Tuesday, December 08, 2015 12:58 AM To: Yuan Yao-B46683 yao.yuan@freescale.com Cc: Wang Huan-B18965 alison.wang@freescale.com; u-boot@lists.denx.de Subject: Re: [PATCH 3/5] ls102xa: Enable snoop and DVM message requests.
On 12/04/2015 10:59 PM, Yuan Yao wrote:
Signed-off-by: Yuan Yao yao.yuan@freescale.com
arch/arm/cpu/armv7/ls102xa/soc.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/arch/arm/cpu/armv7/ls102xa/soc.c b/arch/arm/cpu/armv7/ls102xa/soc.c index 6036473..97ba6d5 100644 --- a/arch/arm/cpu/armv7/ls102xa/soc.c +++ b/arch/arm/cpu/armv7/ls102xa/soc.c @@ -40,8 +40,14 @@ int arch_soc_init(void)
/* * Enable snoop requests and DVM message requests for
* Slave insterface S4 (A7 core cluster)
*/* All the slave insterfaces.
- out_le32(&cci->slave[0].snoop_ctrl,
CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);
- out_le32(&cci->slave[1].snoop_ctrl,
CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);
- out_le32(&cci->slave[2].snoop_ctrl,
out_le32(&cci->slave[4].snoop_ctrl, CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);
Yuan,
I'm not sure both snoop request and DVM are supported on every port. Do you need to check them before setting?
York