<br><br><div class="gmail_quote">On Tue, Dec 16, 2008 at 6:14 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;">
Hi,<br>
<div class="Ih2E3d"><br>
On Mon, Dec 15, 2008 at 7:28 PM, Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>> wrote:<br>
</div><div class="Ih2E3d">> 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>
</div>...<br>
<div class="Ih2E3d">><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>
</div>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</blockquote><div><br>I don't think the size should be a config option.  Why should the size change?  If it is to add features, the features should be the config options.<br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I couldn't figure out a way to tell Kconfig that the VIRTMEM variables<br>
depend on EXPERT to be visible in menu, but still must be set in<br>
.config if EXPERT is not set but YABEL is set.<br>
Thus i introduced a workaround in yabel/compat/functions.c to detect<br>
wether the CONFIG_xxx variables are defined and if they are not<br>
defined, default to 16MB/1MB size.<br>
In this workaround i dont like the fact, that the defaults (16MB for<br>
location, 1MB for size) are set twice... once in Kconfig if EXPERT and<br>
once in yabel/compat/functions.c if !EXPERT and thus no CONFIG_xxx<br>
variables. Any recommendations for that problem?</blockquote><div><br>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.<br>
<br>Thanks,<br>Myles <br></div></div><br>