[coreboot] [PATCH] Remove warnings
Myles Watson
mylesgw at gmail.com
Fri Oct 30 03:11:30 CET 2009
On Thu, Oct 29, 2009 at 7:35 PM, Peter Stuge <peter at stuge.se> wrote:
> Myles Watson wrote:
> > These three patches get rid of a few more warnings for me.
> >
> > abuild tested.
> >
> > Signed-off-by: Myles Watson <mylesgw at gmail.com>
>
> Acked-by: Peter Stuge <peter at stuge.se>
>
Rev 4890.
Thanks,
Myles
>
> > PS. It looks like part of the reason we have so many warnings is
> > the misuse of #define __ROMCC__ to mean "early code before the
> > device tree is available" __ROMCC__ is also used to get rid of
> > prototypes because romcc doesn't support them. What's the right
> > thing to do here? Maybe we should split the meanings so that we
> > test for __GNUC__ when we mean "not romcc", and we test for
> > __ROMCC__ (renamed, of course, later) when we mean early init?
>
> I'd like __ROMCC__ to stay, but I agree very much with a new #define
> BEFOREDEVICETREE or somesuch.
>
How about __PRE_RAM__, since it could be CAR but it is before RAM init?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091029/24928f11/attachment.html>
More information about the coreboot
mailing list