<div dir="ltr">On Fri, Oct 10, 2008 at 11:40 AM, ron minnich <span dir="ltr"><<a href="mailto:rminnich@gmail.com">rminnich@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><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 Fri, Oct 10, 2008 at 12:25 AM, Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>> wrote:<br>
<br>
> Sorry for the mailbox flooding, I'm slowly learning to step back and look at<br>
> things before jumping to conclusions. Geode and i586 also have ROM_CODE_SEG<br>
> defined, this didn't, that's why the jump was going to nowhere. Do those<br>
> defines belong in this file, somewhere else, or in Kconfig? Are the i586<br>
> values the same as they should be for c7?<br>
<br>
</div>ROM_CODE_SEG is probably the same for all x86 but we did not want to<br>
assume anything.<br>
<div class="Ih2E3d"><br>
><br>
> I'm now getting hung up because XIP_ROM_[SIZE,BASE] isn't defined. Do we<br>
> still need XIP and those defines? Are the value limitations the same as they<br>
> were in v2?<br>
><br>
<br>
</div>XIP, hmm. Those are now called CONFIG_XIP_*. Although what meaning<br>
they have in v3 is questionable to me. I have never used it.<br>
<br>
Note the error in this snippet from  arch/x86/i586/stage0.S<br>
<br>
#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)<br>
<div class="Ih2E3d">        /* enable write base caching so we can do execute in place<br>
         * on the flash rom.<br>
</div>         */<br>
<div class="Ih2E3d">        movl    $0x202, %ecx<br>
        xorl    %edx, %edx<br>
        movl    $(XIP_ROM_BASE | MTRR_TYPE_WRBACK), %eax<br>
</div>                      ^^^^^^^^^^^^^^^^^<br>
                     WRONG NAME!<br>
        wrmsr<br>
<div class="Ih2E3d"><br>
        movl    $0x203, %ecx<br>
        movl    $0x0000000f, %edx<br>
        movl    $(~(XIP_ROM_SIZE - 1) | 0x800), %eax<br>
        wrmsr<br>
</div>#endif /* XIP_ROM_SIZE && XIP_ROM_BASE */</blockquote><div><br>Oops, I meant to respond to this. It seems that file is the only place CONFIG_XIP* shows up:<br><br>corey@Mobulus:~/coreboot/coreboot-v3$ grep -rn CONFIG_XIP *<br>
arch/x86/i586/.svn/text-base/stage0.S.svn-base:245:#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)<br>arch/x86/i586/stage0.S:245:#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)<br>
<br>I'm gonna try commenting out the XIP stuff and seeing what it does.<br><br>Thanks,<br>Corey<br><br></div></div><br></div>