From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Tue, 07 Mar 2023 13:04:51 +0100 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 1pZW3m-00CHOU-2G for lore@lore.pengutronix.de; Tue, 07 Mar 2023 13:04:51 +0100 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 1pZW3l-0000eK-TU; Tue, 07 Mar 2023 13:04:49 +0100 Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1pZW3i-0000dU-HZ; Tue, 07 Mar 2023 13:04:46 +0100 Message-ID: <819c7790-39f7-1679-bfd1-80bd931c13df@pengutronix.de> Date: Tue, 7 Mar 2023 13:04:46 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.2 Content-Language: en-US To: Roland Hieber , distrokit@pengutronix.de References: <20230212221449.1644628-1-rhi@pengutronix.de> From: Ahmad Fatoum In-Reply-To: <20230212221449.1644628-1-rhi@pengutronix.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [DistroKit] [PATCH 0/8] v7a: add redundant update support via RAUC 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: , 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 Hello Roland, On 12.02.23 23:14, Roland Hieber wrote: > With this patch series, the first v7a platforms (beaglebone black, riotboard, > rpi3, qemu-vexpress) gain support for redundant boot on two root partitions > which can be updated with RAUC. Thanks for your patches. I don't like the prospect of having permanent patches in DistroKit to enable redundant boot. With CONFIG_EXTERNAL_DTS_FRAGMENTS, you can have barebox append device tree files specified by the BSP after the device trees built by barebox. With e.g. an #ifdef imx6s_riotboard_dts include guard, you can ensure you don't break multi-image setups. This should make any patches to barebox source unnecessary. What do you think? Cheers, Ahmad > > Roland Hieber (8): > rauc-udev: add a compatibility layer for mapping partitions > v7a: add RAUC support for qemu-vexpress > v7a: add RAUC support for beaglebone black > v7a: add RAUC support for rpi3 > v7a: add RAUC support for riotboard > v7a: kernel: enable features necessary for RAUC > rauc: add initial support > v7a: enable RAUC bundle creation > > configs/platform-mips/platformconfig | 20 ++- > configs/platform-mipsel/platformconfig | 20 ++- > configs/platform-rpi1/platformconfig | 18 ++- > .../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 + > .../barebox-mx6-defaultenv/init/bootsource | 6 +- > .../barebox-mx6-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-mx6.config | 3 +- > configs/platform-v7a/barebox-mx6.config.diff | 2 + > .../barebox-rpi2-defaultenv/init/bootsource | 5 +- > .../barebox-rpi2-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-rpi2.config | 3 +- > configs/platform-v7a/barebox-rpi2.config.diff | 2 + > .../init/bootsource | 2 +- > .../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-vexpress.config | 3 +- > .../platform-v7a/barebox-vexpress.config.diff | 2 + > .../config/images/beaglebone.config | 4 + > .../config/images/riotboard.config | 6 + > .../platform-v7a/config/images/rpi2.config | 5 + > .../config/images/vexpress.config | 7 +- > configs/platform-v7a/kernelconfig | 75 ++++++++++- > ...-bootstate-node-into-a-generic-templ.patch | 123 ++++++++++++++++++ > ...ack-add-beaglebone-black-state-entry.patch | 69 ++++++++++ > ...ts-bcm2837-rpi-3-b-add-state-support.patch | 59 +++++++++ > ...ts-imx6s-riotboard-add-state-backend.patch | 53 ++++++++ > .../patches/barebox-2022.09.0/series | 4 + > configs/platform-v7a/platformconfig | 31 ++++- > configs/platform-v7a_noneon/platformconfig | 20 ++- > configs/platform-v8a/platformconfig | 19 ++- > configs/platform-x86_64/platformconfig | 20 ++- > configs/ptxconfig | 111 ++++++++++++++-- > projectroot/etc/rauc/system.conf | 16 +++ > .../lib/systemd/system/rauc-mark-good.service | 14 ++ > projectroot/usr/lib/udev/of_base_compatible | 4 + > .../lib/udev/rules.d/90-rauc-partitions.rules | 35 +++++ > rules/rauc-udev.in | 8 ++ > rules/rauc-udev.make | 37 ++++++ > 56 files changed, 799 insertions(+), 36 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 > create mode 100644 configs/platform-v7a/barebox-mx6-defaultenv/nv/boot.default > create mode 100644 configs/platform-v7a/barebox-mx6-defaultenv/nv/bootchooser.state_prefix > create mode 100644 configs/platform-v7a/barebox-mx6-defaultenv/nv/bootchooser.system0.boot > create mode 100644 configs/platform-v7a/barebox-mx6-defaultenv/nv/bootchooser.system1.boot > create mode 100644 configs/platform-v7a/barebox-mx6-defaultenv/nv/bootchooser.targets > create mode 100644 configs/platform-v7a/barebox-rpi2-defaultenv/nv/boot.default > create mode 100644 configs/platform-v7a/barebox-rpi2-defaultenv/nv/bootchooser.state_prefix > create mode 100644 configs/platform-v7a/barebox-rpi2-defaultenv/nv/bootchooser.system0.boot > create mode 100644 configs/platform-v7a/barebox-rpi2-defaultenv/nv/bootchooser.system1.boot > create mode 100644 configs/platform-v7a/barebox-rpi2-defaultenv/nv/bootchooser.targets > create mode 100644 configs/platform-v7a/barebox-vexpress-defaultenv/nv/boot.default > create mode 100644 configs/platform-v7a/barebox-vexpress-defaultenv/nv/bootchooser.state_prefix > create mode 100644 configs/platform-v7a/barebox-vexpress-defaultenv/nv/bootchooser.system0.boot > create mode 100644 configs/platform-v7a/barebox-vexpress-defaultenv/nv/bootchooser.system1.boot > create mode 100644 configs/platform-v7a/barebox-vexpress-defaultenv/nv/bootchooser.targets > create mode 100644 configs/platform-v7a/patches/barebox-2022.09.0/0101-arm-dts-refactor-bootstate-node-into-a-generic-templ.patch > create mode 100644 configs/platform-v7a/patches/barebox-2022.09.0/0102-arm-dts-boneblack-add-beaglebone-black-state-entry.patch > create mode 100644 configs/platform-v7a/patches/barebox-2022.09.0/0103-arm-dts-bcm2837-rpi-3-b-add-state-support.patch > create mode 100644 configs/platform-v7a/patches/barebox-2022.09.0/0104-arm-dts-imx6s-riotboard-add-state-backend.patch > create mode 100644 projectroot/etc/rauc/system.conf > create mode 100644 projectroot/usr/lib/systemd/system/rauc-mark-good.service > create mode 100755 projectroot/usr/lib/udev/of_base_compatible > create mode 100644 projectroot/usr/lib/udev/rules.d/90-rauc-partitions.rules > create mode 100644 rules/rauc-udev.in > create mode 100644 rules/rauc-udev.make > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |