Re: [Xen-devel] [PATCH v5 6/7] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-13 Thread Julien Grall
Hi, On 12/12/18 11:56 PM, Stefano Stabellini wrote: On Wed, 12 Dec 2018, Julien Grall wrote: On 11/12/2018 22:23, Stefano Stabellini wrote: On Tue, 11 Dec 2018, Julien Grall wrote: Furthermore, you are forwarding unsanitized values to the firmware. For instance, what would happen if the numbe

Re: [Xen-devel] [PATCH v5 6/7] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-12 Thread Stefano Stabellini
On Wed, 12 Dec 2018, Julien Grall wrote: > On 11/12/2018 22:23, Stefano Stabellini wrote: > > On Tue, 11 Dec 2018, Julien Grall wrote: > > > Hi Stefano, > > > > > > On 03/12/2018 21:03, Stefano Stabellini wrote: > > > > From: "Edgar E. Iglesias" > > > > > > > > From: Edgar E. Iglesias > > > >

Re: [Xen-devel] [PATCH v5 6/7] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-12 Thread Julien Grall
On 11/12/2018 22:23, Stefano Stabellini wrote: On Tue, 11 Dec 2018, Julien Grall wrote: Hi Stefano, On 03/12/2018 21:03, Stefano Stabellini wrote: From: "Edgar E. Iglesias" From: Edgar E. Iglesias zynqmp_eemi uses the defined functions and structs to decide whether to make a call to the

Re: [Xen-devel] [PATCH v5 6/7] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-11 Thread Stefano Stabellini
On Tue, 11 Dec 2018, Julien Grall wrote: > Hi Stefano, > > On 03/12/2018 21:03, Stefano Stabellini wrote: > > From: "Edgar E. Iglesias" > > > > From: Edgar E. Iglesias > > > > zynqmp_eemi uses the defined functions and structs to decide whether to > > make a call to the firmware, or to simply

Re: [Xen-devel] [PATCH v5 6/7] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-11 Thread Julien Grall
Hi Stefano, On 03/12/2018 21:03, Stefano Stabellini wrote: From: "Edgar E. Iglesias" From: Edgar E. Iglesias zynqmp_eemi uses the defined functions and structs to decide whether to make a call to the firmware, or to simply return a predefined value. Signed-off-by: Edgar E. Iglesias Signed-

[Xen-devel] [PATCH v5 6/7] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-03 Thread Stefano Stabellini
From: "Edgar E. Iglesias" From: Edgar E. Iglesias zynqmp_eemi uses the defined functions and structs to decide whether to make a call to the firmware, or to simply return a predefined value. Signed-off-by: Edgar E. Iglesias Signed-off-by: Stefano Stabellini --- Changes in v5: - remove mmio_a