[coreboot] buildrom rom size patch

Ward Vandewege ward at gnu.org
Mon Mar 10 20:43:57 CET 2008


On Fri, Mar 07, 2008 at 11:02:58AM -0700, Myles Watson wrote:
> This patch makes use of the CONFIG_COREBOOT_V3_ROM_SIZE variable in buildrom.
> The hope is that eventually we could do some math with it if we want.  The
> only problem is that when you don't have it set, it gives you an error when
> you do make menuconfig.  Here is the error:
> 
> #
> # using defaults found in .config
> #
> .config:23:warning: symbol value '' invalid for COREBOOT_V3_ROM_SIZE
> .config:83:warning: symbol value '' invalid for LAB_PAUSE
> 
> Notice that LAB_PAUSE has the same problem.  Setting the value to be a string
> instead of an int removes the warning, but removes the error checking as well.
> 
> Myles
> 
> Signed-off-by: Myles Watson <mylesgw at gmail.com>

I've build-tested this with the alix.1c target (which needs some patching to
make it work, I'm going  to send those to the list when I'm happy with them).
This patch generates a valid lar image of the specified size (at least lar -l
tells me so).

Acked-by: Ward Vandewege <ward at gnu.org>

-- 
Ward Vandewege <ward at fsf.org>
Free Software Foundation - Senior System Administrator




More information about the coreboot mailing list