[coreboot] [PATCH] __ROMCC__ ifdefs

Myles Watson mylesgw at gmail.com
Fri Nov 6 17:37:19 CET 2009


On Fri, Oct 30, 2009 at 7:24 AM, Myles Watson <mylesgw at gmail.com> wrote:

> I think it's time to separate the two meanings of __ROMCC__.  Right now it
> is used for:
> 1. Use simple functions and data types since we don't have RAM
> 2. Don't use prototypes, romcc doesn't support them.
>
> Right now the patch creates __PRE_RAMINIT__
>
> The name is not the most important part of the patch, and can easily be
> changed.
>
> Here are some of the suggestions so far:
> __PRE_RAM__
> __NO_RAM_YET__
> __DURING_RAMINIT__
> __USE_NO_RAM__
>
> Unless it was obvious to me which meaning was the correct one for a file, I
> added both for now.
>
> Abuild tested.
>
> Signed-off-by: Myles Watson <mylesgw at gmail.com>
>
Same patch with __PRE_RAM__

Still abuild-tested.

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091106/e955d119/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: romcc.diff
Type: text/x-patch
Size: 51440 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091106/e955d119/attachment.diff>


More information about the coreboot mailing list