Re: [Xen-devel] [PATCH] xen/arm: Correctly read the GICv3 Re-Distributor stride

2015-09-30 Thread Ian Campbell
On Wed, 2015-09-30 at 11:29 +, Shameerali Kolothum Thodi wrote: > > > -Original Message- > > From: Ian Campbell [mailto:ian.campb...@citrix.com] > > Sent: 30 September 2015 12:04 > > To: Shameerali Kolothum Thodi; xen-de...@lists.xenproject.org > > Cc: julien.gr...@citrix.com; vijaya.k

Re: [Xen-devel] [PATCH] xen/arm: Correctly read the GICv3 Re-Distributor stride

2015-09-30 Thread Shameerali Kolothum Thodi
> -Original Message- > From: Ian Campbell [mailto:ian.campb...@citrix.com] > Sent: 30 September 2015 12:04 > To: Shameerali Kolothum Thodi; xen-de...@lists.xenproject.org > Cc: julien.gr...@citrix.com; vijaya.ku...@caviumnetworks.com > Subject: Re: [PATCH] xen/arm: Correctly read the GICv

Re: [Xen-devel] [PATCH] xen/arm: Correctly read the GICv3 Re-Distributor stride

2015-09-30 Thread Julien Grall
On 30/09/15 12:04, Ian Campbell wrote: > On Wed, 2015-09-30 at 11:54 +0100, Shameerali Kolothum Thodi wrote: >> The GICv3 driver read a 32 bit value for the re-distributor stride, but >> the dts binding is a two-cell property. > > The binding doc I have says: > > - redistributor-stride : If using

Re: [Xen-devel] [PATCH] xen/arm: Correctly read the GICv3 Re-Distributor stride

2015-09-30 Thread Julien Grall
Hi Shameerali, On 30/09/15 11:54, Shameerali Kolothum Thodi wrote: > The GICv3 driver read a 32 bit value for the re-distributor stride, but s/read/reads/ > the dts binding is a two-cell property. I would mention that two-cell = 64 bit to make clear that the issue is we are reading only one-ce

Re: [Xen-devel] [PATCH] xen/arm: Correctly read the GICv3 Re-Distributor stride

2015-09-30 Thread Ian Campbell
On Wed, 2015-09-30 at 11:54 +0100, Shameerali Kolothum Thodi wrote: > The GICv3 driver read a 32 bit value for the re-distributor stride, but > the dts binding is a two-cell property. The binding doc I have says: - redistributor-stride : If using padding pages, specifies the stride of consecuti

[Xen-devel] [PATCH] xen/arm: Correctly read the GICv3 Re-Distributor stride

2015-09-30 Thread Shameerali Kolothum Thodi
The GICv3 driver read a 32 bit value for the re-distributor stride, but the dts binding is a two-cell property. All instances of rdist stride access and reference has been modified to accommodate 64 bit value. The changes are compiled and verified on HiSilicon Hip05 platform. Signed-off-by: Shame