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 E49C3C433F5 for ; Mon, 14 Mar 2022 15:35:42 +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:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Subject:Cc:To:From:Message-ID:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=PFttWMAyty/ROoR9ZbP/49aiwItVPjjYfpogj6xYIAY=; b=DfO7AlqpUtfnvF lLOXFY/UiIDyug/UUlz+MjsWZMl0h03srD7/Uom+qIJ/LwGb9O/C1aY/OIi791DIV1PUtIu3Hxuxj 27Dy74kcujyXTwh7Djqye6csJv1b2ARp/sk1+3q6n2gczL5Cx+hE2P0cZO6Z0W4FNMa2StPKfEsZh 0MixPSl+Xgaj8Tkt8YpqlzkpbNLvo/i+e2BVcupey9qABGvWct9JLftOX7V0WSw7JdzemHJICnGEn OJCD9SoKQU946XopUxcddKnK+etddPROOvYLg6xMYS3JG++h009swKZ23ByemM6fYqPG2PpzPIDWQ Q6ONHcP2DAQoHtbpmqXA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nTmiG-005piZ-6k; Mon, 14 Mar 2022 15:34:24 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nTmiC-005phy-PQ for linux-arm-kernel@lists.infradead.org; Mon, 14 Mar 2022 15:34:22 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 91944612B8; Mon, 14 Mar 2022 15:34:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E602CC340EE; Mon, 14 Mar 2022 15:34:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1647272058; bh=Bnod0IaDodE5BPjyT8EujzYwnAusdu53AQQMt9y+ho0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=t73gaSIorFbG9uLRYGMqM/SKSnxBXcquakW7oZSLnr5fn/xbNlsCcEZUc3nUf5wEg h86fWkK9Mnr72d+gKXGBvnXJLii41dy44LTzrSUbwNdE3o2Di/+Ey50oC0PK5nmKvp BQprk6DEzypkv4Pt5ukLAnIKwDGPd50xPB0Yft5cKc+1BoZGEFv9Fv18e0onClDcSm jVP543t8JjYbhewEHMtmHfU0I5Mv+3Ih/lC5tCXtQAqChKb95MY2dI3jskRNa8di9z ah7gQD3sLQgiM8QaAZfYd4HbsGiIvglUI5BGm4rE6FlehO6CuYu43MIiUI/PwfGMUJ ayOONeHPot1Rw== Received: from sofa.misterjones.org ([185.219.108.64] helo=why.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1nTmi7-00EOSv-HQ; Mon, 14 Mar 2022 15:34:15 +0000 Date: Mon, 14 Mar 2022 15:34:15 +0000 Message-ID: <87k0cwzgy0.wl-maz@kernel.org> From: Marc Zyngier To: Rob Herring Cc: Vladimir Oltean , devicetree@vger.kernel.org, "linux-kernel@vger.kernel.org" , linux-arm-kernel , Shawn Guo , Li Yang , Biwen Li , Zhiqiang Hou , Kurt Kanzenbach , Rasmus Villemoes , Arnd Bergmann Subject: Re: [PATCH v2 devicetree] Revert "arm64: dts: freescale: Fix 'interrupt-map' parent address cells" In-Reply-To: References: <20211214135852.2934670-1-vladimir.oltean@nxp.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/27.1 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: robh+dt@kernel.org, vladimir.oltean@nxp.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, shawnguo@kernel.org, leoyang.li@nxp.com, biwen.li@nxp.com, Zhiqiang.Hou@nxp.com, kurt@linutronix.de, linux@rasmusvillemoes.dk, arnd@arndb.de X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220314_083421_275951_60E894AE X-CRM114-Status: GOOD ( 25.52 ) 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: , 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 On Mon, 14 Mar 2022 15:15:15 +0000, Rob Herring wrote: > > On Tue, Dec 14, 2021 at 6:59 AM Vladimir Oltean wrote: > > Therefore, the premise of the patch being reverted here is invalid. > > It doesn't matter whether the driver, in its non-standard use of the > > property, complies to the standard format or not, since this property > > isn't expected to be used for interrupt translation by the core. > > I disagree. The non-standard part is that 'interrupt-map' translation > is not transparent. 'interrupt-map' that can't be parsed in the > standard way is just wrong, and I imagine was never the intention > here. We simply cannot have platforms defining their own format for > standard properties. That ship sailed a long while ago. We have a list of offenders, and we can make sure we don't get additional ones. > Reverting this will cause dtc warnings now (IIRC) and just kicks the > can down the road. Reverting is fine for now (I gave Arnd the okay on > IRC), but I think the parsing will need to be updated to honor > #address-cells and detect an old DT (probably by looking at the total > size of 'interrupt-map') and mark that change for stable. That would > only leave a new dt with an old kernel without stable updates broken. > Seems unlikely a device is getting firmware updates, but not OS > updates. Being able to rollback firmware and OS independently is important. The tooling can be taught about the broken instances, which should be enough. Adding to the parsing only makes things harder to maintain, for no real gain. M. -- Without deviation from the norm, progress is not possible. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel