Message-ID: <1b39712b-86c1-4eff-83ea-eb8b180db48c@kernel.org>
Date: Mon, 15 Sep 2025 06:41:45 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Andrew Morton <akpm@...ux-foundation.org>
Cc: Askar Safin <safinaskar@...il.com>, linux-fsdevel@...r.kernel.org,
 linux-kernel@...r.kernel.org, Linus Torvalds
 <torvalds@...ux-foundation.org>,
 Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
 Christian Brauner <brauner@...nel.org>, Al Viro <viro@...iv.linux.org.uk>,
 Jan Kara <jack@...e.cz>, Christoph Hellwig <hch@....de>,
 Jens Axboe <axboe@...nel.dk>, Andy Shevchenko <andy.shevchenko@...il.com>,
 Aleksa Sarai <cyphar@...har.com>,
 Thomas Weißschuh <thomas.weissschuh@...utronix.de>,
 Julian Stecklina <julian.stecklina@...erus-technology.de>,
 Gao Xiang <hsiangkao@...ux.alibaba.com>, Art Nikpal <email2tema@...il.com>,
 Eric Curtin <ecurtin@...hat.com>, Alexander Graf <graf@...zon.com>,
 Rob Landley <rob@...dley.net>, Lennart Poettering <mzxreary@...inter.de>,
 linux-arch@...r.kernel.org, linux-alpha@...r.kernel.org,
 linux-snps-arc@...ts.infradead.org, linux-arm-kernel@...ts.infradead.org,
 linux-csky@...r.kernel.org, linux-hexagon@...r.kernel.org,
 loongarch@...ts.linux.dev, linux-m68k@...ts.linux-m68k.org,
 linux-mips@...r.kernel.org, linux-openrisc@...r.kernel.org,
 linux-parisc@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org,
 linux-riscv@...ts.infradead.org, linux-s390@...r.kernel.org,
 linux-sh@...r.kernel.org, sparclinux@...r.kernel.org,
 linux-um@...ts.infradead.org, x86@...nel.org, Ingo Molnar
 <mingo@...hat.com>, linux-block@...r.kernel.org, initramfs@...r.kernel.org,
 linux-api@...r.kernel.org, linux-doc@...r.kernel.org,
 linux-efi@...r.kernel.org, linux-ext4@...r.kernel.org,
 "Theodore Y . Ts'o" <tytso@....edu>, linux-acpi@...r.kernel.org,
 Michal Simek <monstr@...str.eu>, devicetree@...r.kernel.org,
 Luis Chamberlain <mcgrof@...nel.org>, Kees Cook <kees@...nel.org>,
 Thorsten Blum <thorsten.blum@...ux.dev>, Heiko Carstens <hca@...ux.ibm.com>,
 patches@...ts.linux.dev
Subject: Re: [PATCH RESEND 21/62] init: remove all mentions of root=/dev/ram*

On 14/09/2025 22:13, Andrew Morton wrote:
> On Sun, 14 Sep 2025 12:06:24 +0200 Krzysztof Kozlowski <krzk@...nel.org> wrote:
> 
>>>  Documentation/admin-guide/kernel-parameters.txt          | 3 +--
>>>  Documentation/arch/m68k/kernel-options.rst               | 9 ++-------
>>>  arch/arm/boot/dts/arm/integratorap.dts                   | 2 +-
>>>  arch/arm/boot/dts/arm/integratorcp.dts                   | 2 +-
>>>  arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-cmm.dts     | 2 +-
>>>  .../boot/dts/aspeed/aspeed-bmc-facebook-galaxy100.dts    | 2 +-
>>>  .../arm/boot/dts/aspeed/aspeed-bmc-facebook-minipack.dts | 2 +-
>>>  .../arm/boot/dts/aspeed/aspeed-bmc-facebook-wedge100.dts | 2 +-
>>>  arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-wedge40.dts | 2 +-
>>>  arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yamp.dts    | 2 +-
>>>  .../boot/dts/aspeed/ast2600-facebook-netbmc-common.dtsi  | 2 +-
>>
>> No, don't do that. DTS is always separate.
> 
> Why can't DTS changes be carried in a different tree?


It must be carried in a different kernel tree and it must be ALWAYS a
separate commit. Embedding it in the middle of this patchset and in the
middle of some other commit breaks these two rules.

If you asked why it cannot be carried by VFS (or by any non-SoC tree in
general), it is because DTS is completely independent hardware
description, so by keeping it on separate tree we enforce that rule of
lack of dependency between DTS and any driver or core code.

If there is a dependency here, then it would be a NAK, because there
cannot be such - it would be a breach of contract for outside users (DTS
is shared with other, non-Linux projects).


Best regards,
Krzysztof

Powered by blists - more mailing lists