diff options
author | Karthikeyan Ramasubramanian <kramasub@google.com> | 2020-11-02 16:26:52 -0700 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-11-09 07:24:13 +0000 |
commit | d1c0f958d1986a94ac55ae1d196ba286c311b90f (patch) | |
tree | 2dbba3c69328215132f5eeaa32cc221007bb9dec /src/drivers/generic/max98357a | |
parent | 04582e900102357056aa218e5a413edb52409e87 (diff) | |
download | coreboot-d1c0f958d1986a94ac55ae1d196ba286c311b90f.tar.gz coreboot-d1c0f958d1986a94ac55ae1d196ba286c311b90f.tar.bz2 coreboot-d1c0f958d1986a94ac55ae1d196ba286c311b90f.zip |
acpi: Call acpi_fill_ssdt() only for enabled devices
Individual drivers check whether the concerned device is enabled before
filling in the SSDT. Move the check before calling acpi_fill_ssdt() and
remove the check in the individual drivers.
BUG=None
TEST=util/abuild/abuild
Change-Id: Ib042bec7e8c68b38fafa60a8e965d781bddcd1f0
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47148
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Christian Walter <christian.walter@9elements.com>
Diffstat (limited to 'src/drivers/generic/max98357a')
-rw-r--r-- | src/drivers/generic/max98357a/max98357a.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/generic/max98357a/max98357a.c b/src/drivers/generic/max98357a/max98357a.c index 44f88024902e..9ae4bf4658b9 100644 --- a/src/drivers/generic/max98357a/max98357a.c +++ b/src/drivers/generic/max98357a/max98357a.c @@ -18,7 +18,7 @@ static void max98357a_fill_ssdt(const struct device *dev) const char *path; struct acpi_dp *dp; - if (!dev->enabled || !config) + if (!config) return; const char *scope = acpi_device_scope(dev); |