[SeaBIOS] New build warnings with GCC 5
Paul Menzel
paulepanter at users.sourceforge.net
Tue Jun 2 22:50:30 CEST 2015
Dear Kevin,
Am Montag, den 01.06.2015, 18:36 -0400 schrieb Kevin O'Connor:
> On Sat, May 23, 2015 at 03:47:32PM +0200, Paul Menzel wrote:
> > building SeaBIOS commit 67643955 (make SeaBios compatible with Xen
> > vTPM.) for coreboot with the attached configuration with GCC 5.1.1, (CPP
> > 4.9) the following warnings are printed.
> >
> > $ gcc-5 --version
> > gcc-5 (Debian 5.1.1-7) 5.1.1 20150522
> > Copyright (C) 2015 Free Software Foundation, Inc.
> > This is free software; see the source for copying conditions.
> > There is NO
> > warranty; not even for MERCHANTABILITY or FITNESS FOR A
> > PARTICULAR PURPOSE.
> >
> > $ CC=gcc-5 make
> > Build Kconfig config file
> > Compile checking out/src/misc.o
> > In file included from src/misc.c:13:0:
> > src/string.h:18:13: warning: inline function 'memcpy_far'
> > declared but never defined
> > inline void memcpy_far(u16 d_seg, void *d_far
> > ^
>
> Thanks. It looks like gcc v5 is picky about using the "inline"
> keyword in function declarations. Patch below fixes the warnings for
> me.
[…]
> commit a8a8fc9e35af6d1ee660a5f3a6f4a825f4465824
> Author: Kevin O'Connor <kevin at koconnor.net>
> Date: Mon Jun 1 18:32:06 2015 -0400
>
> Don't forward declare functions with "inline" in headers
>
> Don't mark function definitions in headers with "inline" - it causes
> compile warnings on gcc v5.
>
> Signed-off-by: Kevin O'Connor <kevin at koconnor.net>
[…]
Applying this patch on top of master, I was unable to reproduce the
warnings with GCC 5.1.
Acked-by: Paul Menzel <paulepanter at sourceforge.net>
Thanks,
Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part
URL: <http://www.seabios.org/pipermail/seabios/attachments/20150602/b22ba272/attachment.asc>
More information about the SeaBIOS
mailing list