From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 14 Aug 2023 18:39:04 +0200 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1qVaau-003zEf-Oe for lore@lore.pengutronix.de; Mon, 14 Aug 2023 18:39:04 +0200 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1qVaas-0007qs-VV; Mon, 14 Aug 2023 18:39:02 +0200 Received: from pty.hi.pengutronix.de ([2001:67c:670:100:1d::c5]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qVaYu-0007R3-8E; Mon, 14 Aug 2023 18:37:00 +0200 Received: from rsc by pty.hi.pengutronix.de with local (Exim 4.94.2) (envelope-from ) id 1qVaYt-00CQky-Qk; Mon, 14 Aug 2023 18:36:59 +0200 Date: Mon, 14 Aug 2023 18:36:59 +0200 From: Robert Schwebel To: Roland Hieber Message-ID: References: <20230623124755.2292833-1-rhi@pengutronix.de> <20230623124755.2292833-7-rhi@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230623124755.2292833-7-rhi@pengutronix.de> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Mailman-Approved-At: Mon, 14 Aug 2023 18:39:01 +0200 Subject: Re: [DistroKit] [PATCH 06/14] v7a: add RAUC support for beaglebone black X-BeenThere: distrokit@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: DistroKit Mailinglist List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: distrokit@pengutronix.de Sender: "DistroKit" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: distrokit-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false Applied to next. On Fri, Jun 23, 2023 at 02:47:47PM +0200, Roland Hieber wrote: > * Enable barebox bootchooser framework > * Add a redundant root partition to the genimage config > * Add appropriate variables to the barebox defaultenv so the bootchooser > can decide into which partition to boot > * Add a state backend to the device tree fragment > * Map partitions in rauc-udev compatibility layer > > Signed-off-by: Roland Hieber > --- > .../barebox-am335x-defaultenv/init/bootsource | 4 +- > .../barebox-am335x-defaultenv/nv/boot.default | 1 + > .../nv/bootchooser.state_prefix | 1 + > .../nv/bootchooser.system0.boot | 1 + > .../nv/bootchooser.system1.boot | 1 + > .../nv/bootchooser.targets | 1 + > configs/platform-v7a/barebox-am335x.config | 3 +- > .../platform-v7a/barebox-am335x.config.diff | 2 + > .../config/images/beaglebone.config | 4 ++ > configs/platform-v7a/dts/bootstate.dtsi | 41 +++++++++++++++++++ > .../lib/udev/rules.d/90-rauc-partitions.rules | 7 ++++ > 11 files changed, 63 insertions(+), 3 deletions(-) > create mode 100644 configs/platform-v7a/barebox-am335x-defaultenv/nv/boot.default > create mode 100644 configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.state_prefix > create mode 100644 configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system0.boot > create mode 100644 configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system1.boot > create mode 100644 configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.targets > > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/init/bootsource b/configs/platform-v7a/barebox-am335x-defaultenv/init/bootsource > index 72e71e08c857..b3b73e790334 100644 > --- a/configs/platform-v7a/barebox-am335x-defaultenv/init/bootsource > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/init/bootsource > @@ -5,7 +5,7 @@ if [ -n "$nv.boot.default" ]; then > fi > > if [ $bootsource = mmc ]; then > - global.boot.default="mmc$bootsource_instance net" > + global.boot.default="bootchooser net" > else > - global.boot.default="net mmc0 mmc1" > + global.boot.default="net bootchooser" > fi > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/boot.default b/configs/platform-v7a/barebox-am335x-defaultenv/nv/boot.default > new file mode 100644 > index 000000000000..d7d9a373c50b > --- /dev/null > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/boot.default > @@ -0,0 +1 @@ > +bootchooser net > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.state_prefix b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.state_prefix > new file mode 100644 > index 000000000000..6246412a5c27 > --- /dev/null > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.state_prefix > @@ -0,0 +1 @@ > +state.bootstate > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system0.boot b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system0.boot > new file mode 100644 > index 000000000000..a6746c31d982 > --- /dev/null > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system0.boot > @@ -0,0 +1 @@ > +mmc0.1 > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system1.boot b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system1.boot > new file mode 100644 > index 000000000000..ecdf1ba55ad6 > --- /dev/null > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.system1.boot > @@ -0,0 +1 @@ > +mmc0.2 > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.targets b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.targets > new file mode 100644 > index 000000000000..f0fb14eeaad2 > --- /dev/null > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/bootchooser.targets > @@ -0,0 +1 @@ > +system0 system1 > diff --git a/configs/platform-v7a/barebox-am335x.config b/configs/platform-v7a/barebox-am335x.config > index d91181f4dfe2..30a58497bc95 100644 > --- a/configs/platform-v7a/barebox-am335x.config > +++ b/configs/platform-v7a/barebox-am335x.config > @@ -211,7 +211,7 @@ CONFIG_BTHREAD=y > CONFIG_STATE=y > # CONFIG_STATE_CRYPTO is not set > # CONFIG_STATE_BACKWARD_COMPATIBLE is not set > -# CONFIG_BOOTCHOOSER is not set > +CONFIG_BOOTCHOOSER=y > CONFIG_RESET_SOURCE=y > # CONFIG_MACHINE_ID is not set > # CONFIG_SYSTEMD_OF_WATCHDOG is not set > @@ -301,6 +301,7 @@ CONFIG_CMD_GO=y > # CONFIG_CMD_LOADY is not set > CONFIG_CMD_RESET=y > CONFIG_CMD_UIMAGE=y > +CONFIG_CMD_BOOTCHOOSER=y > # end of Boot > > # > diff --git a/configs/platform-v7a/barebox-am335x.config.diff b/configs/platform-v7a/barebox-am335x.config.diff > index d4ef2e860be7..81e857d116e0 100644 > --- a/configs/platform-v7a/barebox-am335x.config.diff > +++ b/configs/platform-v7a/barebox-am335x.config.diff > @@ -15,11 +15,13 @@ CONFIG_ARM_BOARD_APPEND_ATAG=y > # CONFIG_BAREBOX_UPDATE_AM33XX_EMMC is not set > # CONFIG_BAREBOX_UPDATE_AM33XX_NAND is not set > CONFIG_BAREBOX_UPDATE_AM33XX_SPI_NOR_MLO=y > +CONFIG_BOOTCHOOSER=y > CONFIG_BOOTM_OFTREE_UIMAGE=y > CONFIG_BUS_OMAP_GPMC=y > # CONFIG_CLKDEV_LOOKUP is undefined > # CONFIG_CLOCKSOURCE_BCM283X is undefined > CONFIG_CLOCKSOURCE_TI_DM=y > +CONFIG_CMD_BOOTCHOOSER=y > # CONFIG_CMD_CLK is undefined > # CONFIG_CMD_KEYSTORE is not set > CONFIG_CMD_NAND=y > diff --git a/configs/platform-v7a/config/images/beaglebone.config b/configs/platform-v7a/config/images/beaglebone.config > index 7b1c41dfc320..07d251bfbe69 100644 > --- a/configs/platform-v7a/config/images/beaglebone.config > +++ b/configs/platform-v7a/config/images/beaglebone.config > @@ -20,6 +20,10 @@ image @IMAGE@ { > image = root.ext2 > partition-type = 0x83 > } > + partition root-B { > + image = root.ext2 > + partition-type = 0x83 > + } > partition data { > partition-type = 0x83 > size = 512M > diff --git a/configs/platform-v7a/dts/bootstate.dtsi b/configs/platform-v7a/dts/bootstate.dtsi > index d41c888dbd43..845950712e37 100644 > --- a/configs/platform-v7a/dts/bootstate.dtsi > +++ b/configs/platform-v7a/dts/bootstate.dtsi > @@ -1,3 +1,44 @@ > +/** TI AM335x BeagleBone black ************************************************/ > +#ifdef am335x_boneblack_dts > +/ { > + aliases { > + state = &state_mmc1; > + }; > + > + state_mmc1: state { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "barebox,state"; > + magic = <0xa9cc3f17>; > + backend-type = "raw"; > + backend = <&backend_state_mmc1>; > + backend-storage-type = "direct"; > + backend-stridesize = <0x40>; > + }; > +}; > + > +&mmc1 { > + partitions { > + compatible = "fixed-partitions"; > + #address-cells = <2>; > + #size-cells = <2>; > + > + barebox: partition@0 { > + label = "barebox"; > + reg = <0x0 0x0 0x0 0xc000>; > + }; > + environment: partition@c0000 { > + label = "barebox-environment"; > + reg = <0x0 0xc0000 0x0 0x20000>; > + }; > + backend_state_mmc1: partition@e0000 { > + label = "state"; > + reg = <0x0 0xe0000 0x0 0x20000>; > + }; > + }; > +}; > +#endif > + > /** Generic bootstate node for all platforms **********************************/ > / { > state: state { > diff --git a/projectroot/usr/lib/udev/rules.d/90-rauc-partitions.rules b/projectroot/usr/lib/udev/rules.d/90-rauc-partitions.rules > index 64d30fb96c27..b5c75d1ece7d 100644 > --- a/projectroot/usr/lib/udev/rules.d/90-rauc-partitions.rules > +++ b/projectroot/usr/lib/udev/rules.d/90-rauc-partitions.rules > @@ -15,6 +15,13 @@ KERNEL=="mmcblk0p3", SYMLINK+="disk/by-usage/data" > GOTO="rauc_partitions_end" > LABEL="qemu_vexpress_end" > > +ENV{OF_BASE_COMPATIBLE}!="*ti,am335x-bone-black*", GOTO="beaglebone_black_end" > +KERNEL=="mmcblk0p2", SYMLINK+="disk/by-usage/rootfs0" > +KERNEL=="mmcblk0p3", SYMLINK+="disk/by-usage/rootfs1" > +KERNEL=="mmcblk0p4", SYMLINK+="disk/by-usage/data" > +GOTO="rauc_partitions_end" > +LABEL="beaglebone_black_end" > + > # fallback for boards not yet supported by RAUC > KERNEL=="mmcblk0p3", SYMLINK+="disk/by-usage/data" > > -- > 2.39.2 > > > -- Pengutronix e.K. | Dipl.-Ing. Robert Schwebel | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 |