On Mon, Jun 17, 2024 at 7:31 PM Frank Li <Frank.Li@xxxxxxx> wrote: > + reg_pcie0: regulator-pcie { > + compatible = "regulator-fixed"; > + regulator-name = "PCIE_WLAN_EN"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + vin-supply = <®_m2_pwr>; > + gpio = <&i2c7_pcal6524 6 GPIO_ACTIVE_HIGH>; > + enable-active-high; > + regulator-always-on; As this regulator has a consumer, 'regulator-always-on' can be removed. > + }; > + > + reg_slot_pwr: regulator-slot-pwr { > + compatible = "regulator-fixed"; > + regulator-name = "PCIe slot-power"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + gpio = <&i2c7_pcal6524 14 GPIO_ACTIVE_HIGH>; > + enable-active-high; > + regulator-always-on; Ditto.