From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 17 Feb 2025 20:26:02 +0100 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1tk6kl-003bWE-1e for lore@lore.pengutronix.de; Mon, 17 Feb 2025 20:26:02 +0100 Received: from bombadil.infradead.org ([2607:7c80:54:3::133]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1tk6kk-00058k-8V for lore@pengutronix.de; Mon, 17 Feb 2025 20:26:02 +0100 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ZykbrJCQ3oMHb8+4Oi7AQpaP63BD7DymMT1fg8FVVDQ=; b=1EX6l1P/ddkTvJiym9qNzrTXVD NyV5RKLTJSbmSjP4SBXRpITDF7uyCOSxphoZGDSNmVEuZPD4/620203y37ivKSJRPh/qLlz7fyBNJ xZZTz636h8ZSUXa6veDT9NKx42qN1B5vysT1aekG/Sh8XkWrrlN8wAiKmUHZPiDOf609+aEqTWoFl UObCjY9vgAfgHQGKpkpSRr6aNZds5/ZgnZD9gYZlzGlO7RM36sQQUREYoxe/jtythPCqA4BFl+6DE pcgtQsH3IEhxUWW2I1rXwQPLM+DTfuIWY74F8jgzMMA412VFdaWE96UjItCkCr0vxO94a2y0ZmJ9E 5yAyAmtg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tk6kB-00000005kDa-1VpZ; Mon, 17 Feb 2025 19:25:27 +0000 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tk6k8-00000005kCb-43wX for barebox@lists.infradead.org; Mon, 17 Feb 2025 19:25:26 +0000 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1tk6k7-0004xB-Kq; Mon, 17 Feb 2025 20:25:23 +0100 Received: from pty.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::c5]) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1tk6k7-001Skq-1U; Mon, 17 Feb 2025 20:25:23 +0100 Received: from mfe by pty.whiteo.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1tk6k7-002dFw-1A; Mon, 17 Feb 2025 20:25:23 +0100 Date: Mon, 17 Feb 2025 20:25:23 +0100 From: Marco Felsch To: Ahmad Fatoum Cc: barebox@lists.infradead.org Message-ID: <20250217192523.crgvcg2phxhce46d@pengutronix.de> References: <20250217180833.3955657-1-a.fatoum@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250217180833.3955657-1-a.fatoum@pengutronix.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250217_112525_008256_80969B4C X-CRM114-Status: GOOD ( 20.53 ) X-BeenThere: barebox@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "barebox" X-SA-Exim-Connect-IP: 2607:7c80:54:3::133 X-SA-Exim-Mail-From: barebox-bounces+lore=pengutronix.de@lists.infradead.org X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.whiteo.stw.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-5.2 required=4.0 tests=AWL,BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.2 Subject: Re: [PATCH 0/6] of: partition: add Linux CONFIG_OF_PARTITION-compatible adaptive fixup mode X-SA-Exim-Version: 4.2.1 (built Wed, 08 May 2019 21:11:16 +0000) X-SA-Exim-Scanned: Yes (on metis.whiteo.stw.pengutronix.de) Hi Ahmad, thanks for working on this! On 25-02-17, Ahmad Fatoum wrote: > barebox has for many years supported a superset of the upstream OF > partitions binding by parsing partitions out of SD/eMMC and EEPROM > in addition to MTD. > > Unfortunately, we didn't upstream our binding and a different binding > went upstream into Linux v6.13-rc1 along with Linux commit > 2e3a191e89f9 ("block: add support for partition table defined in OF"). > > Since then, if CONFIG_OF_PARTITION, which is disabled by default, is > enabled, Linux will ignore any GPT/MBR if an SD/eMMC node has a > fixed-partitions compatible subnode. > > This is different from the barebox behavior, which allows both OF > partitions and GPT/MBR to co-exist as long as they don't conflict. > > As many barebox boards: > > - Place the environment in non-partitioned space prior to the first > partition > > - Fixup all fixed-partitions into the kernel device tree by default Could we add a warning/notice that the board should be converted to make use of the barebox GPT parttype-uuid? It's 2025 and IMHO there is no reason for not using the GPT with the barebox state part-type UUID in place. This is of course not related to this patchset. Regards, Marco > this can result in breakage if a newer kernel is booted with > CONFIG_OF_PARTITION enabled. > > To resolve this, this series introduces the barebox,fixed-partitions > compatible and uses that in Linux device tree fixups for SD/eMMC > by default. > > Ahmad Fatoum (6): > of: factor out of_node_is_fixed_partitions check > of: of_path: support barebox,fixed-partitions > ARM: IMX8MP: var-dart-dt8mcustomboard.dts: use new-style partition > binding > ARM: dts: prefix all non-MTD fixed-partitions with barebox, > of: partition: refactor of_partition_binding checks into switch > of: partition: add Linux CONFIG_OF_PARTITION-compatible adaptive fixup > mode > > .../devicetree/bindings/mtd/partition.rst | 27 +++++++- > Documentation/user/state.rst | 4 +- > arch/arm/dts/am335x-phytec-state.dtsi | 2 +- > .../dts/fsl-ls1046a-tqmls1046a-mbls10xxa.dts | 2 +- > arch/arm/dts/imx50-kindle-common.dtsi | 2 +- > arch/arm/dts/imx51-zii-rdu1.dts | 2 +- > arch/arm/dts/imx6dl-colibri-iris.dts | 2 +- > arch/arm/dts/imx6qdl-phytec-mira.dtsi | 2 +- > arch/arm/dts/imx6qdl-phytec-phycard-som.dtsi | 2 +- > arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi | 2 +- > arch/arm/dts/imx6qdl-phytec-state.dtsi | 2 +- > arch/arm/dts/imx6qdl-zii-rdu2.dtsi | 2 +- > arch/arm/dts/imx6ul-liteboard.dts | 4 +- > arch/arm/dts/imx6ul-phytec-state.dtsi | 2 +- > arch/arm/dts/imx6ul-tqma6ul-common.dtsi | 4 +- > arch/arm/dts/imx6ul-webasto-ccbv2.dts | 2 +- > arch/arm/dts/imx6ul-webasto-marvel.dts | 2 +- > arch/arm/dts/imx7d-ac-sxb.dtsi | 4 +- > arch/arm/dts/imx7d-flex-concentrator-mfg.dts | 2 +- > arch/arm/dts/imx7d-gome-e143_01.dts | 2 +- > arch/arm/dts/imx7d-phyboard-zeta.dts | 2 +- > arch/arm/dts/imx7s-warp.dts | 2 +- > arch/arm/dts/imx8mm-innocomm-wb15-evk.dts | 4 +- > arch/arm/dts/imx8mm-phyboard-polis-rdk.dts | 4 +- > arch/arm/dts/imx8mp-debix-model-a.dts | 4 +- > arch/arm/dts/imx8mp-debix-som-a-bmb-08.dts | 4 +- > arch/arm/dts/imx8mp-karo-qsxp-ml81.dtsi | 2 +- > .../dts/imx8mp-var-dart-dt8mcustomboard.dts | 25 ++++---- > arch/arm/dts/rk3566-cm3-io.dts | 4 +- > arch/arm/dts/rk3566-pinetab2.dtsi | 4 +- > arch/arm/dts/rk3568-bpi-r2-pro.dts | 4 +- > arch/arm/dts/rk3568-evb1-v10.dts | 4 +- > arch/arm/dts/rk3568-mecsbc.dts | 4 +- > arch/arm/dts/rk3568-rock-3a.dts | 4 +- > arch/arm/dts/rk3568-wolfvision-pf5.dts | 2 +- > arch/arm/dts/rk3588-orangepi-5-plus.dts | 4 +- > arch/arm/dts/rk3588-rock-5b.dts | 4 +- > arch/arm/dts/socfpga_arria10_achilles.dts | 2 +- > arch/arm/dts/socfpga_arria10_mercury_aa1.dts | 2 +- > arch/arm/dts/state-example.dtsi | 4 +- > drivers/of/base.c | 6 +- > drivers/of/of_path.c | 9 ++- > drivers/of/partition.c | 62 ++++++++++++------- > include/of.h | 6 ++ > 44 files changed, 149 insertions(+), 96 deletions(-) > > -- > 2.39.5 > > >