diff options
| author | Oliver O'Halloran <oohall@gmail.com> | 2016-12-21 15:52:23 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2017-01-04 17:15:34 +1100 |
| commit | 458b3f8e38e4e61c880ba640d4a30550b2693813 (patch) | |
| tree | a28eb170728b50a9e9ed4306e82804fc13ed8304 /hw/fsp | |
| parent | f38ed2656a5f9983948b79ccf5e1d344035bf531 (diff) | |
| download | blackbird-skiboot-458b3f8e38e4e61c880ba640d4a30550b2693813.tar.gz blackbird-skiboot-458b3f8e38e4e61c880ba640d4a30550b2693813.zip | |
console: don't use __flush_console() outside console.c
There is only one use of this function outside of console.c and that
usage is broken. As the name suggests this is an internal function that
is only safe when the console lock held is held. flush_console() will
acquire the lock for the caller so that should be used instead.
Reviewed-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw/fsp')
| -rw-r--r-- | hw/fsp/fsp-console.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/fsp/fsp-console.c b/hw/fsp/fsp-console.c index 0080d732..46ac28bb 100644 --- a/hw/fsp/fsp-console.c +++ b/hw/fsp/fsp-console.c @@ -751,7 +751,7 @@ void fsp_console_poll(void *data __unused) continue; } if (fs->log_port) { - __flush_console(true); + flush_console(); opal_update_pending_evt(fs->irq, 0); } else { #ifdef OPAL_DEBUG_CONSOLE_POLL |
