Thread (3 messages) 3 messages, 2 authors, 2012-10-30

Re: [PATCH] libfdt: introduce fdt type annotation for use by endian checkers

From: Stephen Warren <hidden>
Date: 2012-10-30 22:24:05
Also in: u-boot

Possibly related (same subject, not in this thread)

On 10/30/2012 03:57 PM, Kim Phillips wrote:
Projects such as linux and u-boot run sparse on libfdt.  libfdt
contains the notion of endianness via usage of endian conversion
functions such as fdt32_to_cpu.  As such, in order to pass endian
checks, libfdt has to annotate its fdt variables as big endian.
This patch does that ifdef __CHECKER__ (a symbol sparse defines),
for two new fdt types: fdt32_t and fdt64_t, and subsequently
silences warnings emitted by sparse when parsing libfdt.
Should libfdt patches be committed to the main dtc repository (which I
assume is also upstream for libfdt?) rather than U-Boot first?
Otherwise, if we want to bring in a new libfdt from upstream, that would
trash all the U-Boot-specific changes in U-Boot's copy of libfdt.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help