From f4842bbc145da72c37321fed876b3140f79c6aa7 Mon Sep 17 00:00:00 2001 From: Felix Singer Date: Fri, 12 Jan 2024 21:21:06 +0100 Subject: sb/intel: Use boolean for pcie_hotplug_map attribute Change-Id: Ia5e1ee683fa3d8d216ece26426e9870981ede2ba Signed-off-by: Felix Singer Reviewed-on: https://review.coreboot.org/c/coreboot/+/79932 Reviewed-by: Patrick Rudolph Tested-by: build bot (Jenkins) --- src/southbridge/intel/bd82x6x/chip.h | 2 +- src/southbridge/intel/common/pciehp.c | 3 ++- src/southbridge/intel/common/pciehp.h | 4 +++- src/southbridge/intel/i82801ix/chip.h | 3 ++- src/southbridge/intel/i82801jx/chip.h | 3 ++- 5 files changed, 10 insertions(+), 5 deletions(-) diff --git a/src/southbridge/intel/bd82x6x/chip.h b/src/southbridge/intel/bd82x6x/chip.h index 30c2675513..66d5cb13fe 100644 --- a/src/southbridge/intel/bd82x6x/chip.h +++ b/src/southbridge/intel/bd82x6x/chip.h @@ -65,7 +65,7 @@ struct southbridge_intel_bd82x6x_config { int docking_supported; - uint8_t pcie_hotplug_map[8]; + bool pcie_hotplug_map[8]; /* Ports which can be routed to either EHCI or xHCI. */ uint32_t xhci_switchable_ports; diff --git a/src/southbridge/intel/common/pciehp.c b/src/southbridge/intel/common/pciehp.c index b5496192b0..d5766d8f90 100644 --- a/src/southbridge/intel/common/pciehp.c +++ b/src/southbridge/intel/common/pciehp.c @@ -4,10 +4,11 @@ #include #include #include +#include #include "pciehp.h" -void intel_acpi_pcie_hotplug_generator(u8 *hotplug_map, int port_number) +void intel_acpi_pcie_hotplug_generator(bool *hotplug_map, int port_number) { int port; int have_hotplug = 0; diff --git a/src/southbridge/intel/common/pciehp.h b/src/southbridge/intel/common/pciehp.h index b4e65c4f46..beb210c7b0 100644 --- a/src/southbridge/intel/common/pciehp.h +++ b/src/southbridge/intel/common/pciehp.h @@ -1,9 +1,11 @@ /* SPDX-License-Identifier: GPL-2.0-only */ +#include + #ifndef SOUTHBRIDGE_INTEL_COMMON_PCIEHP_H #define SOUTHBRIDGE_INTEL_COMMON_PCIEHP_H -void intel_acpi_pcie_hotplug_generator(u8 *hotplug_map, int port_number); +void intel_acpi_pcie_hotplug_generator(bool *hotplug_map, int port_number); void intel_acpi_pcie_hotplug_scan_slot(struct bus *bus); #endif diff --git a/src/southbridge/intel/i82801ix/chip.h b/src/southbridge/intel/i82801ix/chip.h index ec7b977081..d52a0c2006 100644 --- a/src/southbridge/intel/i82801ix/chip.h +++ b/src/southbridge/intel/i82801ix/chip.h @@ -3,6 +3,7 @@ #ifndef SOUTHBRIDGE_INTEL_I82801IX_CHIP_H #define SOUTHBRIDGE_INTEL_I82801IX_CHIP_H +#include #include enum { @@ -73,7 +74,7 @@ struct southbridge_intel_i82801ix_config { uint8_t scale : 2; } pcie_power_limits[6]; - uint8_t pcie_hotplug_map[8]; + bool pcie_hotplug_map[8]; /* Additional LPC IO decode ranges */ uint32_t gen1_dec; diff --git a/src/southbridge/intel/i82801jx/chip.h b/src/southbridge/intel/i82801jx/chip.h index ae31d4f224..f12a67e877 100644 --- a/src/southbridge/intel/i82801jx/chip.h +++ b/src/southbridge/intel/i82801jx/chip.h @@ -3,6 +3,7 @@ #ifndef SOUTHBRIDGE_INTEL_I82801JX_CHIP_H #define SOUTHBRIDGE_INTEL_I82801JX_CHIP_H +#include #include enum { @@ -59,7 +60,7 @@ struct southbridge_intel_i82801jx_config { uint8_t scale : 2; } pcie_power_limits[6]; - uint8_t pcie_hotplug_map[8]; + bool pcie_hotplug_map[8]; /* Additional LPC IO decode ranges */ uint32_t gen1_dec; -- cgit v1.2.3