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

Myles Watson mylesgw at gmail.com
Tue Dec 16 18:05:02 CET 2008


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

> On Tue, Dec 16, 2008 at 5:28 PM, Myles Watson <mylesgw at gmail.com> wrote:
> >>
> >> Ok, thats true... i will remove it... i have played with more than 1MB
> >> virtual memory once, but it isnt really needed...
> >>
> ...
> >
> > 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
> >
> >
>
> Update patch attached. Removed size setting, changed location setting
> as proposed.

Looks good, but I don't see the default value.  It's just in the comment.
If you add the default value and send the updated patches, we can commit.

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


More information about the coreboot mailing list