Re: [Xen-devel] [PATCH v7 4/6] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-19 Thread Stefano Stabellini
On Wed, 19 Dec 2018, Julien Grall wrote: > Hi Stefano, > > On 18/12/2018 22:36, Stefano Stabellini wrote: > > On Tue, 18 Dec 2018, Julien Grall wrote: > > > Hi, > > > > > > On 12/17/18 10:10 PM, Stefano Stabellini wrote: > > > > +/* These calls are safe and always allowed. */ > > > > +ca

Re: [Xen-devel] [PATCH v7 4/6] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-19 Thread Julien Grall
Hi Stefano, On 18/12/2018 22:36, Stefano Stabellini wrote: On Tue, 18 Dec 2018, Julien Grall wrote: Hi, On 12/17/18 10:10 PM, Stefano Stabellini wrote: +/* These calls are safe and always allowed. */ +case EEMI_FID(ZYNQMP_SIP_SVC_CALL_COUNT): +case EEMI_FID(ZYNQMP_SIP_SVC_UID): +

Re: [Xen-devel] [PATCH v7 4/6] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-18 Thread Stefano Stabellini
On Tue, 18 Dec 2018, Julien Grall wrote: > Hi, > > On 12/17/18 10:10 PM, Stefano Stabellini wrote: > > +/* These calls are safe and always allowed. */ > > +case EEMI_FID(ZYNQMP_SIP_SVC_CALL_COUNT): > > +case EEMI_FID(ZYNQMP_SIP_SVC_UID): > > +case EEMI_FID(ZYNQMP_SIP_SVC_VERSION):

Re: [Xen-devel] [PATCH v7 4/6] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-18 Thread Stefano Stabellini
On Tue, 18 Dec 2018, Julien Grall wrote: > Hi Stefano, > > On 12/17/18 10:10 PM, Stefano Stabellini wrote: > > +/* These calls are safe and always allowed. */ > > +case EEMI_FID(ZYNQMP_SIP_SVC_CALL_COUNT): > > +case EEMI_FID(ZYNQMP_SIP_SVC_UID): > > +case EEMI_FID(ZYNQMP_SIP_SVC_V

Re: [Xen-devel] [PATCH v7 4/6] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-18 Thread Julien Grall
Hi, On 12/17/18 10:10 PM, Stefano Stabellini wrote: +/* These calls are safe and always allowed. */ +case EEMI_FID(ZYNQMP_SIP_SVC_CALL_COUNT): +case EEMI_FID(ZYNQMP_SIP_SVC_UID): +case EEMI_FID(ZYNQMP_SIP_SVC_VERSION): +case EEMI_FID(PM_GET_TRUSTZONE_VERSION): +case EEMI

Re: [Xen-devel] [PATCH v7 4/6] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-18 Thread Julien Grall
Hi Stefano, On 12/17/18 10:10 PM, Stefano Stabellini wrote: +/* These calls are safe and always allowed. */ +case EEMI_FID(ZYNQMP_SIP_SVC_CALL_COUNT): +case EEMI_FID(ZYNQMP_SIP_SVC_UID): +case EEMI_FID(ZYNQMP_SIP_SVC_VERSION): I am a bit surprised that you implement those one

[Xen-devel] [PATCH v7 4/6] xen/arm: zynqmp: implement zynqmp_eemi

2018-12-17 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 v7: - add in-code c