[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAE-0n52CVuc_kQbpwEnHEEODS8jPt52v6P=gAdVxqenPu4ehkw@mail.gmail.com>
Date: Thu, 31 Mar 2022 14:18:31 -0700
From: Stephen Boyd <swboyd@...omium.org>
To: Bjorn Andersson <bjorn.andersson@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Satya Priya Kakitapalli <quic_c_skakit@...cinc.com>
Cc: Lee Jones <lee.jones@...aro.org>,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>,
Das Srinagesh <gurus@...eaurora.org>,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, quic_collinsd@...cinc.com,
quic_subbaram@...cinc.com, quic_jprakash@...cinc.com
Subject: Re: [PATCH V8 7/7] arm64: dts: qcom: sc7280: Add pm8008 support for sc7280-idp
Quoting Satya Priya Kakitapalli (Temp) (2022-03-30 03:44:01)
>
> On 3/25/2022 11:18 PM, Stephen Boyd wrote:
> > #include "pm8008.dtsi"
> >
> > and then the nodes below.
>
>
> You mean
>
> pm8008_bus: &i2c1 {
>
> status = "okay";
>
> };
>
> &pm8008_bus {
>
> #include "pm8008.dtsi"
>
> };
>
> and remaining nodes, right?
>
No. I meant what I wrote.
&pm8008_bus: &i2c1 {
status = "okay";
};
#include "pm8008.dtsi"
and then the pm8008.dtsi file would have
&pm8008_bus {
pm8008: pmic@8 {
compatible = "qcom,pm8008";
...
};
pm8008_regulators: regulators@9 {
compatible = "qcom,pm8008-regulators";
...
};
};
and then this patch would modify those nodes with phandles.
Powered by blists - more mailing lists