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:york...@freescale.com]
> Sent: Tuesday, December 08, 2015 12:58 AM
> To: Yuan Yao-B46683 <yao.y...@freescale.com>
> Cc: Wang Huan-B18965 <alison.w...@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.y...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to