[U-Boot] [PATCH 1/2] ARM: bootm: allow skipping fdt memory node fixup

[Tom Rini]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/20/2013 04:55 PM, Peter Korsgaard wrote:
>>>>>> "Tom" == Tom Rini <trini at ti.com> writes:
> 
> Hi,
> 
> Tom> It's not so simple.  The first DTs I checked are the am335x
> ones Tom> and I see they hardcode to 256MB which isn't right, but
> no one has Tom> caught this since it's blindly fixed up.  So as
> Wolfgang suggests Tom> next, we might need to go with replacing on
> 0x0 entries and Tom> warning when they don't match what U-Boot has
> detected.
> 
> It is correct for the (classic) beaglebone. This also hits people
> using appended dtb unless they have CONFIG_ARM_ATAG_DTB_COMPAT
> enabled (which the probably do though).

So yes, there are some cases where 256MB is the right value.  But we
always know dynamically how much we have, so we don't need to
hard-code even in that case.  And once we do change the behaviour, the
appeneded tree hack shouldn't be relevant anyhow :)

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRJUqnAAoJENk4IS6UOR1WAmgQALOHeh+p9aRLT+Hb0ZjpJk0c
XGJzCO1EpdNpQBsgPxI2AzhPEHsVHgcdzXmzljHS4ukPWDbqihM4j7o9OHjuxAGY
+8Yebw/E/1F7+wKXy013/3QFbl9mdd1OpTU95xarnaoe4+dP5pfa+Q/kH5mrc0U2
xiA4kDmInityA/DUuEP4d8mhnLQyn11CKS/LPKdT40HUGDk6ffzkmUCYEHBpwk11
QN7w6icIGE9JrMsQUOPir7CNDZsNCbeAtB8+gpStHxL63cHqxAh9R3yFyUc2uVHo
5NNkc71R0qLIrKMTbDjIfm1soTEWP94ob6VVPQafqSC7wQrK5r+2DNg9I7dN7xIr
js5Y/BBbPuUZfMWiNWiyF+ONuoNNrz1+iLm9iLdaJajf5EuBQN/qUzNGKFTCsLne
nR6cR0x/Ow+rN1TDmnSvmS845XUDK2qLY5nSthKieuFtoDY88j6VpR4ZO7+B6sSa
9MVx6LV8SO4USewl1FIpXoe6KC2AjdSK2rJN5i2SSQD6uxYzPCfBKezG7gi8Mfua
v4M9W/jDU/avXo861XLeKgcZx0zA7PHxDhqgZDVbFs3QH3eobzZHGog001vM6mvr
dEdUGxu0dVF9ocbF/2F6XxdMVxt2CK7ddbGCZy5V1R6ZCvOVZetCbQao8GB7ZqLU
LMZf4b3iDGdrOfM57n5m
=8tud
-----END PGP SIGNATURE-----