diff options
author | Bill Richardson <wfrichar@chromium.org> | 2011-09-23 14:42:26 -0700 |
---|---|---|
committer | Bill Richardson <wfrichar@chromium.org> | 2011-09-23 16:04:15 -0700 |
commit | 4313fba2fb928f662a63b7566f235291dc1455f7 (patch) | |
tree | 593510a01782588a495e8cc954aef23f8f9efd3b /firmware/lib/vboot_display.c | |
parent | 132e6e0c8cfa49a470199374e2331e3bb2ea21d6 (diff) | |
download | vboot-4313fba2fb928f662a63b7566f235291dc1455f7.tar.gz |
VbExBeep() now returns VbError_t instead of void
This enables us to support playing sounds in the background if the BIOS
allows it, so we don't have to block while beeping is happening. The new
declaration is:
VbError_t VbExBeep(uint32_t msec, uint32_t frequency);
If the audio codec can run in the background, then:
zero frequency means OFF, non-zero frequency means ON
zero msec means return immediately, non-zero msec means delay (and
then OFF if needed)
else:
non-zero msec and non-zero frequency means ON, delay, OFF, return
zero msec or zero frequency means do nothing and return immediately
The return value is used by the caller to determine the capabilities. The
implementation should always do the best it can if it cannot fully support
all features - for example, beeping at a fixed frequency if frequency
support is not available. At a minimum, it must delay for the specified
non-zero duration.
Currently, VbExBeep() is called only when displaying the dev-mode screen.
BUG=none
TEST=manual
I've tested on x86 and ARM, all timeouts and noises work as before.
Note that ARM and coreboot will require a corresponding change to their
VbExBeep() implementations, which will have to be handled with separate,
simultaneous CLs.
Change-Id: I3417ae4b99d9d0aee63f2ccaeed39b61d4333e5d
Reviewed-on: http://gerrit.chromium.org/gerrit/8234
Tested-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-by: Stefan Reinauer <reinauer@google.com>
Diffstat (limited to 'firmware/lib/vboot_display.c')
-rw-r--r-- | firmware/lib/vboot_display.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/firmware/lib/vboot_display.c b/firmware/lib/vboot_display.c index bd2a93c8..b0166e81 100644 --- a/firmware/lib/vboot_display.c +++ b/firmware/lib/vboot_display.c @@ -315,7 +315,7 @@ VbError_t VbDisplayScreenFromGBB(VbCommonParams* cparams, uint32_t screen, */ if (gbb->major_version == GBB_MAJOR_VER && gbb->minor_version >= 1 && (gbb->flags != 0)) { - (void)VbExDisplayDebugInfo("GBB.flags is nonzero"); + (void)VbExDisplayDebugInfo("gbb.flags is nonzero"); } |