Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-10 Thread Julien Grall
On 09/06/2015 09:03, Shameerali Kolothum Thodi wrote: Hi Julien/Zoli, Hi Shameer, http://xenbits.xen.org/gitweb/?p=people/julieng/xen-unstable.git;a=shortlog;h=refs/heads/for-huawei/gicv2-on-gicv3 I had a run of this staging on our D01 board. Thank you for testing. Xen fails to boot with

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-09 Thread Shameerali Kolothum Thodi
n Grall; Ian Campbell Cc: xen-de...@lists.xenproject.org; stefano.stabell...@citrix.com; t...@xen.org; Shameerali Kolothum Thodi; Zoltan Kiss Subject: Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2 On 01/06/15 14:18, Zoltan Kiss wrote: > Sure. Btw. do you have

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-03 Thread Ian Campbell
On Mon, 2015-06-01 at 12:25 +0100, Zoltan Kiss wrote: > Reviewed-by: Zoltan Kiss > Tested-by: Shameerali Kolothum Thodi Acked + applied, thanks. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Julien Grall
On 01/06/15 14:18, Zoltan Kiss wrote: > Sure. Btw. do you have them in a public repo somewhere? It would make it > a little bit easier to apply and test. It's based on the latest staging: http://xenbits.xen.org/gitweb/?p=people/julieng/xen-unstable.git;a=shortlog;h=refs/heads/for-huawei/gicv2-on-

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Zoltan Kiss
On 01/06/15 13:11, Julien Grall wrote: On 01/06/15 12:25, Zoltan Kiss wrote: Hi, Yes, we managed to test it, and it works. Then only thing I've found is this bit: +/* Only 1020 interrupts are supported */ +gicv2_info.nr_lines = min(1020U, nr_lines); This interrupt controller only su

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Julien Grall
On 01/06/15 12:25, Zoltan Kiss wrote: > Hi, > > Yes, we managed to test it, and it works. Then only thing I've found is > this bit: > > +/* Only 1020 interrupts are supported */ > +gicv2_info.nr_lines = min(1020U, nr_lines); > > This interrupt controller only supports 511, so 1020 should

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Zoltan Kiss
Hi, Yes, we managed to test it, and it works. Then only thing I've found is this bit: +/* Only 1020 interrupts are supported */ +gicv2_info.nr_lines = min(1020U, nr_lines); This interrupt controller only supports 511, so 1020 should be replaced. We had such checking in the code in th

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Ian Campbell
On Mon, 2015-06-01 at 12:13 +0100, Julien Grall wrote: > Hi, > > On 18/05/15 14:36, Zoltan Kiss wrote: > > > > > > On 15/05/15 22:08, Julien Grall wrote: > >> Hi Zoltan, > >> > >> On 07/05/2015 13:37, Zoltan Kiss wrote: > >> > >>> On 07/05/15 10:32, Ian Campbell wrote: > On Thu, 2015-05-07

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-06-01 Thread Julien Grall
Hi, On 18/05/15 14:36, Zoltan Kiss wrote: > > > On 15/05/15 22:08, Julien Grall wrote: >> Hi Zoltan, >> >> On 07/05/2015 13:37, Zoltan Kiss wrote: >> >>> On 07/05/15 10:32, Ian Campbell wrote: On Thu, 2015-05-07 at 09:52 +0100, Zoltan Kiss wrote: > Looks good at first glance, let me try

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-05-18 Thread Zoltan Kiss
On 15/05/15 22:08, Julien Grall wrote: Hi Zoltan, On 07/05/2015 13:37, Zoltan Kiss wrote: On 07/05/15 10:32, Ian Campbell wrote: On Thu, 2015-05-07 at 09:52 +0100, Zoltan Kiss wrote: Looks good at first glance, let me try it on a board. On 06/05/15 19:52, Julien Grall wrote: [...] I'm c

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-05-15 Thread Julien Grall
Hi Zoltan, On 07/05/2015 13:37, Zoltan Kiss wrote: On 07/05/15 10:32, Ian Campbell wrote: On Thu, 2015-05-07 at 09:52 +0100, Zoltan Kiss wrote: Looks good at first glance, let me try it on a board. On 06/05/15 19:52, Julien Grall wrote: [...] I'm concerned to see a newly driver (pushed las

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-05-07 Thread Zoltan Kiss
Hi, On 07/05/15 10:32, Ian Campbell wrote: On Thu, 2015-05-07 at 09:52 +0100, Zoltan Kiss wrote: Looks good at first glance, let me try it on a board. On 06/05/15 19:52, Julien Grall wrote: [...] I'm concerned to see a newly driver (pushed last march) already orphan. Does Huawei still plan t

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-05-07 Thread Ian Campbell
On Thu, 2015-05-07 at 09:52 +0100, Zoltan Kiss wrote: > Looks good at first glance, let me try it on a board. > > On 06/05/15 19:52, Julien Grall wrote: [...] > > I'm concerned to see a newly driver (pushed last march) already orphan. > > Does Huawei still plan to maintain this driver? I share Ju

Re: [Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-05-07 Thread Zoltan Kiss
Looks good at first glance, let me try it on a board. On 06/05/15 19:52, Julien Grall wrote: The GIC hip04 driver was differring from GICv2. I suspect that some of the changes in the common GIC code make boot fail on hip04. Although, I don't have a platform to check so it has been only build tes

[Xen-devel] [PATCH] xen/arm: gic-hip04: Resync the driver with the GICv2

2015-05-06 Thread Julien Grall
The GIC hip04 driver was differring from GICv2. I suspect that some of the changes in the common GIC code make boot fail on hip04. Although, I don't have a platform to check so it has been only build tested. List of GICv2 commit ported to the HIP04: commit ce12e6dba4b2d120e35dffd95a745452224e7