[U-Boot] [PATCH v2 2/5] fastboot: sparse: resync common/image-sparse.c (part 1)

Steve Rae srae at broadcom.com
Thu Jun 16 19:29:39 CEST 2016


On Wed, Jun 15, 2016 at 1:18 AM, Maxime Ripard
<maxime.ripard at free-electrons.com> wrote:
>
> On Tue, Jun 07, 2016 at 11:19:36AM -0700, Steve Rae wrote:
> > This file originally came from upstream code.
> >
> > While retaining the storage abstraction feature, this is the first
> > set of the changes required to resync with the
> >   cmd_flash_mmc_sparse_img()
> > in the file
> >   aboot.c
> > from
> >   https://us.codeaurora.org/cgit/quic/la/kernel/lk/plain/app/aboot/aboot.c?h=LE.BR.1.2.1
> >
> > Signed-off-by: Steve Rae <srae at broadcom.com>
>
> Again, please split that in several patches to have one patch
> per-change you're doing.
>
> This is just impossible to review.

And I think you just reinforced the point:
   this code was so far away from the original upstream code that it
is not even recognizable anymore....
Furthermore, I have attempted to split this into several commits --
however, I kept breaking bi-sect so I abandoned that idea.
Instead, I extensively reviewed:
  git log --follow -p -- common/image-sparse.c
to determine which changes needed to be retained in U-Boot.
This series, therefore, provides working code that is resync'ed with
the upstream code, and becomes a platform for planning any future
changes.
Thanks, Steve

>
> Maxime
>
> --
> Maxime Ripard, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com


More information about the U-Boot mailing list