Patrick Georgi submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Duncan Laurie: Looks good to me, approved
drivers/wifi/generic: Use is_dev_enabled() instead of dev->enabled

This change replaces the checks for dev->enabled with the helper
function `is_dev_enabled()`.

Change-Id: Iacceda396c9300bbfa124e76fb9c99d86313ea0f
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/46904
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M src/drivers/wifi/generic/acpi.c
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/drivers/wifi/generic/acpi.c b/src/drivers/wifi/generic/acpi.c
index 33862d9..ac2c5eb 100644
--- a/src/drivers/wifi/generic/acpi.c
+++ b/src/drivers/wifi/generic/acpi.c
@@ -222,7 +222,7 @@
{
const char *path;

- if (!dev->enabled)
+ if (!is_dev_enabled(dev))
return;

path = acpi_device_path(dev);
@@ -247,7 +247,7 @@
{
const char *path;

- if (!dev->enabled)
+ if (!is_dev_enabled(dev))
return;

path = acpi_device_path(dev->bus->dev);

To view, visit change 46904. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iacceda396c9300bbfa124e76fb9c99d86313ea0f
Gerrit-Change-Number: 46904
Gerrit-PatchSet: 3
Gerrit-Owner: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Duncan Laurie <dlaurie@chromium.org>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged