[U-Boot] [PATCH v5 00/14] Add PSCI support for Jetson TK1/Tegra124 + CNTFRQ fix

Tom Rini
2015-04-08T16:22:01Z

On Wed, Apr 08, 2015 at 06:13:23PM +0200, Jan Kiszka wrote:
> On 2015-04-08 17:54, Tom Rini wrote:
> > On Wed, Apr 08, 2015 at 04:12:21PM +0200, Jan Kiszka wrote:
> >> On 2015-04-08 16:02, Tom Rini wrote:
> >>> On Wed, Apr 08, 2015 at 03:55:44PM +0200, Jan Kiszka wrote:
> >>>> On 2015-04-08 15:43, Tom Rini wrote:
> >>>>> On Wed, Apr 08, 2015 at 10:37:40AM +0200, Jan Kiszka wrote:
> >>>>>> On 2015-03-19 16:02, Thierry Reding wrote:
> >>>>>>> On Mon, Mar 09, 2015 at 08:00:10AM +0100, Jan Kiszka wrote:
> >>>>>>>> Changes in v4:
> >>>>>>>> - rebased over master
> >>>>>>>> - implemented psci_get_cpu_id as weak function
> >>>>>>>> - implemented psci_disable/enable_smp as weak functions
> >>>>>>>> - adjusted register interface of psci_get_cpu_stack_top
> >>>>>>>>
> >>>>>>>> This version (+ the non-cached memory init fix) can also be found at
> >>>>>>>> https://github.com/siemens/u-boot/tree/jetson-tk1-v5.
> >>>>>>>>
> >>>>>>>> Jan
> >>>>>>>>
> >>>>>>>> CC: Ian Campbell <ijc@hellion.org.uk>
> >>>>>>>> CC: Marc Zyngier <marc.zyngier@arm.com>
> >>>>>>>>
> >>>>>>>> Ian Campbell (3):
> >>>>>>>> tegra124: Add more registers to struct mc_ctlr
> >>>>>>>> jetson-tk1: Add PSCI configuration options and reserve secure code
> >>>>>>>> tegra124: Reserve secure RAM using MC_SECURITY_CFG{0, 1}_0
> >>>>>>>>
> >>>>>>>> Jan Kiszka (11):
> >>>>>>>> sun7i: Remove duplicate call to psci_arch_init
> >>>>>>>> ARM: Factor out common psci_get_cpu_id
> >>>>>>>> ARM: Factor out reusable psci_cpu_off_common
> >>>>>>>> ARM: Factor out reusable psci_cpu_entry
> >>>>>>>> ARM: Factor out reusable psci_get_cpu_stack_top
> >>>>>>>> ARM: Put target PC for PSCI CPU_ON on per-CPU stack
> >>>>>>>> virt-dt: Allow reservation of secure region when in a RAM carveout
> >>>>>>>> tegra: Make tegra_powergate_power_on public
> >>>>>>>> tegra: Add ap_pm_init hook
> >>>>>>>> tegra124: Add PSCI support for Tegra124
> >>>>>>>> tegra: Set CNTFRQ for secondary CPUs
> >>>>>>>>
> >>>>>>>> arch/arm/cpu/armv7/psci.S | 121 ++++++++++++++++++++++++++++
> >>>>>>>> arch/arm/cpu/armv7/sunxi/psci.S | 112 ++++---------------------
> >>>>>>>> arch/arm/cpu/armv7/virt-dt.c | 29 +++++++
> >>>>>>>> arch/arm/cpu/armv7/virt-v7.c | 5 ++
> >>>>>>>> arch/arm/include/asm/arch-tegra/ap.h | 5 ++
> >>>>>>>> arch/arm/include/asm/arch-tegra/powergate.h | 1 +
> >>>>>>>> arch/arm/include/asm/arch-tegra124/flow.h | 6 ++
> >>>>>>>> arch/arm/include/asm/arch-tegra124/mc.h | 35 +++++++-
> >>>>>>>> arch/arm/include/asm/armv7.h | 1 +
> >>>>>>>> arch/arm/include/asm/system.h | 1 +
> >>>>>>>> arch/arm/lib/bootm-fdt.c | 5 ++
> >>>>>>>> arch/arm/mach-tegra/Makefile | 4 +
> >>>>>>>> arch/arm/mach-tegra/ap.c | 15 ++++
> >>>>>>>> arch/arm/mach-tegra/powergate.c | 2 +-
> >>>>>>>> arch/arm/mach-tegra/psci.S | 114 ++++++++++++++++++++++++++
> >>>>>>>> arch/arm/mach-tegra/tegra124/Kconfig | 2 +
> >>>>>>>> arch/arm/mach-tegra/tegra124/Makefile | 4 +
> >>>>>>>> arch/arm/mach-tegra/tegra124/ap.c | 55 +++++++++++++
> >>>>>>>> board/nvidia/common/board.c | 4 +
> >>>>>>>> include/configs/jetson-tk1.h | 5 ++
> >>>>>>>> 20 files changed, 428 insertions(+), 98 deletions(-)
> >>>>>>>> create mode 100644 arch/arm/mach-tegra/psci.S
> >>>>>>>> create mode 100644 arch/arm/mach-tegra/tegra124/ap.c
> >>>>>>>
> >>>>>>> For the series:
> >>>>>>>
> >>>>>>> Reviewed-by: Thierry Reding <treding@nvidia.com>
> >>>>>>> Tested-by: Thierry Reding <treding@nvidia.com>
> >>>>>>
> >>>>>> Thanks for all feedback - what's next? Anything missing? Should I rebase
> >>>>>> (didn't check yet if that is necessary) and resend?
> >>>>>
> >>>>> Please rebase (it doesn't apply cleanly) and throw Reviewed/Tested-by's
> >>>>> into the commit messages and then I'll apply, thanks!
> >>>>>
> >>>>
> >>>> Base on master (there it applies fine but causes a trivial build
> >>>> warning) or some other tree?
> >>>
> >>> Applying: virt-dt: Allow reservation of secure region when in a RAM
> >>> carveout
> >>> Using index info to reconstruct a base tree...
> >>> Falling back to patching base and 3-way merge...
> >>> Auto-merging arch/arm/lib/bootm-fdt.c
> >>> Auto-merging arch/arm/include/asm/armv7.h
> >>> CONFLICT (content): Merge conflict in arch/arm/include/asm/armv7.h
> >>> Auto-merging arch/arm/cpu/armv7/virt-dt.c
> >>> CONFLICT (content): Merge conflict in arch/arm/cpu/armv7/virt-dt.c
> >>> Recorded preimage for 'arch/arm/cpu/armv7/virt-dt.c'
> >>> Recorded preimage for 'arch/arm/include/asm/armv7.h'
> >>> Failed to merge in the changes.
> >>> Patch failed at 0008 virt-dt: Allow reservation of secure region when in
> >>> a RAM carveout
> >>>
> >>> Is what I saw trying to git am -3 the current series to top of tree
> >>> master.
> >>
> >> adcc5705? Strange, doesn't happen here.
> >
> > Nope, 820ca15
>
> Then please don't forget to push that baseline eventually ;). Or is
> public git.denx.de updated only once a day or so? There used to be some
> delay in the past, IIRC.

Bah, 820ca15 wasn't public and was the first 7 parts of the series
applied, but same problem with git am on adcc5705 :)

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150408/9fb65eb2/attachment.sig>