<br><br><div class="gmail_quote">On Thu, Apr 8, 2010 at 4:12 PM, Joseph Smith <span dir="ltr"><<a href="mailto:joe@settoplinux.org">joe@settoplinux.org</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;">
<br>
<br>
<br>
On Thu, 8 Apr 2010 16:02:09 -0600, "Myles Watson" <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
wrote:<br>
><br>
><br>
<div><div></div><div class="h5">>> -----Original Message-----<br>
>> From: Joseph Smith [mailto:<a href="mailto:joe@settoplinux.org">joe@settoplinux.org</a>]<br>
>> Sent: Thursday, April 08, 2010 3:59 PM<br>
>> To: Myles Watson<br>
>> Cc: Stefan Reinauer; <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
>> Subject: Re: [coreboot] [PATCH] CAR on Intel P6 Series - Support for<br>
>> model_6bx<br>
>><br>
>> >> > On 4/8/10 11:32 PM, Joseph Smith wrote:<br>
>> >> >>>> Index: src/cpu/intel/model_6bx/Kconfig<br>
>> >> >>>><br>
>> ===================================================================<br>
>> >> >>>> --- src/cpu/intel/model_6bx/Kconfig        (revision 0)<br>
>> >> >>>> +++ src/cpu/intel/model_6bx/Kconfig        (revision 0)<br>
>> >> >>>> @@ -0,0 +1,3 @@<br>
>> >> >>>> +config CPU_INTEL_CORE<br>
>> >> >>>> +  bool<br>
>> >> >>>> +  select SMP<br>
>> >><br>
>> > This looks like it was copied directly from<br>
> cpu/intel/model_6ex/Kconfig.<br>
>> ><br>
>> > You are redefining CPU_INTEL_CORE here.  This is probably where you<br>
>> wanted<br>
>> > to define CPU_INTEL_MODEL_6BX.<br>
>> ><br>
>> Ah ok thanks Myles.<br>
>><br>
>> So there is not actually a:<br>
>><br>
>> #define CPU_INTEL_MODEL_6BX blabla<br>
>><br>
>> preprocessing directive anywhere. It just needs to be defined in<br>
Kconfig.<br>
><br>
> That's right.  "config FOO" defines CONFIG_FOO.  "select FOO" just sets<br>
it<br>
> if it exists.  You can check in your .config file to make sure the<br>
symbols<br>
> you expect to be defined are showing up.<br>
><br>
</div></div>Hmm, I wonder why this did not throw an error at me?</blockquote><div><br>Selecting undefined config options doesn't throw an error.  I think it would be nice if it did, but there might be some reason that I don't know of why you want to be able to select undefined things and have nothing happen.<br>
<br>Thanks,<br>Myles<br><br> <br></div></div>