diff options
author | Frans Hendriks <fhendriks@eltan.com> | 2021-02-01 11:44:37 +0100 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2021-02-05 09:42:19 +0000 |
commit | a9caa50e8ac08338caf29c8452d624e316e07581 (patch) | |
tree | 444af3e557fa8444cf36d4aed3cf1dd1d16551c7 /src/device/azalia_device.c | |
parent | 3d6d1075b2ba17a357143f518715a911e09b38ec (diff) |
device: correct code style
Revise the following aspects to follow coreboot's coding style:
- Drop braces for single-statement condition and loop bodies.
- Use `__func__` to print the current function's name.
- Reflow pointer dereferences to fit in a single line.
- Adjust the `*` position in pointer variable declarations.
- Drop unnecessary `else` statements.
BUG = N/A
TEST = Build Compulab Intense-PC with secure oprom enabled
Change-Id: I780251d946d5bea97658476d61d25555ec768dfc
Signed-off-by: Frans Hendriks <fhendriks@eltan.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49963
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/device/azalia_device.c')
-rw-r--r-- | src/device/azalia_device.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/device/azalia_device.c b/src/device/azalia_device.c index e1899f13c0..e383821d55 100644 --- a/src/device/azalia_device.c +++ b/src/device/azalia_device.c @@ -174,9 +174,9 @@ static int wait_for_valid(u8 *base) reg32 |= HDA_ICII_BUSY | HDA_ICII_VALID; write32(base + HDA_ICII_REG, reg32); - while (timeout--) { + while (timeout--) udelay(1); - } + timeout = 50; while (timeout--) { reg32 = read32(base + HDA_ICII_REG); |