[sodaville] [PATCH 3/4] of/dtc: force dtb size to modulo 32 bytes

H. Peter Anvin hpa at linux.intel.com
Fri Nov 12 11:47:31 EST 2010


On 11/11/2010 04:03 PM, dirk.brandewie at gmail.com wrote:
> From: Dirk Brandewie <dirk.brandewie at gmail.com>
> 
> This patch forces the size of the DTB to be modulo 32 bytes. This is
> needed to support linking multiple DTB's into a single section in the
> image. GCC wants structures to be 32 byte aligned without this change
> DTB's after the first in the section may not be properly aligned so
> the flat tree parsing code will fall over.
> 
> Signed-off-by: Dirk Brandewie <dirk.brandewie at gmail.com>

I don't think 32 is a universal number; it should depend on the ABI.  On
x86, for one thing, I'm pretty sure that there is no particular
alignment requirements beyond the natural alignment of the data items,
for example.

Although 32 is probably conservative on any platform, please flag the
origin of this with a comment, or make it a constant defined in a header
file... otherwise, if this breaks for whatever reason it'll be near
impossible to find.

	-hpa


More information about the devicetree-discuss mailing list