diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2020-12-05 19:02:49 +0100 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2020-12-17 20:17:44 +0000 |
| commit | a1a317ebf65c63017e02d017284f8564facaa1c4 (patch) | |
| tree | 5e5f5f8e4e657349deb1ecac4aa8e0a2caef2bcd /src/soc/intel | |
| parent | ac857ca3b166d385caa4faf62b4f1b8fc3b3f2da (diff) | |
| download | coreboot-a1a317ebf65c63017e02d017284f8564facaa1c4.tar.xz | |
soc/intel/common/hda_verb.c: Clarify mask usage
The `azalia_set_bits` will mask out all bits, so just use zero for
clarity. The resulting behavior is the same in both cases.
Change-Id: I27777f1e836fa973859629d48964060bec02c87a
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48357
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Diffstat (limited to 'src/soc/intel')
| -rw-r--r-- | src/soc/intel/common/hda_verb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/intel/common/hda_verb.c b/src/soc/intel/common/hda_verb.c index ecf5a5167d..5a4b467b8a 100644 --- a/src/soc/intel/common/hda_verb.c +++ b/src/soc/intel/common/hda_verb.c @@ -24,7 +24,7 @@ int hda_codec_detect(u8 *base) write8(base + HDA_STATESTS_REG, 0xf); /* Turn off the link and poll RESET# bit until it reads back as 0 */ - if (azalia_set_bits(base + HDA_GCTL_REG, HDA_GCTL_CRST, ~HDA_GCTL_CRST) < 0) + if (azalia_set_bits(base + HDA_GCTL_REG, HDA_GCTL_CRST, 0) < 0) goto no_codec; /* Turn on the link and poll RESET# bit until it reads back as 1 */ |