[Skiboot] [PATCH 0/6] Various build fixes

Joel Stanley joel at jms.id.au
Tue Jul 21 13:23:09 AEST 2015


I've been building skiboot with clang. The Makefile changes aren't in a
commitable state at this point, but a bunch of fixes can go in.

There are two ways go here: Patch 1 and 2 are not necessary if we apply
patch 5: Enforce GNU 11 as default C dialect. Conversely, if that's too
controversial, then patch 1 and 2 can go in to avoid the compiler warnings
while we decide on what dialect we want to enforce.

Patch 3 is a real bug that was found by clang. There has been another attempt
at a fix posted to the list, but it has not gone in yet.

Joel Stanley (6):
  libflash: Put variable sized type at end of struct
  trace: Put variable sized type at end of struct
  fsp/chiptod: Fix initialised variable
  Add prototype in asm-offsets to silence clang
  Enforce GNU 11 as default C dialect
  ccan: Update short_types module to fix warning

 Makefile.main                      |  2 +-
 asm/asm-offsets.c                  |  2 ++
 ccan/short_types/_info             | 12 +++++++++---
 ccan/short_types/short_types.h     | 15 +++++++++------
 ccan/short_types/test/run-endian.c | 20 ++++++++++++++++++++
 ccan/short_types/test/run.c        | 15 ++-------------
 core/trace.c                       |  2 +-
 hw/chiptod.c                       | 10 +++++-----
 hw/fsp/fsp-chiptod.c               | 12 +++++++-----
 include/chiptod.h                  |  2 +-
 libflash/libffs.c                  |  2 +-
 11 files changed, 58 insertions(+), 36 deletions(-)
 create mode 100644 ccan/short_types/test/run-endian.c

-- 
2.1.4



More information about the Skiboot mailing list