summaryrefslogtreecommitdiff
path: root/src/soc
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2022-11-02 19:01:34 +0100
committerFelix Singer <service+coreboot-gerrit@felixsinger.de>2024-04-11 18:17:33 +0000
commit4c333a6b4607db65b93020a892c5e770fbb65828 (patch)
treeffd40eced5dd243efd25d8d6cdbc104e98dca7d6 /src/soc
parent5eecafe89ea34f035dc310b89e6f77c5e55e1b64 (diff)
soc/intel/**/fast_spi.c: Reorganize some statements
Avoid calling `acpi_device_scope()` and `fast_spi_acpi_hid()` if the result won't be used. Also, reorder a condition so that compile-time constants appear first, so as to help the compiler optimize it out. Change-Id: I42ce55c2978ad9c593c359c5decd5842fb3a97a1 Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/69168 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
Diffstat (limited to 'src/soc')
-rw-r--r--src/soc/intel/common/block/fast_spi/fast_spi.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/soc/intel/common/block/fast_spi/fast_spi.c b/src/soc/intel/common/block/fast_spi/fast_spi.c
index 74449df746..53826447c6 100644
--- a/src/soc/intel/common/block/fast_spi/fast_spi.c
+++ b/src/soc/intel/common/block/fast_spi/fast_spi.c
@@ -502,17 +502,17 @@ static const char *fast_spi_acpi_name(const struct device *dev)
*/
static void fast_spi_fill_ssdt(const struct device *dev)
{
+ /* Do not add SSDT if the fast SPI device is hidden. */
+ if (!CONFIG(FAST_SPI_GENERATE_SSDT) || dev->hidden)
+ return;
+
const char *scope = acpi_device_scope(dev);
const char *hid = fast_spi_acpi_hid(dev);
- struct resource *res;
- /* Do not add SSDT if the fast SPI device is hidden. */
- if (dev->hidden || !CONFIG(FAST_SPI_GENERATE_SSDT))
- return;
if (!scope || !hid)
return;
- res = probe_resource(dev, PCI_BASE_ADDRESS_0);
+ struct resource *res = probe_resource(dev, PCI_BASE_ADDRESS_0);
if (!res)
return;