Netdev List
 help / color / mirror / Atom feed
From: Henrik Grimler <henrik.grimler@axis.com>
To: Jesper Nilsson <jesper.nilsson@axis.com>,
	Lars Persson <lars.persson@axis.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Richard Cochran <richardcochran@gmail.com>
Cc: <linux-arm-kernel@axis.com>, <kernel@axis.com>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<netdev@vger.kernel.org>,
	Henrik Grimler <henrik.grimler@axis.com>
Subject: [PATCH v2 0/5] ARM: dts: artpec: cleanup with the help of dt-check-style
Date: Fri, 2 Oct 2026 11:49:11 +0200	[thread overview]
Message-ID: <20261002-b4-artpec6-cleanup-v2-0-8452b5d045ad@axis.com> (raw)

Fix most of the warnings that are reported by
scripts/dtc/dt-check-style, to make the files use the preferred
style. By doing so the files become easier to read and maintain.

I envision these patches to go in through the ARM SOC tree.

Sorting of nodes have been left out from this patchset, as that
creates a huge, un-reviewable, patch. After the patches some warnings
remain when dt-check-style (with Krzysztof's recent fixes [1]) is run
in strict mode:

  axis/artpec6.dtsi:77: [child-name-order] child node 'psci' out of name order
  axis/artpec6.dtsi:80: [property-name] property name "psci_version" is using discouraged style
  axis/artpec6.dtsi:82: [property-name] property name "system_reset" is using discouraged style
  axis/artpec6.dtsi:91: [child-name-order] child node 'ext-clk' out of name order
  axis/artpec6.dtsi:97: [child-name-order] child node 'eth-phy-ref-clk' out of name order
  axis/artpec6.dtsi:103: [child-name-order] child node 'clkctrl' out of name order
  axis/artpec6.dtsi:127: [child-name-order] child node 'interrupt-controller' out of name order
  axis/artpec6.dtsi:134: [child-name-order] child node 'cache-controller' out of name order
  axis/artpec6.dtsi:163: [child-name-order] child node 'pcie' out of name order
  axis/artpec6.dtsi:233: [child-name-order] child node 'amba' out of name order
  axis/artpec6.dtsi:292: [child-address-order] child node @f8010000 out of address order
  axis/artpec6.dtsi:314: [property-name] property name "snps,wr_osr_lmt" is using discouraged style
  axis/artpec6.dtsi:315: [property-name] property name "snps,rd_osr_lmt" is using discouraged style
  axis/artpec6.dtsi:324: [child-name-order] child node 'rx-queues-config' out of name order

Fixing these are left for the future.

[1] https://github.com/krzk/linux/tree/b4/dts-style-checker-blank-lines

Signed-off-by: Henrik Grimler <henrik.grimler@axis.com>
---
Changes in v2:
- Squash two patches and drop patch fixing false-positives (@krzk)
- Clarify that node re-naming has been tested and found to not break
  driver's dt parsing (@krzk)
- Use uppercase for ARTPEC-6 in commit messages, as it is an
  abbrevation, and that is how we stylise it
- Link to v1: https://patch.msgid.link/20260928-b4-artpec6-cleanup-v1-0-e3b75e10acc6@axis.com

To: Jesper Nilsson <jesper.nilsson@axis.com>
To: Lars Persson <lars.persson@axis.com>
To: Rob Herring <robh@kernel.org>
To: Krzysztof Kozlowski <krzk+dt@kernel.org>
To: Conor Dooley <conor+dt@kernel.org>
To: Richard Cochran <richardcochran@gmail.com>
Cc: linux-arm-kernel@axis.com
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: netdev@vger.kernel.org

---
Henrik Grimler (5):
      ARM: dts: axis: artpec6: avoid _ in node names
      ARM: dts: axis: artpec6: align all continued lines
      ARM: dts: axis: artpec6: use multiple tuples in regs and ranges
      ARM: dts: axis: artpec6: sort all properties
      ARM: dts: axis: artpec6: add more blank lines

 arch/arm/boot/dts/axis/artpec6-devboard.dts |  9 +--
 arch/arm/boot/dts/axis/artpec6.dtsi         | 88 ++++++++++++++++-------------
 2 files changed, 55 insertions(+), 42 deletions(-)
---
base-commit: f5f84daefcd92d7a630066635ecea1433ed5eac7
change-id: 20260928-b4-artpec6-cleanup-3dfb30a26b9d

Best regards,
-- 
Henrik Grimler <henrik.grimler@axis.com>


             reply	other threads:[~2026-10-02  9:50 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-02  9:49 Henrik Grimler [this message]
2026-10-02  9:49 ` [PATCH v2 1/5] ARM: dts: axis: artpec6: avoid _ in node names Henrik Grimler
2026-10-02  9:49 ` [PATCH v2 2/5] ARM: dts: axis: artpec6: align all continued lines Henrik Grimler
2026-10-02  9:49 ` [PATCH v2 3/5] ARM: dts: axis: artpec6: use multiple tuples in regs and ranges Henrik Grimler
2026-10-02  9:49 ` [PATCH v2 4/5] ARM: dts: axis: artpec6: sort all properties Henrik Grimler
2026-10-02  9:49 ` [PATCH v2 5/5] ARM: dts: axis: artpec6: add more blank lines Henrik Grimler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20261002-b4-artpec6-cleanup-v2-0-8452b5d045ad@axis.com \
    --to=henrik.grimler@axis.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jesper.nilsson@axis.com \
    --cc=kernel@axis.com \
    --cc=krzk+dt@kernel.org \
    --cc=lars.persson@axis.com \
    --cc=linux-arm-kernel@axis.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=richardcochran@gmail.com \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox