From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756672AbbAZQZx (ORCPT ); Mon, 26 Jan 2015 11:25:53 -0500 Received: from mail-wi0-f175.google.com ([209.85.212.175]:51362 "EHLO mail-wi0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755578AbbAZQXQ (ORCPT ); Mon, 26 Jan 2015 11:23:16 -0500 From: Pali =?utf-8?q?Roh=C3=A1r?= To: Tony Lindgren Subject: Re: N900 v3.19-rc5 arm atags_to_fdt.c is broken Date: Mon, 26 Jan 2015 17:23:10 +0100 User-Agent: KMail/1.13.7 (Linux/3.13.0-44-generic; KDE/4.14.2; x86_64; ; ) Cc: Nicolas Pitre , "linux-omap" , Linux Kernel Mailing List , Aaro Koskinen , Sebastian Reichel , Ivaylo Dimitrov , Pavel Machek , Russell King , linux-arm-kernel@lists.infradead.org, Laura Abbott References: <201501232239.55801@pali> <201501232336.36187@pali> <20150126161454.GE28663@atomide.com> In-Reply-To: <20150126161454.GE28663@atomide.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart1746215.nYFFRZPYm0"; protocol="application/pgp-signature"; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit Message-Id: <201501261723.10505@pali> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --nextPart1746215.nYFFRZPYm0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On Monday 26 January 2015 17:14:55 Tony Lindgren wrote: > * Pali Roh=C3=A1r [150123 14:39]: > > On Friday 23 January 2015 22:39:55 Pali Roh=C3=A1r wrote: > > > Hello, > > >=20 > > > when I boot zImage with appended DT n900 in qemu > > > fdt_open_into() function called from file > > > arch/arm/boot/compressed/atags_to_fdt.c (in function > > > atags_to_fdt) always returns -FDT_ERR_NOSPACE. > > >=20 > > > It means that all ATAGS (including cmdline arguments) > > > passed by bootloader are ignored. > > >=20 > > > On real n900 device I see that booted DT version also > > > ignore cmdline arguments from bootloader. I cannot debug > > > decompress code on real device, but I think it is same > > > problem as in qemu. > >=20 > > Looks like this quick patch is fixing above problem: > >=20 > > diff --git a/arch/arm/boot/compressed/head.S > > b/arch/arm/boot/compressed/head.S index 68be901..4a7d75b > > 100644 > > --- a/arch/arm/boot/compressed/head.S > > +++ b/arch/arm/boot/compressed/head.S > > @@ -268,7 +268,7 @@ restart: adr r0, LC0 > >=20 > > * area. No GOT fixup has occurred yet, but none of=20 the > > * code we're about to call uses any global variable. > > =09 > > */ > >=20 > > - add sp, sp, #0x10000 > > + add sp, sp, #0x20000 > >=20 > > stmfd sp!, {r0-r3, ip, lr} > > mov r0, r8 > > mov r1, r6 > >=20 > > @@ -289,7 +289,7 @@ restart: adr r0, LC0 > >=20 > > bleq atags_to_fdt > > =09 > > ldmfd sp!, {r0-r3, ip, lr} > >=20 > > - sub sp, sp, #0x10000 > > + sub sp, sp, #0x20000 > >=20 > > #endif > > =20 > > mov r8, r6 @ use the appended device tree >=20 > Nico, got any ideas about this one? This seems like a > regression somewhere.. >=20 > Regards, >=20 > Tony $ du -b arch/arm/boot/dts/omap3-n900.dtb=20 70212 arch/arm/boot/dts/omap3-n900.dtb $ echo $((0x10000)) 65536 I would say, problem is because omap3-n900 binary DT is too large =2D-=20 Pali Roh=C3=A1r pali.rohar@gmail.com --nextPart1746215.nYFFRZPYm0 Content-Type: application/pgp-signature; name=signature.asc Content-Description: This is a digitally signed message part. -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iEYEABECAAYFAlTGae4ACgkQi/DJPQPkQ1KeuwCfb7Wd+CVB+Ft/jVbF2F8rS1eA ajMAnjnsCQNHqH+Gqj2uoieNgESeIaPQ =FX6k -----END PGP SIGNATURE----- --nextPart1746215.nYFFRZPYm0--