diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2020-12-05 19:06:55 +0100 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2020-12-17 20:25:09 +0000 |
| commit | 2e0053b84098512c5b3c98394998e66a8f83803b (patch) | |
| tree | 14eeb88ab35e53010f17ae01eda093665dd88c3b /src/southbridge/intel/i82801jx | |
| parent | a1a317ebf65c63017e02d017284f8564facaa1c4 (diff) | |
azalia: Use `azalia_enter_reset` function
Also tidy up some adjacent comments.
Change-Id: I2e881900a52e42ab3f43ffe96cfbdcc63ff02e23
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48358
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Diffstat (limited to 'src/southbridge/intel/i82801jx')
| -rw-r--r-- | src/southbridge/intel/i82801jx/azalia.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/src/southbridge/intel/i82801jx/azalia.c b/src/southbridge/intel/i82801jx/azalia.c index bea2e9000b..5efbc9f601 100644 --- a/src/southbridge/intel/i82801jx/azalia.c +++ b/src/southbridge/intel/i82801jx/azalia.c @@ -15,8 +15,7 @@ static int codec_detect(u8 *base) { u32 reg32; - /* Set Bit 0 to 0 to enter reset state (BAR + 0x8)[0] */ - if (azalia_set_bits(base + HDA_GCTL_REG, HDA_GCTL_CRST, 0) < 0) + if (azalia_enter_reset(base) < 0) goto no_codec; if (azalia_exit_reset(base) < 0) @@ -31,9 +30,8 @@ static int codec_detect(u8 *base) return reg32; no_codec: - /* Codec Not found */ - /* Put HDA back in reset (BAR + 0x8) [0] */ - azalia_set_bits(base + HDA_GCTL_REG, 1, 0); + /* Codec not found, put HDA back in reset */ + azalia_enter_reset(base); printk(BIOS_DEBUG, "Azalia: No codec!\n"); return 0; } |