[U-Boot] [PATCH 0/6] miscellaneous ubispl and ubi improvements

[Heiko Schocher]

Hello Markus,

Am 15.04.2019 um 17:32 schrieb Markus Klotzbuecher:
> From: Markus Klotzbuecher <markus.klotzbuecher at kistler.com>
> 
> Hello Heiko and Kyungmin
> 
> This series contains a couple of UBI and UBI SPL improvements, notably
> a ubispl extension to allow loading volumes by name. The motivation is
> to use the UBI atomic volume rename functionality to allow double copy
> software updates of U-Boot on UBI. To do that we configured the SPL to
> always load the same volume name (e.g. "u-boot"), and the software
> updater always installs into the secondary volume "u-boot_r". After a
> successful upgrade, these two volume names are switched.

Ok ... Hmm... if you use the bootcounter in SPL, you may have a chance
to unbreak he board, if "u-boot" does not boot?

How do you detect a successfull upgrade ?

> This extension is protected by #ifdefs as it will somewhat slow down
> loading of volumes by id. This is because the code needs to disable
> the optimization of ignoring all volume ids which are not
> to-be-loaded, since these can only be resolved after attaching.
> 
> We have tested both with and without fastmap enabled and both paths
> seems to work reliably.

Fine, thanks!

> I also moved the ubispl config to Kconfig. There seem to be two boards

Thanks!

> that use this (Maintainers cc'ed on patch). I have not updated their
> defconfigs (not being sure if I should) but would be pleased to do so.

I prefer that you do so, but we will see, if the board maintainers
give some feedback.

Give me some time to look through your patches.

bye,
Heiko

> Hamish Guthrie (2):
>    env: ubi: support configurable VID offset
>    ubispl: add support for loading volumes by name
> 
> Markus Klotzbuecher (4):
>    env: ubi: KConfig: add CONFIG_ENV_UBI_VOLUME_REDUND
>    ubi: fix UBI_SILENCE_MSG
>    ubispl: migrate configuration to Kconfig
>    ubispl: introduce separate CONFIG_UBI_SPL_SILENCE_MSG
> 
>   common/spl/Kconfig           |  98 ++++++++++++++++
>   common/spl/spl_ubi.c         |   7 ++
>   drivers/mtd/ubi/Kconfig      |   2 +-
>   drivers/mtd/ubi/io.c         |   2 +
>   drivers/mtd/ubispl/ubispl.c  | 215 ++++++++++++++++++++++++++++++++++-
>   drivers/mtd/ubispl/ubispl.h  |   9 +-
>   env/Kconfig                  |  12 ++
>   env/ubi.c                    |  17 ++-
>   include/ubispl.h             |   6 +
>   scripts/config_whitelist.txt |  13 ---
>   10 files changed, 360 insertions(+), 21 deletions(-)
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de