| Message ID | 20220606160421.1641778-2-luca@z3ntu.xyz (mailing list archive) |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | [1/2] ARM: dts: qcom: msm8974: re-add missing pinctrl | expand |
Hi Luca,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on robh/for-next v5.19-rc1 next-20220609]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/intel-lab-lkp/linux/commits/Luca-Weiss/ARM-dts-qcom-msm8974-re-add-missing-pinctrl/20220607-002318
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git f2906aa863381afb0015a9eb7fefad885d4e5a56
config: arm-randconfig-c002-20220608 (https://download.01.org/0day-ci/archive/20220609/202206091706.4QP4H2JW-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project b92436efcb7813fc481b30f2593a4907568d917a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/intel-lab-lkp/linux/commit/68b0d8fbd2f10572118c481ea108f3dfb7f46a4f
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Luca-Weiss/ARM-dts-qcom-msm8974-re-add-missing-pinctrl/20220607-002318
git checkout 68b0d8fbd2f10572118c481ea108f3dfb7f46a4f
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
also defined at arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts:291.18-293.3
>> ERROR: Input tree has errors, aborting (use -f to force output)
On Mon 06 Jun 11:04 CDT 2022, Luca Weiss wrote: > As part of a recent cleanup commit, the remoteproc supplies for adsp and > modem were removed from msm8974.dtsi and now need to be set in the > device dts. Do so. > > Fixes: f300826d27be ("ARM: dts: qcom-msm8974: Sort and clean up nodes") > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- > Bjorn, could you please pick this up for -fixes so it lands in an > upcoming 5.19-rc? > As reported by lkp, this doesn't build. Am I missing something? Regards, Bjorn
Hi Bjorn, On Freitag, 1. Juli 2022 05:15:05 CEST Bjorn Andersson wrote: > On Mon 06 Jun 11:04 CDT 2022, Luca Weiss wrote: > > As part of a recent cleanup commit, the remoteproc supplies for adsp and > > modem were removed from msm8974.dtsi and now need to be set in the > > device dts. Do so. > > > > Fixes: f300826d27be ("ARM: dts: qcom-msm8974: Sort and clean up nodes") > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > > --- > > Bjorn, could you please pick this up for -fixes so it lands in an > > upcoming 5.19-rc? > > As reported by lkp, this doesn't build. Am I missing something? I briefly mentioned in IRC, maybe you missed it. Castor is actually still missing pm8841 regulators upstream. I'll send this old (rebased) commit from you to add them, is that ok with you? https://github.com/msm8974-mainline/linux/commit/ d01e20807a00e78570abf3dd91bea555dff06fa9 Otherwise we can just keep castor out of this patch for now and add it later, I don't really mind either way. Regards Luca > > Regards, > Bjorn
diff --git a/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts b/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts index 9493886a5c0d..2d699de713ce 100644 --- a/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts +++ b/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts @@ -313,6 +313,17 @@ otg { }; }; +&remoteproc_adsp { + cx-supply = <&pm8841_s2>; +}; + +&remoteproc_mss { + cx-supply = <&pm8841_s2>; + mss-supply = <&pm8841_s3>; + mx-supply = <&pm8841_s1>; + pll-supply = <&pm8941_l12>; +}; + &rpm_requests { pm8841-regulators { compatible = "qcom,rpm-pm8841-regulators"; diff --git a/arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine.dtsi b/arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine.dtsi index 1d21de46f85c..60bf96304687 100644 --- a/arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine.dtsi +++ b/arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine.dtsi @@ -183,6 +183,17 @@ &pm8941_wled { qcom,num-strings = <2>; }; +&remoteproc_adsp { + cx-supply = <&pm8841_s2>; +}; + +&remoteproc_mss { + cx-supply = <&pm8841_s2>; + mss-supply = <&pm8841_s3>; + mx-supply = <&pm8841_s1>; + pll-supply = <&pm8941_l12>; +}; + &rpm_requests { pm8841-regulators { compatible = "qcom,rpm-pm8841-regulators"; diff --git a/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts b/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts index 9bd8faea61a5..48f0c47a45ef 100644 --- a/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts +++ b/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts @@ -288,6 +288,17 @@ lcd_dcdc_en_pin_a: lcd-dcdc-en-active { }; +&remoteproc_adsp { + cx-supply = <&pm8841_s2>; +}; + +&remoteproc_mss { + cx-supply = <&pm8841_s2>; + mss-supply = <&pm8841_s3>; + mx-supply = <&pm8841_s1>; + pll-supply = <&pm8941_l12>; +}; + &rpm_requests { pm8941-regulators { compatible = "qcom,rpm-pm8941-regulators";
As part of a recent cleanup commit, the remoteproc supplies for adsp and modem were removed from msm8974.dtsi and now need to be set in the device dts. Do so. Fixes: f300826d27be ("ARM: dts: qcom-msm8974: Sort and clean up nodes") Signed-off-by: Luca Weiss <luca@z3ntu.xyz> --- Bjorn, could you please pick this up for -fixes so it lands in an upcoming 5.19-rc? .../boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts | 11 +++++++++++ arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine.dtsi | 11 +++++++++++ .../qcom-msm8974pro-sony-xperia-shinano-castor.dts | 11 +++++++++++ 3 files changed, 33 insertions(+)