[coreboot] Fwd: [RFC] Here we go... the SLOF biosemu forcoreboot-v3

Pattrick Hueper phueper at hueper.net
Tue Dec 16 14:14:00 CET 2008


On Mon, Dec 15, 2008 at 7:28 PM, Peter Stuge <peter at stuge.se> wrote:
> On the other hand, it is of course expected to have descriptive
> file names, which makes biosemu.c a good name for a file doing BIOS
> emulation in any case.
> It could go into Kconfig but users should not be bothered with it.
> Maybe make it changeable for experts only.

here are two more patches,

- rename biosemu to YABEL in Kconfig, CONFIG variables and directory
- make the virtual memory location and size editable in Kconfig

I couldn't figure out a way to tell Kconfig that the VIRTMEM variables
depend on EXPERT to be visible in menu, but still must be set in
.config if EXPERT is not set but YABEL is set.
Thus i introduced a workaround in yabel/compat/functions.c to detect
wether the CONFIG_xxx variables are defined and if they are not
defined, default to 16MB/1MB size.
In this workaround i dont like the fact, that the defaults (16MB for
location, 1MB for size) are set twice... once in Kconfig if EXPERT and
once in yabel/compat/functions.c if !EXPERT and thus no CONFIG_xxx
variables. Any recommendations for that problem?

If you approve of these two patches i will merge them into the patch
set that contains the complete series and is only made up of 4 patches
and repost that series.

Signed-off-by: Pattrick Hueper <phueper at hueper.net>

Cheers, Patty
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-rename-BIOSEMU-into-YABEL-in-Kconfig-and-all-CONFIG.patch
Type: text/x-diff
Size: 253314 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081216/3409eb17/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-make-the-virtual-memory-location-and-size-used-by-YA.patch
Type: text/x-diff
Size: 2463 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081216/3409eb17/attachment-0001.bin>

More information about the coreboot mailing list