| Message ID | 20210404164914.712946-1-bjorn.andersson@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | [GIT,PULL] Qualcomm ARM64 DT updates for 5.13 | expand |
From: Arnd Bergmann <arnd@arndb.de> On Sun, 4 Apr 2021 11:49:14 -0500, Bjorn Andersson wrote: > The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15: > > Linux 5.12-rc2 (2021-03-05 17:33:41 -0800) > > are available in the Git repository at: > > https://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-for-5.13 > > [...] I noticed a couple of new warnings from 'make dtbs_check W=1': qcom/sc7180.dtsi:1204.21-1220.6: Warning (avoid_unnecessary_addr_size): /soc@0/geniqup@ac0000/i2c@a8c000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property qcom/sc7180.dtsi:965.21-981.6: Warning (avoid_unnecessary_addr_size): /soc@0/geniqup@8c0000/i2c@890000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property qcom/sdm845.dtsi:3912.23-4045.5: Warning (simple_bus_reg): /soc@0/camss@a00000: simple-bus unit address format error, expected "acb3000" qcom/sdm845.dtsi:4041.10-4044.6: Warning (graph_child_address): /soc@0/camss@a00000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary qcom/sdm845.dtsi:4102.32-4129.5: Warning (simple_bus_reg): /soc@0/dsi-opp-table: missing or empty reg/ranges property qcom/msm8916-samsung-a5u-eur.dt.yaml: spmi@200f000: reg: [[33615872, 4096], [37748736, 4194304], [46137344, 4194304], [58720256, 2097152], [33595392, 8448]] is too long qcom/sc7180-trogdor-lazor-r0.dt.yaml: gmu@506a000: compatible:0: 'qcom,adreno-gmu-618.0' is not one of ['qcom,adreno-gmu-630.2'] qcom/sc7180-trogdor-lazor-r1-kb.dt.yaml: memory@80900000: 'device_type' is a required property qcom/sdm850-lenovo-yoga-c630.dt.yaml: memory@97b00000: 'device_type' is a required property Maybe see if you can address these in a follow-up, to avoid regressions. Merged into arm/dt, thanks! merge commit: 61bac46eed14ad73cd41c5b7075350fef78ca4be Arnd
On Thu 08 Apr 10:24 CDT 2021, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > On Sun, 4 Apr 2021 11:49:14 -0500, Bjorn Andersson wrote: > > The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15: > > > > Linux 5.12-rc2 (2021-03-05 17:33:41 -0800) > > > > are available in the Git repository at: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-for-5.13 > > > > [...] > > I noticed a couple of new warnings from 'make dtbs_check W=1': > > qcom/sc7180.dtsi:1204.21-1220.6: Warning (avoid_unnecessary_addr_size): /soc@0/geniqup@ac0000/i2c@a8c000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property > qcom/sc7180.dtsi:965.21-981.6: Warning (avoid_unnecessary_addr_size): /soc@0/geniqup@8c0000/i2c@890000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property qcom/sdm845.dtsi:3912.23-4045.5: Warning (simple_bus_reg): /soc@0/camss@a00000: simple-bus unit address format error, expected "acb3000" > qcom/sdm845.dtsi:4041.10-4044.6: Warning (graph_child_address): /soc@0/camss@a00000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary > qcom/sdm845.dtsi:4102.32-4129.5: Warning (simple_bus_reg): /soc@0/dsi-opp-table: missing or empty reg/ranges property > qcom/msm8916-samsung-a5u-eur.dt.yaml: spmi@200f000: reg: [[33615872, 4096], [37748736, 4194304], [46137344, 4194304], [58720256, 2097152], [33595392, 8448]] is too long > qcom/sc7180-trogdor-lazor-r0.dt.yaml: gmu@506a000: compatible:0: 'qcom,adreno-gmu-618.0' is not one of ['qcom,adreno-gmu-630.2'] > qcom/sc7180-trogdor-lazor-r1-kb.dt.yaml: memory@80900000: 'device_type' is a required property > qcom/sdm850-lenovo-yoga-c630.dt.yaml: memory@97b00000: 'device_type' is a required property > I saw your other email regarding these and per Rob's request we've ramped up our effort on cleaning these out. I've however still not figured out a good way to incorporate the dtbs_check in my patch handling workflow. > Maybe see if you can address these in a follow-up, to avoid regressions. > I'm about to send you a another set of pull requests with some more goodies that was lingering on the list. I will take a look to see if I can follow up on that with some fixes for above warnings - and take another look at incorporating dtbs_check in my workflow. > Merged into arm/dt, thanks! > Thank you! Bjorn > merge commit: 61bac46eed14ad73cd41c5b7075350fef78ca4be > > Arnd
On Thu, Apr 8, 2021 at 6:06 PM Bjorn Andersson <bjorn.andersson@linaro.org> wrote: > On Thu 08 Apr 10:24 CDT 2021, Arnd Bergmann wrote: > > Maybe see if you can address these in a follow-up, to avoid regressions. > > > > I'm about to send you a another set of pull requests with some more > goodies that was lingering on the list. I will take a look to see if I > can follow up on that with some fixes for above warnings - and take > another look at incorporating dtbs_check in my workflow. Ok, sounds good. As I mentioned, I'm also just learning how to use dtbs_check properly. Unfortunately, it takes ages to run on a non-parallel build, and using 'make -j32' as I normally do means the output gets reordered with every run. It's probably ok if you figure out how to only run it for the files you actually change in the Qualcomm tree. Arnd
On Thu, 8 Apr 2021 at 23:05, Arnd Bergmann <arnd@kernel.org> wrote: > > On Thu, Apr 8, 2021 at 6:06 PM Bjorn Andersson > <bjorn.andersson@linaro.org> wrote: > > On Thu 08 Apr 10:24 CDT 2021, Arnd Bergmann wrote: > > > Maybe see if you can address these in a follow-up, to avoid regressions. > > > > > > > I'm about to send you a another set of pull requests with some more > > goodies that was lingering on the list. I will take a look to see if I > > can follow up on that with some fixes for above warnings - and take > > another look at incorporating dtbs_check in my workflow. > > Ok, sounds good. As I mentioned, I'm also just learning how to use > dtbs_check properly. Unfortunately, it takes ages to run on a > non-parallel build, > and using 'make -j32' as I normally do means the output gets reordered > with every run. > > It's probably ok if you figure out how to only run it for the files you > actually change in the Qualcomm tree. Disabling all non-Qualcomm arches makes `make dtbs_check` a little bit less painful.
On Thu 08 Apr 16:53 CDT 2021, Dmitry Baryshkov wrote: > On Thu, 8 Apr 2021 at 23:05, Arnd Bergmann <arnd@kernel.org> wrote: > > > > On Thu, Apr 8, 2021 at 6:06 PM Bjorn Andersson > > <bjorn.andersson@linaro.org> wrote: > > > On Thu 08 Apr 10:24 CDT 2021, Arnd Bergmann wrote: > > > > Maybe see if you can address these in a follow-up, to avoid regressions. > > > > > > > > > > I'm about to send you a another set of pull requests with some more > > > goodies that was lingering on the list. I will take a look to see if I > > > can follow up on that with some fixes for above warnings - and take > > > another look at incorporating dtbs_check in my workflow. > > > > Ok, sounds good. As I mentioned, I'm also just learning how to use > > dtbs_check properly. Unfortunately, it takes ages to run on a > > non-parallel build, > > and using 'make -j32' as I normally do means the output gets reordered > > with every run. > > > > It's probably ok if you figure out how to only run it for the files you > > actually change in the Qualcomm tree. > > Disabling all non-Qualcomm arches makes `make dtbs_check` a little bit > less painful. > Yes, this is a trick I'm applying as well. It would however be nice if one didn't have to disable CONFIG_ARCH_* to achieve this, or even better if one could run dtbs_check on a single .dtb. In particular since many of the warnings are resolved by fixing the bindings, which causes dtbs_check to recheck all files. Regards, Bjorn
On Fri, 9 Apr 2021 at 02:05, Bjorn Andersson <bjorn.andersson@linaro.org> wrote: > > On Thu 08 Apr 16:53 CDT 2021, Dmitry Baryshkov wrote: > > > On Thu, 8 Apr 2021 at 23:05, Arnd Bergmann <arnd@kernel.org> wrote: > > > > > > On Thu, Apr 8, 2021 at 6:06 PM Bjorn Andersson > > > <bjorn.andersson@linaro.org> wrote: > > > > On Thu 08 Apr 10:24 CDT 2021, Arnd Bergmann wrote: > > > > > Maybe see if you can address these in a follow-up, to avoid regressions. > > > > > > > > > > > > > I'm about to send you a another set of pull requests with some more > > > > goodies that was lingering on the list. I will take a look to see if I > > > > can follow up on that with some fixes for above warnings - and take > > > > another look at incorporating dtbs_check in my workflow. > > > > > > Ok, sounds good. As I mentioned, I'm also just learning how to use > > > dtbs_check properly. Unfortunately, it takes ages to run on a > > > non-parallel build, > > > and using 'make -j32' as I normally do means the output gets reordered > > > with every run. > > > > > > It's probably ok if you figure out how to only run it for the files you > > > actually change in the Qualcomm tree. > > > > Disabling all non-Qualcomm arches makes `make dtbs_check` a little bit > > less painful. > > > > Yes, this is a trick I'm applying as well. > > It would however be nice if one didn't have to disable CONFIG_ARCH_* to > achieve this, or even better if one could run dtbs_check on a single > .dtb. In particular since many of the warnings are resolved by fixing > the bindings, which causes dtbs_check to recheck all files. As a quick hack: add the following rule to the top-level Makefile: %.dt.yaml: include/config/kernel.release scripts_dtc $(Q)$(MAKE) $(build)=Documentation/devicetree/bindings Documentation/devicetree/bindings/processed-schema.json $(Q)$(MAKE) $(build)=$(dtstree) $(dtstree)/$@ CHECK_DTBS=y Now you can run checks for a single device tree by running 'make ARCH=arm64 ...... sdm845-db845.dy.yaml' I don't know if it's worth including this into the upstream source tree.
The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15: Linux 5.12-rc2 (2021-03-05 17:33:41 -0800) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-for-5.13 for you to fetch changes up to e9269650db2f5b4aeeafced1a42b2e51d10bcdf5: arm64: dts: qcom: qrb5165-rb5: switch into using GPIO for SPI0 CS (2021-03-18 09:35:42 -0500) ---------------------------------------------------------------- Qualcomm ARM64 DT updates for 5.13 This extends the initial SM8350 description merged in v5.12 with CPUfreq, SMMU, UFS, RPMHPD, SPMI, USB and remoteproc support. It adds initial PMIC definitions for the 6 PMICs found on the MTP and it introduces the new SM8350 Hardware Development Kit (HDK). SC7180 is further polished, the DisplayPort portion of the QMP phy is defined and several new SKUs of the Trogdor devices are introduced. The new SC7280 platform is introduced, with RPMH, RPMHPD, RPMCC, SPMI, CPU idle, SMMU and watchdog defined. SDM845 gains the camera related nodes and some cleanups. For SM8250 it brings some cleanups and migrates SPI0 to use GPIO for chip select. ---------------------------------------------------------------- Abhishek Kumar (1): arm64: dts: qcom: sc7180: add GO_LAZOR variant property for lazor Alexandru M Stan (1): arm64: dts: qcom: sc7180-trogdor: Remove fp control pins in prep for coachz Bjorn Andersson (3): arm64: dts: qcom: Introduce SM8350 HDK dt-bindings: arm: qcom: Add SM8350 HDK arm64: dts: qcom: pm8150: Enable RTC Dmitry Baryshkov (4): arm64: dts: qcom: sm8250: split spi pinctrl config arm64: dts: qcom: sm8250: further split of spi pinctrl config arm64: dts: qcom: sm8250: add pinctrl for SPI using GPIO as a CS arm64: dts: qcom: qrb5165-rb5: switch into using GPIO for SPI0 CS Douglas Anderson (7): arm64: dts: qcom: Move sc7180 MI2S config to board files and make pulldown arm64: dts: qcom: Prep sc7180-trogdor trackpad IRQ for new boards arm64: dts: qcom: Unify the sc7180-trogdor panel nodes arm64: dts: qcom: Add sc7180-lazor-limozeen skus arm64: dts: qcom: Add sc7180-trogdor-pompom skus arm64: dts: qcom: Add sc7180-trogdor-coachz skus arm64: dts: qcom: sc7180: Avoid glitching SPI CS at bootup on trogdor Jack Pham (2): arm64: dts: qcom: sm8350: add USB and PHY device nodes arm64: dts: qcom: sm8350-mtp: enable USB nodes Matthias Kaehlcke (5): arm64: dts: qcom: sc7180: trogdor: Add label to charger thermal zone arm64: dts: qcom: sc7180: Disable charger thermal zone for lazor arm64: dts: qcom: sc7180: trogdor: Fix trip point config of charger thermal zone arm64: dts: qcom: sc7180: trogdor: Use ADC TM channel 0 instead of 1 for charger temperature arm64: dts: qcom: sc7180: Set up lazor r3+ as sc7180-lite SKUs Maulik Shah (3): arm64: dts: qcom: sc7280: Add RSC and PDC devices arm64: dts: qcom: sc7280: Add reserved memory for fw arm64: dts: qcom: sc7280: Add cpuidle states Rajendra Nayak (4): dt-bindings: arm: qcom: Document sc7280 SoC and board arm64: dts: qcom: sc7280: Add basic dts/dtsi files for sc7280 soc arm64: dts: qcom: SC7280: Add rpmhcc clock controller node arm64: dts: qcom: sc7280: Add rpmh power-domain node Robert Foss (3): arm64: dts: qcom: sdm845: Add CAMSS ISP node arm64: dts: qcom: sdm845-db845c: Configure regulators for camss node arm64: dts: qcom: sdm845-db845c: Enable ov8856 sensor and connect to ISP Sai Prakash Ranjan (9): arm64: dts: qcom: sc7180: Rename the qmp node to power-controller arm64: dts: qcom: sdm845: Rename the qmp node to power-controller arm64: dts: qcom: sm8250: Rename the qmp node to power-controller arm64: dts: qcom: sm8350: Rename the qmp node to power-controller arm64: dts: qcom: sm8250: Fix level triggered PMU interrupt polarity arm64: dts: qcom: sm8350: Fix level triggered PMU interrupt polarity arm64: dts: qcom: sm8250: Fix timer interrupt to specify EL2 physical timer arm64: dts: qcom: sc7280: Add device node for APPS SMMU arm64: dts: qcom: sc7280: Add APSS watchdog node Sandeep Maheswaram (1): arm64: dts: qcom: sc7180: Use pdc interrupts for USB instead of GIC interrupts Shawn Guo (4): arm64: dts: qcom: sdm845: fix number of pins in 'gpio-ranges' arm64: dts: qcom: sm8150: fix number of pins in 'gpio-ranges' arm64: dts: qcom: sm8250: fix number of pins in 'gpio-ranges' arm64: dts: qcom: sm8350: fix number of pins in 'gpio-ranges' Srinivas Kandagatla (1): arm64: dts: qcom: db845c: fix correct powerdown pin for WSA881x Stephen Boyd (4): arm64: dts: qcom: sc7180: Update dts for DP phy inside QMP phy arm64: dts: qcom: trogdor: Only wakeup from pen eject arm64: dts: qcom: Disable camera clk on sc7180-trogdor devices by default arm64: dts: qcom: sc7180: Drop duplicate dp_hot_plug_det node in trogdor Venkata Lakshmi Narayana Gubba (1): arm64: dts: qcom: sc7180: Remove clock for bluetooth on Trogdor Vinod Koul (25): arm64: dts: qcom: sm8350: Add apss_smmu node arm64: dts: qcom: sm8350: Add UFS nodes arm64: dts: qcom: sm8350-mtp: enable UFS nodes arm64: dts: qcom: sm8350-mtp: add regulator names arm64: dts: qcom: sm8350: fix typo arm64: dts: qcom: sm8350: Add rpmhpd node arm64: dts: qcom: sm8350: Add rmtfs node arm64: dts: qcom: sm8350: Add SMP2P nodes arm64: dts: qcom: sm8350: Add remoteprocs arm64: dts: qcom: sm8350-mtp: Enable remoteprocs arm64: dts: qcom: sm8350: Use enums for GCC arm64: dts: qcom: sm8350: Add cpufreq node arm64: dts: qcom: apq8016-sbc: drop qcom,sbc arm64: dts: qcom: msm8916: don't use empty memory node dt-bindings: arm: qcom: Document ipq6018-cp01 board arm64: dts: qcom: msm8994: don't use empty memory node dt-bindings: arm: qcom: Document sony boards for apq8094 arm64: dts: qcom: sm8350: Add spmi node arm64: dts: qcom: pmk8350: Add base dts file arm64: dts: qcom: pm8350: Add base dts file arm64: dts: qcom: pm8350b: Add base dts file arm64: dts: qcom: pm8350c: Add base dts file arm64: dts: qcom: pmr735a: Add base dts file arm64: dts: qcom: pmr735B: Add base dts file arm64: dts: qcom: sm8350-mtp: Add PMICs satya priya (1): arm64: dts: qcom: sc7280: Add SPMI PMIC arbiter device for SC7280 Documentation/devicetree/bindings/arm/qcom.yaml | 18 + arch/arm64/boot/dts/qcom/Makefile | 13 + arch/arm64/boot/dts/qcom/apq8016-sbc.dts | 2 +- arch/arm64/boot/dts/qcom/msm8916.dtsi | 4 +- arch/arm64/boot/dts/qcom/msm8994.dtsi | 4 +- arch/arm64/boot/dts/qcom/pm8150.dtsi | 4 +- arch/arm64/boot/dts/qcom/pm8350.dtsi | 25 + arch/arm64/boot/dts/qcom/pm8350b.dtsi | 25 + arch/arm64/boot/dts/qcom/pm8350c.dtsi | 25 + arch/arm64/boot/dts/qcom/pmk8350.dtsi | 25 + arch/arm64/boot/dts/qcom/pmr735a.dtsi | 25 + arch/arm64/boot/dts/qcom/pmr735b.dtsi | 25 + arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 18 +- .../boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dts | 18 + .../boot/dts/qcom/sc7180-trogdor-coachz-r1.dts | 154 +++++ .../boot/dts/qcom/sc7180-trogdor-coachz-r2-lte.dts | 18 + .../boot/dts/qcom/sc7180-trogdor-coachz-r2.dts | 15 + .../arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi | 240 +++++++ .../qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dts | 34 + .../qcom/sc7180-trogdor-lazor-limozeen-nots.dts | 26 + .../dts/qcom/sc7180-trogdor-lazor-limozeen.dts | 42 ++ .../boot/dts/qcom/sc7180-trogdor-lazor-r0.dts | 9 + .../boot/dts/qcom/sc7180-trogdor-lazor-r1.dts | 9 + .../boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dts | 5 +- .../boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dts | 4 +- .../boot/dts/qcom/sc7180-trogdor-lazor-r3.dts | 10 + arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor.dtsi | 39 +- .../boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dts | 14 + .../boot/dts/qcom/sc7180-trogdor-pompom-r1.dts | 26 + .../boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dts | 14 + .../boot/dts/qcom/sc7180-trogdor-pompom-r2.dts | 44 ++ .../arm64/boot/dts/qcom/sc7180-trogdor-pompom.dtsi | 288 ++++++++ arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dts | 35 +- arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 162 ++--- arch/arm64/boot/dts/qcom/sc7180.dtsi | 51 +- arch/arm64/boot/dts/qcom/sc7280-idp.dts | 47 ++ arch/arm64/boot/dts/qcom/sc7280.dtsi | 608 +++++++++++++++++ arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 27 +- arch/arm64/boot/dts/qcom/sdm845.dtsi | 139 +++- arch/arm64/boot/dts/qcom/sm8150.dtsi | 2 +- arch/arm64/boot/dts/qcom/sm8250-mtp.dts | 4 - arch/arm64/boot/dts/qcom/sm8250.dtsi | 548 ++++++++-------- arch/arm64/boot/dts/qcom/sm8350-hdk.dts | 319 +++++++++ arch/arm64/boot/dts/qcom/sm8350-mtp.dts | 116 ++++ arch/arm64/boot/dts/qcom/sm8350.dtsi | 726 ++++++++++++++++++++- 45 files changed, 3553 insertions(+), 453 deletions(-) create mode 100644 arch/arm64/boot/dts/qcom/pm8350.dtsi create mode 100644 arch/arm64/boot/dts/qcom/pm8350b.dtsi create mode 100644 arch/arm64/boot/dts/qcom/pm8350c.dtsi create mode 100644 arch/arm64/boot/dts/qcom/pmk8350.dtsi create mode 100644 arch/arm64/boot/dts/qcom/pmr735a.dtsi create mode 100644 arch/arm64/boot/dts/qcom/pmr735b.dtsi create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r2-lte.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r2.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom.dtsi create mode 100644 arch/arm64/boot/dts/qcom/sc7280-idp.dts create mode 100644 arch/arm64/boot/dts/qcom/sc7280.dtsi create mode 100644 arch/arm64/boot/dts/qcom/sm8350-hdk.dts