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

[Heiko Schocher <hs@denx.de>]

Hello Markus,

may I am to dummy, but I cannot find your patches in patchwork :-(

any ideas?

bye,
Heiko

Am 16.04.2019 um 05:52 schrieb Heiko Schocher:
> Hello Markus,
> 
> Am 15.04.2019 um 17:32 schrieb Markus Klotzbuecher:
>> From: Markus Klotzbuecher <markus.klotzbuecher@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