<br><br><div class="gmail_quote">On Tue, Dec 16, 2008 at 9:26 AM, Pattrick Hueper <span dir="ltr"><<a href="mailto:phueper@hueper.net">phueper@hueper.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Tue, Dec 16, 2008 at 5:21 PM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
><br>
><br>
> On Tue, Dec 16, 2008 at 6:14 AM, Pattrick Hueper <<a href="mailto:phueper@hueper.net">phueper@hueper.net</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> On Mon, Dec 15, 2008 at 7:28 PM, Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>> wrote:<br>
>> > On the other hand, it is of course expected to have descriptive<br>
>> > file names, which makes biosemu.c a good name for a file doing BIOS<br>
>> > emulation in any case.<br>
>> ><br>
>> ...<br>
>> ><br>
>> > It could go into Kconfig but users should not be bothered with it.<br>
>> > Maybe make it changeable for experts only.<br>
>> ><br>
>><br>
>> here are two more patches,<br>
>><br>
>> - rename biosemu to YABEL in Kconfig, CONFIG variables and directory<br>
>> - make the virtual memory location and size editable in Kconfig<br>
><br>
<br>
</div>Ok, thats true... i will remove it... i have played with more than 1MB<br>
virtual memory once, but it isnt really needed...<br>
<div class="Ih2E3d"><br>
><br>
>><br>
> I would take out the default setting in Kconfig.  If the person has to be an<br>
> expert to change it, and they're changing it, there shouldn't be a default.<br>
><br>
<br>
</div>Hm... taking out the default would mean, that if you select expert you<br>
_have_ to enter a value... since it is not preset? That seems a little<br>
bit too much expertism to expect... providing a sensible default seems<br>
to make sense IMHO.</blockquote><div>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.<br>
<br>Thanks,<br>Myles <br></div></div><br>