[U-Boot] [PATCH 3/5] ls102xa: Enable snoop and DVM message requests.

[Yao Yuan]

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 at freescale.com]
> Sent: Tuesday, December 08, 2015 12:58 AM
> To: Yuan Yao-B46683 <yao.yuan at freescale.com>
> Cc: Wang Huan-B18965 <alison.wang at freescale.com>; u-boot at 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 at 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,
> > +		 CCI400_DVM_MESSAGE_REQ_EN | CCI400_SNOOP_REQ_EN);
> >  	out_le32(&cci->slave[4].snoop_ctrl,
> >  		 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