[U-Boot] [PATCH] common/image.c: Make boot_get_ramdisk() perform a check for Android images

[Paul Kocialkowski]

Hi,

Le lundi 05 octobre 2015 à 14:23 -0500, Rob Herring a écrit :
> On Tue, Sep 1, 2015 at 8:50 AM, Paul Kocialkowski <contact at paulk.fr>
> wrote:
> > Le jeudi 27 août 2015 à 15:42 -0400, Tom Rini a écrit :
> > > In 2dd4632 the check for where a ramdisk is found on an Android
> > > image
> > > was got moved into the "normal" loop here, causing people to have
> > > to
> > > pass the kernel address in the ramdisk address location in order
> > > to have
> > > Android boot still.  This changed previous behavior so perform a
> > > check
> > > early in the function to see if we have an Android image and if
> > > so use
> > > that as where to look for the ramdisk (which is what the rest of
> > > the
> > > code here expects).
> > 
> > That patch does fix my problem (the ramdisk is now correctly passed
> > to
> > the kernel). I suggest that you merge it ASAP.
> 
> Doesn't look like this was ever merged or respun. I had some
> comments,
> but have no issue if they addressed separately as part of some
> refactoring.

Well, I would like to see this getting merged soon, because Android
booting on sniper (LG Optimus Black) is currently broken.

Tom, would you consider picking it up for the current rc round?
I realize I had forgotten about this patch and should have suggested it
earlier on in the cycle.

Either way, I should test the current rc on my device and report back
if there is anything else going wrong.

-- 
Paul Kocialkowski, Replicant developer

Replicant is a fully free Android distribution running on several
devices, a free software mobile operating system putting the emphasis
on
freedom and privacy/security.

Website: https://www.replicant.us/
Blog: https://blog.replicant.us/
Wiki/tracker/forums: https://redmine.replicant.us/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20151007/d99b0234/attachment.sig>