Re: [PATCH 2/2] dt-check-style: Handle indented fragments in indent-consistent

[Sven Peter <sven@kernel.org>]

On 9/1/26 10:22, Krzysztof Kozlowski wrote:
> 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.

Fair enough, just ignore this series then.
I'll just do the whitespace fix then and un-indent the -dieX files for 
apple which I think should make them pass the checker then.


Sven