Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-27 Thread Sumit Garg
On Fri, 27 Jul 2018 at 18:19, Mark Rutland wrote: > > On Thu, Jul 26, 2018 at 02:12:04PM +0530, Sumit Garg wrote: > > On Thu, 26 Jul 2018 at 13:20, Daniel Thompson > > wrote: > > > I guess it could implement a secure monitor call to provide it. In > > > fact I find it a rather pleasing approach.

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-27 Thread Mark Rutland
On Thu, Jul 26, 2018 at 02:12:04PM +0530, Sumit Garg wrote: > On Thu, 26 Jul 2018 at 13:20, Daniel Thompson > wrote: > > I guess it could implement a secure monitor call to provide it. In > > fact I find it a rather pleasing approach. However I think it still loops > > us round to pretty much the

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-27 Thread Ard Biesheuvel
On 27 July 2018 at 13:37, Sumit Garg wrote: > On Fri, 27 Jul 2018 at 16:40, Daniel Thompson > wrote: >> >> On 26/07/18 09:42, Sumit Garg wrote: >> > On Thu, 26 Jul 2018 at 13:20, Daniel Thompson >> > wrote: >> >> >> >> On Thu, Jul 26, 2018 at 09:39:37AM +0200, Ard Biesheuvel wrote: >> >>> On 26

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-27 Thread Sumit Garg
On Fri, 27 Jul 2018 at 16:40, Daniel Thompson wrote: > > On 26/07/18 09:42, Sumit Garg wrote: > > On Thu, 26 Jul 2018 at 13:20, Daniel Thompson > > wrote: > >> > >> On Thu, Jul 26, 2018 at 09:39:37AM +0200, Ard Biesheuvel wrote: > >>> On 26 July 2018 at 09:36, Daniel Thompson > >>> wrote: > >>>

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-27 Thread Daniel Thompson
On 26/07/18 09:42, Sumit Garg wrote: On Thu, 26 Jul 2018 at 13:20, Daniel Thompson wrote: On Thu, Jul 26, 2018 at 09:39:37AM +0200, Ard Biesheuvel wrote: On 26 July 2018 at 09:36, Daniel Thompson wrote: On Wed, Jul 25, 2018 at 12:04:58PM +0200, Ard Biesheuvel wrote: On 23 July 2018 at 15:1

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-26 Thread Sumit Garg
On Thu, 26 Jul 2018 at 13:20, Daniel Thompson wrote: > > On Thu, Jul 26, 2018 at 09:39:37AM +0200, Ard Biesheuvel wrote: > > On 26 July 2018 at 09:36, Daniel Thompson > > wrote: > > > On Wed, Jul 25, 2018 at 12:04:58PM +0200, Ard Biesheuvel wrote: > > >> On 23 July 2018 at 15:19, Sumit Garg wro

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-26 Thread Daniel Thompson
On Thu, Jul 26, 2018 at 09:39:37AM +0200, Ard Biesheuvel wrote: > On 26 July 2018 at 09:36, Daniel Thompson wrote: > > On Wed, Jul 25, 2018 at 12:04:58PM +0200, Ard Biesheuvel wrote: > >> On 23 July 2018 at 15:19, Sumit Garg wrote: > >> > OP-TEE is optional on Developerbox controlled via SCP firm

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-26 Thread Ard Biesheuvel
On 26 July 2018 at 09:36, Daniel Thompson wrote: > On Wed, Jul 25, 2018 at 12:04:58PM +0200, Ard Biesheuvel wrote: >> On 23 July 2018 at 15:19, Sumit Garg wrote: >> > OP-TEE is optional on Developerbox controlled via SCP firmware. To check >> > if we need to delete OP-TEE DT node, we use DRAM1 re

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-26 Thread Daniel Thompson
On Wed, Jul 25, 2018 at 12:04:58PM +0200, Ard Biesheuvel wrote: > On 23 July 2018 at 15:19, Sumit Garg wrote: > > OP-TEE is optional on Developerbox controlled via SCP firmware. To check > > if we need to delete OP-TEE DT node, we use DRAM1 region info as SCP > > firmware conditionally carves out

Re: [edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-25 Thread Ard Biesheuvel
On 23 July 2018 at 15:19, Sumit Garg wrote: > OP-TEE is optional on Developerbox controlled via SCP firmware. To check > if we need to delete OP-TEE DT node, we use DRAM1 region info as SCP > firmware conditionally carves out Secure memory from DRAM1 region. > > Cc: Ard Biesheuvel > Cc: Leif Lind

[edk2] [PATCH edk2-platforms v2 1/1] Silicon/SynQuacer: add optional OP-TEE DT node

2018-07-23 Thread Sumit Garg
OP-TEE is optional on Developerbox controlled via SCP firmware. To check if we need to delete OP-TEE DT node, we use DRAM1 region info as SCP firmware conditionally carves out Secure memory from DRAM1 region. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreemen