1. Patchew
  2. linux
  3. View series

[PATCH v3 0/7] dtc: dt-check-style: Improvements for handling continued lines

Krzysztof Kozlowski posted 7 patches 2 weeks, 2 days ago
There is a newer version of this series
.../devicetree/bindings/dts-coding-style.rst       |   5 +-
scripts/dtc/dt-check-style                         | 240 ++++++++++-----------
.../dtc/dt-style-selftest/bad/dts-cont-align.dts   |   1 +
.../dtc/dt-style-selftest/bad/dts-line-length.dts  |   3 +-
.../dt-style-selftest/bad/dts-property-order.dts   |   3 +
.../dt-style-selftest/bad/dts-property-order.dtso  |   3 +
.../dtc/dt-style-selftest/bad/dts-trailing-ws.dts  |   8 +
.../dtc/dt-style-selftest/bad/yaml-cont-align.yaml |   1 +
.../dtc/dt-style-selftest/bad/yaml-hex-case.yaml   |   5 +-
.../dt-style-selftest/bad/yaml-line-length.yaml    |   3 +-
.../dtc/dt-style-selftest/bad/yaml-prop-order.yaml |   5 +
.../dt-style-selftest/bad/yaml-trailing-ws.yaml    |   5 +-
.../expected/dts-cont-align.dts.txt                |  11 +-
.../expected/dts-line-length.dts.txt               |   1 +
.../expected/dts-property-order.dts.txt            |   7 +-
.../expected/dts-property-order.dtso.txt           |   7 +-
.../expected/dts-property-pairing.dts.txt          |   6 +-
.../expected/dts-trailing-ws.dts.txt               |   1 +
.../expected/yaml-cont-align.yaml.txt              |   3 +-
.../expected/yaml-hex-case.yaml.txt                |   2 +
.../expected/yaml-line-length.yaml.txt             |   1 +
.../expected/yaml-prop-order.yaml.txt              |   1 +
.../expected/yaml-prop-pairing.yaml.txt            |   6 +-
.../expected/yaml-trailing-ws.yaml.txt             |   2 +
.../dtc/dt-style-selftest/good/dts-cont-align.dts  |   1 +
.../dt-style-selftest/good/dts-property-order.dts  |   3 +
.../dt-style-selftest/good/dts-property-order.dtso |   3 +
.../dt-style-selftest/good/yaml-cont-align.yaml    |   1 +
28 files changed, 190 insertions(+), 148 deletions(-)
[PATCH v3 0/7] dtc: dt-check-style: Improvements for handling continued lines
Posted by Krzysztof Kozlowski 2 weeks, 2 days ago 
Changes in v3:
- New commit:
  "dtc: dt-check-style: Relax property ordering rules (drop alphabetical)"
- Improve commit msg of "Unduplicate checks in
  check_mixed_indent_chars()" patch.
- Link to v2: https://patch.msgid.link/20260909-b4-dts-style-checker-continued-lines-v2-0-c5cb4c9d9b66@oss.qualcomm.com

Changes in v2:
1. Four new commits 1-4
2. Drop last commit
v1: https://lore.kernel.org/all/20260906180437.166760-6-krzysztof.kozlowski@oss.qualcomm.com/

Best regards,
Krzysztof

---
Krzysztof Kozlowski (7):
      dtc: dt-check-style: Handle continued lines in check_hex_case()
      dtc: dt-check-style: Handle continued lines in check_line_length()
      dtc: dt-check-style: Handle continued lines in check_trailing_whitespace()
      dtc: dt-check-style: Unduplicate checks in check_mixed_indent_chars()
      dtc: dt-check-style: Right strip whitespaces, leftovers before comments
      dtc: dt-check-style: Properly detect comments in multi-line properties
      dtc: dt-check-style: Relax property ordering rules (drop alphabetical)

 .../devicetree/bindings/dts-coding-style.rst       |   5 +-
 scripts/dtc/dt-check-style                         | 240 ++++++++++-----------
 .../dtc/dt-style-selftest/bad/dts-cont-align.dts   |   1 +
 .../dtc/dt-style-selftest/bad/dts-line-length.dts  |   3 +-
 .../dt-style-selftest/bad/dts-property-order.dts   |   3 +
 .../dt-style-selftest/bad/dts-property-order.dtso  |   3 +
 .../dtc/dt-style-selftest/bad/dts-trailing-ws.dts  |   8 +
 .../dtc/dt-style-selftest/bad/yaml-cont-align.yaml |   1 +
 .../dtc/dt-style-selftest/bad/yaml-hex-case.yaml   |   5 +-
 .../dt-style-selftest/bad/yaml-line-length.yaml    |   3 +-
 .../dtc/dt-style-selftest/bad/yaml-prop-order.yaml |   5 +
 .../dt-style-selftest/bad/yaml-trailing-ws.yaml    |   5 +-
 .../expected/dts-cont-align.dts.txt                |  11 +-
 .../expected/dts-line-length.dts.txt               |   1 +
 .../expected/dts-property-order.dts.txt            |   7 +-
 .../expected/dts-property-order.dtso.txt           |   7 +-
 .../expected/dts-property-pairing.dts.txt          |   6 +-
 .../expected/dts-trailing-ws.dts.txt               |   1 +
 .../expected/yaml-cont-align.yaml.txt              |   3 +-
 .../expected/yaml-hex-case.yaml.txt                |   2 +
 .../expected/yaml-line-length.yaml.txt             |   1 +
 .../expected/yaml-prop-order.yaml.txt              |   1 +
 .../expected/yaml-prop-pairing.yaml.txt            |   6 +-
 .../expected/yaml-trailing-ws.yaml.txt             |   2 +
 .../dtc/dt-style-selftest/good/dts-cont-align.dts  |   1 +
 .../dt-style-selftest/good/dts-property-order.dts  |   3 +
 .../dt-style-selftest/good/dts-property-order.dtso |   3 +
 .../dt-style-selftest/good/yaml-cont-align.yaml    |   1 +
 28 files changed, 190 insertions(+), 148 deletions(-)
---
base-commit: 694b801ceb1bea205878f2136e8b25c888e51be7
change-id: 20260909-b4-dts-style-checker-continued-lines-c6895bacb860

Best regards,
--  
Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Re: [PATCH v3 0/7] dtc: dt-check-style: Improvements for handling continued lines
Posted by Rob Herring 1 week, 3 days ago 
On Wed, Sep 09, 2026 at 03:10:34PM +0200, Krzysztof Kozlowski wrote:
> Changes in v3:
> - New commit:
>   "dtc: dt-check-style: Relax property ordering rules (drop alphabetical)"
> - Improve commit msg of "Unduplicate checks in
>   check_mixed_indent_chars()" patch.
> - Link to v2: https://patch.msgid.link/20260909-b4-dts-style-checker-continued-lines-v2-0-c5cb4c9d9b66@oss.qualcomm.com
> 
> Changes in v2:
> 1. Four new commits 1-4
> 2. Drop last commit
> v1: https://lore.kernel.org/all/20260906180437.166760-6-krzysztof.kozlowski@oss.qualcomm.com/
> 
> Best regards,
> Krzysztof
> 
> ---
> Krzysztof Kozlowski (7):
>       dtc: dt-check-style: Handle continued lines in check_hex_case()
>       dtc: dt-check-style: Handle continued lines in check_line_length()
>       dtc: dt-check-style: Handle continued lines in check_trailing_whitespace()
>       dtc: dt-check-style: Unduplicate checks in check_mixed_indent_chars()
>       dtc: dt-check-style: Right strip whitespaces, leftovers before comments
>       dtc: dt-check-style: Properly detect comments in multi-line properties
>       dtc: dt-check-style: Relax property ordering rules (drop alphabetical)

Patch 7 doesn't apply.

> base-commit: 694b801ceb1bea205878f2136e8b25c888e51be7

And I don't have this commit in my tree to resolve the conflicts.

Rob
Re: [PATCH v3 0/7] dtc: dt-check-style: Improvements for handling continued lines
Posted by Krzysztof Kozlowski 1 week, 3 days ago 
On 15/09/2026 16:27, Rob Herring wrote:
> On Wed, Sep 09, 2026 at 03:10:34PM +0200, Krzysztof Kozlowski wrote:
>> Changes in v3:
>> - New commit:
>>   "dtc: dt-check-style: Relax property ordering rules (drop alphabetical)"
>> - Improve commit msg of "Unduplicate checks in
>>   check_mixed_indent_chars()" patch.
>> - Link to v2: https://patch.msgid.link/20260909-b4-dts-style-checker-continued-lines-v2-0-c5cb4c9d9b66@oss.qualcomm.com
>>
>> Changes in v2:
>> 1. Four new commits 1-4
>> 2. Drop last commit
>> v1: https://lore.kernel.org/all/20260906180437.166760-6-krzysztof.kozlowski@oss.qualcomm.com/
>>
>> Best regards,
>> Krzysztof
>>
>> ---
>> Krzysztof Kozlowski (7):
>>       dtc: dt-check-style: Handle continued lines in check_hex_case()
>>       dtc: dt-check-style: Handle continued lines in check_line_length()
>>       dtc: dt-check-style: Handle continued lines in check_trailing_whitespace()
>>       dtc: dt-check-style: Unduplicate checks in check_mixed_indent_chars()
>>       dtc: dt-check-style: Right strip whitespaces, leftovers before comments
>>       dtc: dt-check-style: Properly detect comments in multi-line properties
>>       dtc: dt-check-style: Relax property ordering rules (drop alphabetical)
> 
> Patch 7 doesn't apply.
> 
>> base-commit: 694b801ceb1bea205878f2136e8b25c888e51be7
> 
> And I don't have this commit in my tree to resolve the conflicts.

Ah my bad, I had a WIP commit somewhere in the history used as base for
it. I'll rebase and send a new version

Best regards,
Krzysztof

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.