Re: [PATCH 2/2] dt-check-style: Handle indented fragments in indent-consistent
From: Krzysztof Kozlowski
Date: Tue Sep 01 2026 - 04:29:49 EST
On Sun, Aug 30, 2026 at 08:24:55PM +0200, Sven Peter wrote:
> diff --git a/scripts/dtc/dt-style-selftest/expected/dts-fragment-indent.dtsi.txt b/scripts/dtc/dt-style-selftest/expected/dts-fragment-indent.dtsi.txt
> new file mode 100644
> index 000000000000..84ec8eb73468
> --- /dev/null
> +++ b/scripts/dtc/dt-style-selftest/expected/dts-fragment-indent.dtsi.txt
> @@ -0,0 +1,3 @@
> +# mode=strict
> +bad/dts-fragment-indent.dtsi:9: [indent-consistent] indent mismatch (expected depth 2 * '\t')
> +bad/dts-fragment-indent.dtsi:10: [indent-consistent] indent mismatch (expected depth 2 * '\t')
> diff --git a/scripts/dtc/dt-style-selftest/good/dts-fragment.dtsi b/scripts/dtc/dt-style-selftest/good/dts-fragment.dtsi
> new file mode 100644
> index 000000000000..58735112d3aa
> --- /dev/null
> +++ b/scripts/dtc/dt-style-selftest/good/dts-fragment.dtsi
> @@ -0,0 +1,19 @@
> +// SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +/* Include fragment indented for its inclusion context. */
> +
> + bus@10000 {
So here is a good example... and this is code is wrong.
You CANNOT have MMIO node outside of MMIO bus or a new node outside of
root node indented with one tab.
Best regards,
Krzysztof