From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 45618C433F5 for ; Fri, 7 Jan 2022 19:41:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:List-Subscribe:List-Help: List-Post:List-Archive:List-Unsubscribe:List-Id:Message-ID:References:Subject :In-Reply-To:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=mNmUr6r75Dvb+C21IOMNtgv6iLbOvX91pHqu0hv4Lf0=; b=1IYAfhMvYQT3ra gKIlFOLf+/S6QETz9vWAI/wNnN3Cl1DcS63lr519lWgfx/4xaKKvAOHUH4cbB6Kv2St6hIyZ7yEr2 daQQ3acCULjYkZAaM5deUVtweG/K9bhYN3Tm2muntj+3X0/MyBZjnXZgK7qSlS5zS7MGmoA45RoHu UU6pReq7bzsLyR1B1g7Rcl564elC9ObS7IaaYYC+ON5+eo6HDbJm0sWJi8JSBPqDMqLnPysqn9ceP Dkygjd5b8YqFTJdnY4RXv/lMkFNn2UEd1hteYwmSzwCyAwMQ+w4M78A/hXTGqkzfXvTd5nf28Na60 QbiuRGTDxZY+o221UiVg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5v5s-0055n9-Hi; Fri, 07 Jan 2022 19:40:08 +0000 Received: from sibelius.xs4all.nl ([83.163.83.176]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5v5n-0055lZ-N1 for linux-arm-kernel@lists.infradead.org; Fri, 07 Jan 2022 19:40:06 +0000 Received: from localhost (bloch.sibelius.xs4all.nl [local]) by bloch.sibelius.xs4all.nl (OpenSMTPD) with ESMTPA id ac96d287; Fri, 7 Jan 2022 20:39:53 +0100 (CET) Date: Fri, 7 Jan 2022 20:39:53 +0100 (CET) From: Mark Kettenis To: Krzysztof Kozlowski Cc: jszhang@kernel.org, shawnguo@kernel.org, leoyang.li@nxp.com, robh+dt@kernel.org, linux@armlinux.org.uk, andrew@lunn.ch, sebastian.hesselbarth@gmail.com, thierry.reding@gmail.com, jonathanh@nvidia.com, hayashi.kunihiko@socionext.com, mhiramat@kernel.org, nobuhiro1.iwamatsu@toshiba.co.jp, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-tegra@vger.kernel.org In-Reply-To: <99115cc4-32f6-d217-68be-33256a6993a8@canonical.com> (message from Krzysztof Kozlowski on Fri, 7 Jan 2022 13:47:03 +0100) Subject: Re: [PATCH 0/7] arm/arm64: dts: Remove unused num-viewport from pcie node References: <20211229160245.1338-1-jszhang@kernel.org> <99115cc4-32f6-d217-68be-33256a6993a8@canonical.com> Message-ID: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220107_114004_113287_9AAA25E3 X-CRM114-Status: GOOD ( 20.26 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org > Date: Fri, 7 Jan 2022 13:47:03 +0100 > From: Krzysztof Kozlowski > > On 29/12/2021 17:50, Mark Kettenis wrote: > >> From: Jisheng Zhang > >> Date: Thu, 30 Dec 2021 00:02:38 +0800 > >> > >> After commit 281f1f99cf3a("PCI: dwc: Detect number of iATU windows"), > >> the number of iATU windows is detected at runtime, what's more, > >> the 'num-viewport' property parsing has been removed, so remove the > >> unused num-viewport from pcie node(s). > >> > >> It's too late for linux-5.17-rc1, I will rebase and send out v2 if > >> necessary when 5.17-rc1 is released. > > > > Please no. This only makes the device trees unnecessarily > > incompatible with older kernels > > Anyone who is running a new DTB with older kernel is doomed anyway, not > only because of this change but hundreds of other similar cleanups, e.g. > making DTS conforming to dtschema. Are you sure there are such use cases > of using new DTB with old kernel? I cannot imagine making a stable > product with such scenario... Well, many of those changes just affect the node names, which aren't part of the ABI. And adding missing properties or compatibles doesn't break things either. But yes, we keep seeing diffs to "cleanup" bindings and device trees, especially in the context of converting them to dtschema. And that's just wrong. If old device trees don't pass validation, the default assumption should be that the schema is wrong; not the other way around. > > and other OSes that do rely on the > > "num-viewport" property. > > Right. We should have move the DTS out of the kernel when it was still > small. :) I don't think the number/size of DTs really matters. But yes, moving them to a separate repository would certainly make it more obvious that they should not be tied to a particular kernel version and that they are shared with other projects such as U-Boot and other OSes. > > It really doesn't hurt to keep this property > > even if future Linux kernels no longer look at it. > > For Exynos PCIe, the property is still required by bindings, so > definitely it cannot be removed from DTS. I did not check the other > bindings. > > Best regards, > Krzysztof Cheers, Mark _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel