From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework
Date: Tue, 27 Sep 2016 13:23:54 -0400 [thread overview]
Message-ID: <20160927172354.GG4884@bill-the-cat> (raw)
In-Reply-To: <1474968546-8285-1-git-send-email-hongbo.zhang@nxp.com>
On Tue, Sep 27, 2016 at 05:29:00PM +0800, macro.wave.z at gmail.com wrote:
> From: Hongbo Zhang <hongbo.zhang@nxp.com>
>
> 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 5/6 introduce new ARMv8 framework and set it up.
> The last 6/6 adds a most simple implementation on NXP LS1043 platform, to
> verify this framework.
>
> This patch set mainly introduces ARMv8 PSCI framework, for easier review and
> merge, further PSCI implementation on LS1043 is coming later.
>
> Hongbo Zhang (6):
> ARMv8: LS1043A: change macro CONFIG_ARMV8_PSCI definition
> ARMv8: Add secure sections for PSCI text and data
> ARMv8: Add basic PSCI framework
> ARMv8: Setup PSCI memory and dt
> ARMv8: Enable SMC instruction
> ARMv8: LS1043A: Enable LS1043A default PSCI support
Conceptually this is good. I have some issues around order of the
patches, and where the Kconfig entries end up. Looking over the series
we introduce usage of some CONFIG symbols prior to declaring them in
Kconfig. This is more of a hard no now as it will break bisecting when
the test for no new CONFIG symbols is tripped. The other problem is
that I think the symbols you're adding in
board/freescale/ls1043ardb/Kconfig need to be in
arch/arm/cpu/armv8/Kconfig and then use default ... if ... to give the
right address for the layerscape boards.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160927/5cec1182/attachment.sig>
next prev parent reply other threads:[~2016-09-27 17:23 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-27 9:29 [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework macro.wave.z at gmail.com
2016-09-27 9:29 ` [U-Boot] [PATCH 1/6] ARMv8: LS1043A: change macro CONFIG_ARMV8_PSCI definition macro.wave.z at gmail.com
2016-09-27 9:29 ` [U-Boot] [PATCH 2/6] ARMv8: Add secure sections for PSCI text and data macro.wave.z at gmail.com
2016-09-27 9:29 ` [U-Boot] [PATCH 3/6] ARMv8: Add basic PSCI framework macro.wave.z at gmail.com
2016-09-27 9:29 ` [U-Boot] [PATCH 4/6] ARMv8: Setup PSCI memory and dt macro.wave.z at gmail.com
2016-09-27 16:00 ` york sun
2016-09-28 7:41 ` Hongbo Zhang
2016-09-27 9:29 ` [U-Boot] [PATCH 5/6] ARMv8: Enable SMC instruction macro.wave.z at gmail.com
2016-09-28 7:48 ` Hongbo Zhang
2016-09-27 9:29 ` [U-Boot] [PATCH 6/6] ARMv8: LS1043A: Enable LS1043A default PSCI support macro.wave.z at gmail.com
2016-09-27 17:23 ` Tom Rini [this message]
2016-09-28 7:16 ` [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework Hongbo Zhang
2016-09-28 8:27 ` Hongbo Zhang
2016-10-18 7:18 ` Hongbo Zhang
2016-10-26 18:17 ` york sun
2016-10-27 2:45 ` Hongbo Zhang
2016-10-28 13:30 ` Tom Rini
2016-10-31 3:00 ` Hongbo Zhang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160927172354.GG4884@bill-the-cat \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox