DistroKit Mailinglist
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: Roland Hieber <rhi@pengutronix.de>, distrokit@pengutronix.de
Subject: Re: [DistroKit] [PATCH 0/8] v7a: add redundant update support via RAUC
Date: Tue, 7 Mar 2023 13:04:46 +0100	[thread overview]
Message-ID: <819c7790-39f7-1679-bfd1-80bd931c13df@pengutronix.de> (raw)
In-Reply-To: <20230212221449.1644628-1-rhi@pengutronix.de>

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 |




  parent reply	other threads:[~2023-03-07 12:04 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-12 22:14 Roland Hieber
2023-02-12 22:14 ` [DistroKit] [PATCH 1/8] rauc-udev: add a compatibility layer for mapping partitions Roland Hieber
2023-02-13  9:43   ` Michael Olbrich
2023-03-02  9:33     ` Roland Hieber
2023-02-12 22:14 ` [DistroKit] [PATCH 2/8] v7a: add RAUC support for qemu-vexpress Roland Hieber
2023-03-08  9:33   ` Roland Hieber
2023-02-12 22:14 ` [DistroKit] [PATCH 3/8] v7a: add RAUC support for beaglebone black Roland Hieber
2023-02-12 22:14 ` [DistroKit] [PATCH 4/8] v7a: add RAUC support for rpi3 Roland Hieber
2023-02-12 22:14 ` [DistroKit] [PATCH 5/8] v7a: add RAUC support for riotboard Roland Hieber
2023-02-12 22:14 ` [DistroKit] [PATCH 6/8] v7a: kernel: enable features necessary for RAUC Roland Hieber
2023-02-13  7:33   ` Jan Lübbe
2023-02-12 22:14 ` [DistroKit] [PATCH 7/8] rauc: add initial support Roland Hieber
2023-02-13  7:37   ` Jan Lübbe
2023-03-02  9:41     ` Roland Hieber
2023-03-02 10:34       ` Jan Lübbe
2023-03-02 16:09         ` Michael Olbrich
2023-03-08  9:45           ` Roland Hieber
2023-02-12 22:14 ` [DistroKit] [PATCH 8/8] v7a: enable RAUC bundle creation Roland Hieber
2023-02-13  7:42   ` Jan Lübbe
2023-04-25 18:38     ` Roland Hieber
2023-02-13  7:48 ` [DistroKit] [PATCH 0/8] v7a: add redundant update support via RAUC Marco Felsch
2023-02-13  9:35   ` Roland Hieber
2023-03-07 12:04 ` Ahmad Fatoum [this message]
2023-03-08 10:11   ` Roland Hieber

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=819c7790-39f7-1679-bfd1-80bd931c13df@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=distrokit@pengutronix.de \
    --cc=rhi@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox