On 3/21/2025 6:01 AM, Bryan O'Donoghue wrote:

On 20/03/2025 05:55, Wenbin Yao wrote:

quoted

From: Qiang Yu <redacted>

Enable the pwrctrl driver, which is utilized to manage the power 
supplies
of the devices connected to the PCI slots. This ensures that the voltage
rails of the x8 PCI slots on the X1E80100 - QCP can be correctly turned
on/off if they are described under PCIe port device tree node.

Signed-off-by: Qiang Yu <redacted>
Signed-off-by: Wenbin Yao <redacted>
---
  arch/arm64/configs/defconfig | 1 +
  1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 85ec2fba1..de86d1121 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig

@@ -245,6 +245,7 @@ CONFIG_PCIE_LAYERSCAPE_GEN4=y

  CONFIG_PCI_ENDPOINT=y
  CONFIG_PCI_ENDPOINT_CONFIGFS=y
  CONFIG_PCI_EPF_TEST=m
+CONFIG_PCI_PWRCTL_SLOT=y
  CONFIG_DEVTMPFS=y
  CONFIG_DEVTMPFS_MOUNT=y
  CONFIG_FW_LOADER_USER_HELPER=y
-- 
2.34.1

PCI_PWRCTL_SLOT is a tristate symbol why be a "y" instead of an "m" 
i.e. compile this into the kernel instead of having it be a module ?

It can be compiled as module if this is preferred.

---
bod

-- 
With best wishes
Wenbin