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

[Paul Kocialkowski]

Hi,

Le mercredi 07 octobre 2015 à 20:07 +0200, Paul Kocialkowski a écrit :
> 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.

This is about the only problem I have on the LG Optimus Black P970
(codename sniper) with the current git master, so merging this patch
makes the device usable.

-- 
Paul Kocialkowski 

* Site web : http://www.paulk.fr/
* Blog : http://blog.paulk.fr/
* Dev blog : http://code.paulk.fr/
-------------- 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/20151010/929ad834/attachment.sig>