[U-Boot] [PATCH 0/2] Android and arm64 improvement

[Tom Rini]

On Tue, Feb 06, 2018 at 10:31:10AM +1100, Bin Chen wrote:

> Hello Tom,
> 
> Any feedback on this?

It's on my list to grab soon, thanks!

> 
> On 27 January 2018 at 16:59, Bin Chen <bin.chen at linaro.org> wrote:
> 
> >
> > I rebased the two patches submitted (quite )a while ago on top of
> > current U-boot master fb4413295c765aa8c013650984dc2d908964c81d
> > (and there were no conflicts).
> >
> > The first patch added the support of parsing the second area of Android
> > image so
> > that it can be used for good. On Poplar board, we are using that area for
> > dtb.
> > The first patch shouldn't impact any functionality since
> > it is new.
> >
> > The second patch moves the booti_setup out of booti.c. It was the result
> > of a
> > discusion[1] how to better support the arm64 Android image.
> >
> > I have boot tested it on Poplar board[2] with additional patch[3] that is
> > built on top of the booti. I have hacked version of using bootm but it
> > seems quit
> > mess up - need lots of changes in different places ("Shotgun Surgery") to
> > handle
> > Android specific and/or arm64 specific code. Now, I feel a separate
> > bootandroid command might be a cleaner solution.
> >
> > That being said, the second patch doesn't depend on or impact any change we
> > might have in the future for better Android support.
> >
> > [1] https://lists.denx.de/pipermail/u-boot/2017-July/297933.html
> > [2] https://github.com/96boards-poplar/Documentation
> > [3] https://github.com/pierrchen/u-boot/commit/
> > 8463e6177654026746161487d0f0bd8998bb7a5b
> >
> > Bin Chen (2):
> >   parse the second area of android image
> >   move booti_setup to arch/arm/lig/image.c
> >
> >  arch/arm/lib/Makefile  |  2 +-
> >  arch/arm/lib/image.c   | 76 +++++++++++++++++++++++++++++++++++++++++
> >  cmd/booti.c            | 92 +++++++-----------------------
> > --------------------
> >  common/image-android.c | 19 +++++++++++
> >  include/image.h        | 11 ++++++
> >  5 files changed, 120 insertions(+), 80 deletions(-)
> >  create mode 100644 arch/arm/lib/image.c
> >
> > --
> > 2.15.0
> >
> >
> 
> 
> -- 
> Regards,
> Bin
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180205/62ea4eb4/attachment.sig>