summaryrefslogtreecommitdiff
path: root/data.c
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2017-03-06 12:08:53 +1100
committerDavid Gibson <david@gibson.dropbear.id.au>2017-03-06 12:08:53 +1100
commitbad5b28049e5e0562a8ad91797fb77953a53fa20 (patch)
treed58515b0213f4d1ce2d4012dd19070dc629f5c3d /data.c
parent672ac09ea04d998dfddfdef3070a8af8d480182b (diff)
downloaddevice-tree-compiler-bad5b28049e5e0562a8ad91797fb77953a53fa20.tar.gz
Fix assorted sparse warnings
This fixes a great many sparse warnings on the fdt and libfdt sources. These are mostly due to incorrect mixing of endian annotated and native integer types. This includes fixing a couple of quasi-bugs where we had endian conversions the wrong way around (this will have the right effect in practice, but is certainly conceptually incorrect). This doesn't make the whole tree sparse clean: there are many warnings in bison and lex generated code, and there are a handful of other remaining warnings that are (for now) more trouble than they're worth to fix (and are not genuine bugs). Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'data.c')
-rw-r--r--data.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/data.c b/data.c
index 2eeeed3..aa37a16 100644
--- a/data.c
+++ b/data.c
@@ -171,9 +171,9 @@ struct data data_merge(struct data d1, struct data d2)
struct data data_append_integer(struct data d, uint64_t value, int bits)
{
uint8_t value_8;
- uint16_t value_16;
- uint32_t value_32;
- uint64_t value_64;
+ fdt16_t value_16;
+ fdt32_t value_32;
+ fdt64_t value_64;
switch (bits) {
case 8: