[U-Boot] [PATCH v2 0/4] MTD & UBI fixes

Scott Wood
2013-09-12T00:55:32Z

On Wed, 2013-09-11 at 07:48 +0200, Stefan Roese wrote:
> On 10.09.2013 22:24, Scott Wood wrote:
> > On Sat, 2013-09-07 at 14:11 +0200, Stefan Roese wrote:
> >> On 04.09.2013 16:16, Paul Burton wrote:
> >>> This patchset corrects a few issues I've had whilst using UBI with U-boot.
> >>>
> >>> The first 3 are bug fixes, the 4th is an addition I needed in order to write a
> >>> large root filesystem into my NAND device.
> >>>
> >>> Changes since v1:
> >>> - Fixed style issues in "cmd_ubi: add write.part command..." as per
> >>> Stefan Roese's comments.
> >>> - Expanded upon the condition patch 1 fixes in response to the
> >>> queries from Stefan, see the commit message for further detail.
> >>> - Added patch 3 "cmd_ubi: use int64_t volume size for 'ubi create'"
> >>> which it seems appropriate to include in this series.
> >>>
> >>> Paul Burton (4):
> >>> mtd: driver _read() returns max_bitflips; mtd_read() returns -EUCLEAN
> >>> cmd_mtdparts: use 64 bits for flash size, partition size & offset
> >>> cmd_ubi: use int64_t volume size for 'ubi create'
> >>> cmd_ubi: add write.part command, to write a volume in multiple parts
> >>
> >> All 4 patches:
> >>
> >> Acked-by: Stefan Roese <sr@denx.de>
> >>
> >> Scott, these patches are assigned to you in patchwork. If you are okay
> >> with the NAND side (and everything else), then please feel free to push
> >> all 4 patches via your repository.
> >
> > Patch 1 looks OK to me (at least to the extent that it's just bringing
> > code over from Linux). Patch 2 also looks straightforward.
> >
> > Kyungmin is the UBI maintainer -- the last two patches need his ack.
>
> Yes, Kyungmin is the "official" UBI maintainer. But I have been taking
> care of all UBI (and UBIFS) related patches in the last years
> (collecting and pushing them upstream in the u-boot-ubi git repository).
>
> Kyungmin, how should we handle this in the future. Would you like to
> handle the UBI (and UBIFS) patches yourself? Please let us know how to
> proceed.

OK, I wasn't aware of that (maybe you should be listed as
(co-)custodian?). I guess I can take them, though just in case you'd
rather do so (I wasn't the one who assigned them in patchwork):

Acked-by: Scott Wood <scottwood@freescale.com>

-Scott