mbox series

[GIT,PULL] Qualcomm ARM64 DT updates for 5.13

Message ID 20210404164914.712946-1-bjorn.andersson@linaro.org (mailing list archive)
State Mainlined, archived
Commit e9269650db2f5b4aeeafced1a42b2e51d10bcdf5
Headers show
Series [GIT,PULL] Qualcomm ARM64 DT updates for 5.13 | expand

Pull-request

https://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git tags/qcom-arm64-for-5.13

Message

Bjorn Andersson April 4, 2021, 4:49 p.m. UTC
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

Comments

Arnd Bergmann April 8, 2021, 3:24 p.m. UTC | #1
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
Bjorn Andersson April 8, 2021, 4:06 p.m. UTC | #2
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
Arnd Bergmann April 8, 2021, 8:04 p.m. UTC | #3
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
Dmitry Baryshkov April 8, 2021, 9:53 p.m. UTC | #4
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.
Bjorn Andersson April 8, 2021, 11:05 p.m. UTC | #5
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
Dmitry Baryshkov April 8, 2021, 11:41 p.m. UTC | #6
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.