Re: [U-Boot] [PATCH] arm64: Add SMC and HVC commands

2018-04-09 Thread Michalis Pappas
Thanks for the feedback, I just sent a v2 addressing both issues On 04/09/2018 04:26 AM, Fabio Estevam wrote: On Fri, Apr 6, 2018 at 7:06 AM, Michalis Pappas wrote: +config CMD_HVC + bool "Support the 'hvc' command" + depends on ARM64 + default n No need to pass 'default n

Re: [U-Boot] [PATCH] arm64: Add SMC and HVC commands

2018-04-08 Thread Fabio Estevam
On Fri, Apr 6, 2018 at 7:06 AM, Michalis Pappas wrote: > +config CMD_HVC > + bool "Support the 'hvc' command" > + depends on ARM64 > + default n No need to pass 'default n' as this is already the default behavior. > + help > + Allows issuing Hypervisor Calls (HVC

Re: [U-Boot] [PATCH] arm64: Add SMC and HVC commands

2018-04-08 Thread Michalis Pappas
On 04/08/2018 04:50 PM, Simon Glass wrote: Please expand on this. Why do you want to do this, what sort of things can you do? Other than that: Reviewed-by: Simon Glass The main usecase is development / testing. My motivation was testing the Secure Payload of ARM Trusted Firmware on QEMU. F

Re: [U-Boot] [PATCH] arm64: Add SMC and HVC commands

2018-04-08 Thread Tom Rini
On Sun, Apr 08, 2018 at 07:47:20PM +0300, Michalis Pappas wrote: > > On 04/08/2018 04:50 PM, Simon Glass wrote: > >Please expand on this. Why do you want to do this, what sort of things > >can you do? > > > >Other than that: > > > >Reviewed-by: Simon Glass > The main usecase is development / test

Re: [U-Boot] [PATCH] arm64: Add SMC and HVC commands

2018-04-08 Thread Simon Glass
Hi, On 6 April 2018 at 18:06, Michalis Pappas wrote: > This patch adds smc and hvc commands, that allow issuing Secure Monitor > Calls and Hypervisor Calls conforming to the ARM SMC Calling Convention. > > Add Kconfig items to allow each command to be individually enabled. > > Signed-off-by: Mich