From a853e7acdba6b58e2efb57e186ae0d14ea85fad2 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Mon, 7 Dec 2020 12:28:38 +0100 Subject: nb/intel/sandybridge: Introduce `iosav_run_once_and_wait` Most ofte, `iosav_run_once` precedes a `wait_for_iosav` call. Add a helper function to reduce clutter. The cases where `iosav_run_once` isn't followed by `wait_for_iosav` will be handled in a follow-up. Tested on Asus P8Z77-V LX2, still boots. Change-Id: Ic76f53c2db41512287f41b696a0c4df42a5e0f12 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/48402 Reviewed-by: Felix Held Tested-by: build bot (Jenkins) --- src/northbridge/intel/sandybridge/raminit_iosav.c | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/northbridge/intel/sandybridge/raminit_iosav.c') diff --git a/src/northbridge/intel/sandybridge/raminit_iosav.c b/src/northbridge/intel/sandybridge/raminit_iosav.c index d83dfd8e9a..b3c6db6f78 100644 --- a/src/northbridge/intel/sandybridge/raminit_iosav.c +++ b/src/northbridge/intel/sandybridge/raminit_iosav.c @@ -49,6 +49,12 @@ void wait_for_iosav(int channel) } } +void iosav_run_once_and_wait(const int ch) +{ + iosav_run_once(ch); + wait_for_iosav(ch); +} + void iosav_write_zqcs_sequence(int channel, int slotrank, u32 gap, u32 post, u32 wrap) { const struct iosav_ssq sequence[] = { -- cgit v1.2.3