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

[Myles Watson]

On Tue, Dec 16, 2008 at 9:26 AM, Pattrick Hueper <phueper at hueper.net> wrote:

> 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.

You're right, that also doesn't make sense.  I meant make it so that you
have to provide a value if you want to change it.  So could you introduce a
boolean that deafaults to false and depends on expert that is "set the
address for yabel", then not have a default for the address.

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081216/b1579202/attachment.html>