Skip to content

Leave space for trailing \0 after MAX_FILENAME#13

Open
berolinux wants to merge 1 commit intoMarvellEmbeddedProcessors:atf-v1.4-armada-18.06from
OpenMandrivaSoftware:atf-v1.4-armada-18.06-fixes
Open

Leave space for trailing \0 after MAX_FILENAME#13
berolinux wants to merge 1 commit intoMarvellEmbeddedProcessors:atf-v1.4-armada-18.06from
OpenMandrivaSoftware:atf-v1.4-armada-18.06-fixes

Conversation

@berolinux
Copy link

Detected by gcc 8.x warnings

Signed-off-by: Bernhard Rosenkränzer bero@lindev.ch

Detected by gcc 8.x warnings

Signed-off-by: Bernhard Rosenkränzer <bero@lindev.ch>
isoc-platform-ci pushed a commit that referenced this pull request Dec 25, 2018
Do not power off the CPU1 since there is no way to wake it up
(wake-up is causing CPU0 reset as well duo to HW bug). Quote from errata
Ref #13 [In power saving mode, both cores must be powered off]:
"When Core 0 is on and Core 1 is in power-off state, a Core 1
wake-up resets Core 0 as well and puts Core 0 back to ROM".

To overcome described HW bug instead of powering the CPU off, let it
reach WFI instruction, which is invoked by generic psci_do_cpu_off
function after platform handler finishes. This will put the core in low
power state and give a chance to wake it up.

Before this change, after running secondary kernel via kexec, only one
core was up, now both cores are up.

Change-Id: I87f144867550728055d9b8a2edb84a14539acab7
Signed-off-by: Grzegorz Jaszczyk <jaz@semihalf.com>
Reviewed-on: http://vgitil04.il.marvell.com:8080/60935
Tested-by: iSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: Igal Liberman <igall@marvell.com>
Reviewed-by: Kostya Porotchkin <kostap@marvell.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant