Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-10-30 Thread Hongbo Zhang
Thanks Tom. I am sending out an updated v2 soon, with the related configs updated. On Fri, Oct 28, 2016 at 9:30 PM, Tom Rini wrote: > On Wed, Sep 28, 2016 at 03:16:38PM +0800, Hongbo Zhang wrote: >> On Wed, Sep 28, 2016 at 1:23 AM, Tom Rini wrote: >> > On

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-10-28 Thread Tom Rini
On Wed, Sep 28, 2016 at 03:16:38PM +0800, Hongbo Zhang wrote: > On Wed, Sep 28, 2016 at 1:23 AM, Tom Rini wrote: > > On Tue, Sep 27, 2016 at 05:29:00PM +0800, macro.wav...@gmail.com wrote: > >> From: Hongbo Zhang > >> > >> This patch set introduces ARMv8

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-10-26 Thread Hongbo Zhang
York, Yes I am going to send an update, one patch needs to be reordered as I've mentioned. I didn't get other feedback. But one thing I'm wondering is how to introduce the CONFIG_ options well, Tom's concerns make sense, and I had my explain too, we need a final solution before I send out a v2.

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-10-26 Thread york sun
On 10/18/2016 12:18 AM, Hongbo Zhang wrote: > Ping all, > Some time ago I saw several people mentioned ARMv8 PSCI, are you > interested in leaving your review comments? > I know it is not so easy for reviewing assembly language patches, but > code structures and assembly instructions in this patch

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-10-18 Thread Hongbo Zhang
Ping all, Some time ago I saw several people mentioned ARMv8 PSCI, are you interested in leaving your review comments? I know it is not so easy for reviewing assembly language patches, but code structures and assembly instructions in this patch set are not complicated. When this common framework

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-09-28 Thread Hongbo Zhang
I just explained why and how I tried to place newly introduced macros in arch/arm/cpu/armv8/Kconfig and board/freescale/ls1043ardb/Kconfig. As to the order, is it compulsory to define such a CONFIG_* before using it, even there is #ifdef test when using it? If yes, I have to define all the four

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-09-28 Thread Hongbo Zhang
On Wed, Sep 28, 2016 at 1:23 AM, Tom Rini wrote: > On Tue, Sep 27, 2016 at 05:29:00PM +0800, macro.wav...@gmail.com wrote: >> From: Hongbo Zhang >> >> This patch set introduces ARMv8 PSCI framework, all the PSCI functions are >> implemented a default

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-09-27 Thread Tom Rini
On Tue, Sep 27, 2016 at 05:29:00PM +0800, macro.wav...@gmail.com wrote: > From: Hongbo Zhang > > This patch set introduces ARMv8 PSCI framework, all the PSCI functions are > implemented a default dummy one, it is up to each platform to implement their > own specific ones. >

[U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-09-27 Thread macro . wave . z
From: Hongbo Zhang This patch set introduces ARMv8 PSCI framework, all the PSCI functions are implemented a default dummy one, it is up to each platform to implement their own specific ones. The first 1/6 patch is a prepare clean up for adding ARMv8 PSCI. Patches 2/6 to