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

Pattrick Hueper phueper at hueper.net
Tue Dec 16 17:26:16 CET 2008


On Tue, Dec 16, 2008 at 5:21 PM, Myles Watson <mylesgw at gmail.com> wrote:
>
>
> On Tue, Dec 16, 2008 at 6:14 AM, Pattrick Hueper <phueper at hueper.net> wrote:
>>
>> Hi,
>>
>> 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
>

Ok, thats true... i will remove it... i have played with more than 1MB
virtual memory once, but it isnt really needed...

>
>>
> I would take out the default setting in Kconfig.  If the person has to be an
> expert to change it, and they're changing it, there shouldn't be a default.
>

Hm... taking out the default would mean, that if you select expert you
_have_ to enter a value... since it is not preset? That seems a little
bit too much expertism to expect... providing a sensible default seems
to make sense IMHO.

Patty




More information about the coreboot mailing list