On 01/02/2024 16:55, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > > Add a node for the PMU module of the QCA6391 present on the RB5 board. > Assign its LDO power outputs to the existing Bluetooth module. Add a > node for the PCIe port to sm8250.dtsi and define the WLAN node on it in > the board's .dts and also make it consume the power outputs of the PMU. > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > --- > arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 128 +++++++++++++++++++++-- > arch/arm64/boot/dts/qcom/sm8250.dtsi | 10 ++ > 2 files changed, 127 insertions(+), 11 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts > index cd0db4f31d4a..fab5bebafbad 100644 > --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts > +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts > @@ -108,6 +108,87 @@ lt9611_3v3: lt9611-3v3 { > regulator-always-on; > }; > > + qca6390_pmu: pmu@0 { It does not look like you tested the DTS against bindings. Please run `make dtbs_check W=1` (see Documentation/devicetree/bindings/writing-schema.rst or https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/ for instructions). Best regards, Krzysztof