diff options
author | Antonio Nino Diaz <antonio.ninodiaz@arm.com> | 2017-11-17 16:58:33 +0000 |
---|---|---|
committer | Sandrine Bailleux <sandrine.bailleux@arm.com> | 2017-11-23 10:48:52 +0000 |
commit | 82dd66df3e2e23cc2e7e227cb90ff94cffccf9f1 (patch) | |
tree | 1ee18be6a93458b511875553a9d88cb52ffc2b67 /lib/power_management/suspend/tftf_suspend.c | |
parent | a4090b5754a7f448db8cca4274946fd18e9c9c13 (diff) |
Ensure that PSTATE is updated right after a change
It is needed to add a isb after modifying PSTATE.{D,A,I,F} so that the
effects are seen immediately. For example, this is useful to make sure
that pending interrupt are handled as soon as the interrupts are
enabled.
Some definitions in `include/lib/aarch64/arch_helpers.h` have been moved
inside the file so that isb() is defined before it is used.
This commit makes it unnecesary to have a isb() after any of the
functions modified by this file, so all isb() after them have been
removed across the whole codebase.
The fact that the old version of the functions needed a isb() made it
easy to introduce bugs, as it was easy to forget about the isb(). There
are some cases in the codebase as well, that are now fixed.
Change-Id: If9f9335f8889f2e23900307eb6a6236143577e10
Signed-off-by: Antonio Nino Diaz <antonio.ninodiaz@arm.com>
Diffstat (limited to 'lib/power_management/suspend/tftf_suspend.c')
-rw-r--r-- | lib/power_management/suspend/tftf_suspend.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/power_management/suspend/tftf_suspend.c b/lib/power_management/suspend/tftf_suspend.c index eba161f..9b08448 100644 --- a/lib/power_management/suspend/tftf_suspend.c +++ b/lib/power_management/suspend/tftf_suspend.c @@ -111,7 +111,6 @@ int tftf_suspend(const suspend_info_t *info) flags = read_daif(); disable_irq(); - isb(); INFO("Going into suspend state\n"); |