diff options
| author | Andrew Wu <arw@dmp.com.tw> | 2013-08-02 14:45:03 +0800 |
|---|---|---|
| committer | Anton Kochkov <anton.kochkov@gmail.com> | 2013-08-13 10:11:18 +0200 |
| commit | 9361daf7fdaa6a4fa81c06e4cdfb7b42b2915051 (patch) | |
| tree | 749b1836d8e8e0331af087a69c82d760633126fe /src/southbridge/amd/sb800 | |
| parent | 3be80cce290e3ff2da6abc30addab79f5a5ffa07 (diff) | |
| download | coreboot-9361daf7fdaa6a4fa81c06e4cdfb7b42b2915051.tar.xz | |
Fix some wait_for_valid functions return value from 1 to -1.
codec_init expects wait_for_valid returns -1 for timeout, not 1.
Change-Id: I0f2a3ebb1934d0adaf13765434526bbc9efca9a3
Signed-off-by: Andrew Wu <arw@dmp.com.tw>
Reviewed-on: http://review.coreboot.org/3843
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martin.roth@se-eng.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/southbridge/amd/sb800')
| -rw-r--r-- | src/southbridge/amd/sb800/hda.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/southbridge/amd/sb800/hda.c b/src/southbridge/amd/sb800/hda.c index c43e9490f0..ad8f80f0d8 100644 --- a/src/southbridge/amd/sb800/hda.c +++ b/src/southbridge/amd/sb800/hda.c @@ -132,7 +132,7 @@ static int wait_for_valid(u32 base) udelay(1); } - return 1; + return -1; } static void codec_init(u32 base, int addr) |