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 7022FC4332F for ; Fri, 2 Dec 2022 10:14:12 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject: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=6D7jXPYtNnvHrktQAlOVUhft7RGYdhO0v40zN60GOBQ=; b=me6SgldzZ7p9tl auWPfqWL7UJq0zBQgoeWfdWsRCY4gzkYpHlvXNNHPuE3ixVyfLrzO/uDxiv8gZN1485xRuCWNoFkW gIa/rQVQmPPpD1pA2RqJZAECQoYiokaAMJ2vHF3Hm0ereOCt18Llm1iJTecc1YoxIzzzV8Y1Vpfrs 7e2OpR7192JSQ5cwB3BJalwhjguBxFOxqHCrgmLYehac2B4AITwqbG5LlO5oJJum+9gZv4k/20xr7 VGhhoDXGXzJ9qIZ4wnFacVtXmLJzzXL83tV5ab9eHSx2NlnixBB09wonN/NgXg/aIrnTDsmxTOIZI /EJSm8vPdEHITshnj+8A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p132i-00FLTF-6W; Fri, 02 Dec 2022 10:13:16 +0000 Received: from smtp-out-05.comm2000.it ([212.97.32.73]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p132f-00FLRJ-Bl; Fri, 02 Dec 2022 10:13:14 +0000 Received: from francesco-nb.int.toradex.com (93-49-2-63.ip317.fastwebnet.it [93.49.2.63]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: francesco@dolcini.it) by smtp-out-05.comm2000.it (Postfix) with ESMTPSA id 8965482630A; Fri, 2 Dec 2022 11:12:50 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=mailserver.it; s=mailsrv; t=1669975984; bh=EhXjLytaNloC3jO1CJxW82NXsFq0S8qD5JKyjHxWInE=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=hd0Ck2QDAmK4PlJ0kTMREObFocUpZZJ82Uv8R3IttXT8QzAVkT8Dl77W+TyA+KlBM LTzWsTyyOk8IqQ5Ty0ohGwdqt89z6+p91dIYeAUHl4EpYTiqCc5cnGNl/kuFGBOBMT IpjTGSJXPhJeQNNjTUE1tWEfSpkpBqMVISDOy/geQ0kkml++igPFEaz5iIoodtk6Sc Pf5DzGq+zMENYBiodvZyE4E4M1ReiuyAM+Qopo+0CR0GDJjfmR8LDoPSVQqRiuIACH 2j5YaRKGO3x92iXsY/KGIDwBfuGhDf6TRw3gTS7z8HLRTFCOEnQ3QUBU6x3IQuWQnu b1/VXmEtMqB+w== Date: Fri, 2 Dec 2022 11:12:43 +0100 From: Francesco Dolcini To: Miquel Raynal Cc: Francesco Dolcini , Richard Weinberger , Vignesh Raghavendra , linux-mtd@lists.infradead.org, Marek Vasut , Francesco Dolcini , Shawn Guo , linux-arm-kernel@lists.infradead.org, stable@vger.kernel.org Subject: Re: [PATCH v1] mtd: parsers: ofpart: Fix parsing when size-cells is 0 Message-ID: References: <20221202071900.1143950-1-francesco@dolcini.it> <20221202101418.6b4b3711@xps-13> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20221202101418.6b4b3711@xps-13> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221202_021313_587489_2C6EB4B8 X-CRM114-Status: GOOD ( 24.46 ) 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 Hello Miquel, On Fri, Dec 02, 2022 at 10:14:18AM +0100, Miquel Raynal wrote: > francesco@dolcini.it wrote on Fri, 2 Dec 2022 08:19:00 +0100: > > From: Francesco Dolcini > > > > Add a fallback mechanism to handle the case in which #size-cells is set > > to <0>. According to the DT binding the nand controller node should have > > set it to 0 and this is not compatible with the legacy way of > > specifying partitions directly as child nodes of the nand-controller node. > > I understand the problem, I understand the fix, but I have to say, I > strongly dislike it :) Touching an mtd core driver to fix a single > broken use case like that is... problematic, for the least. I just noticed it 2 days after this patch was backported to a stable kernel, I am just the first one to notice, we are not talking about a single use case. > I am sorry but if a 6.0 kernel breaks because: Not only kernel 6.0 is currently broken. This patch is going to be backported to any stable kernel given the fixes tag it has. > If you really want to workaround U-Boot, either you revert that patch > or you just fix the DT description instead. The parent/child/partitions > scheme has been enforced for maybe 5 years now and for a good reason: a > NAND controller with partitions does not make _any_ sense. There are > plenty of examples out there, imx7-colibri.dtsi has received many > updates since its introduction (for the best), so why not this one? I can and I will update imx7-colibri.dtsi (patch coming), but is this good enough given the kind of boot failure regression this introduce? We are going to have old u-boot around that will not work with it, and the reality is that there are tons of reasons why people do update the linux kernel and dts everyday, but never ever update the bootloader. We cannot tell "All users of the XXX kernel series must upgrade." and at the same time introduce a regression that break the boot and ignore it. Francesco _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel