On Wed, Dec 17, 2008 at 6:51 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</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><div></div><div class="Wj3C7c">On 17.12.2008 22:19, Corey Osgood wrote:<br>
> On Wed, Dec 17, 2008 at 3:24 PM, Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>>wrote:<br>
><br>
><br>
>> On Wed, Dec 17, 2008 at 2:03 PM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
>><br>
>><br>
>>> On Wed, Dec 17, 2008 at 11:43 AM, Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>>wrote:<br>
>>><br>
>>><br>
>>>> On Tue, Dec 16, 2008 at 12:35 PM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>>wrote:<br>
>>>><br>
>>>><br>
>>>>> Corey,<br>
>>>>><br>
>>>>> I've been looking through the v2 code, but I don't see anywhere that pci<br>
>>>>> config reads/writes are enabled before scanning.  If you're interested, you<br>
>>>>> could send me a v2 boot log and I'll try to help more. You could also try<br>
>>>>> doing a direct pci config read to the device to see if it works before the<br>
>>>>> hang.<br>
>>>>><br>
>>>> I got looking at some other drivers and reading some other mails on the<br>
>>>> list, and changing pci_ops to use cf0/cf8 fixed it, and also fixed my<br>
>>>> problems with the ide and sata drivers hanging (so all those phase3<br>
>>>> functions are back in phase6 where they belong). The system now boots to<br>
>>>> memtest and actually sees the memory to test now, but I'm still having a<br>
>>>> problem building the cpu driver, which means booting is EXTREMEMLY slow. The<br>
>>>> stage2 driver is in via/c7.c, but no matter where I try to include it in the<br>
>>>> make files, arch/x86/Makefile or mainboard/jetway/j7f2/Makefile, I get this<br>
>>>> error:<br>
>>>><br>
>>>> make: *** No rule to make target<br>
>>>> `/home/corey/coreboot/coreboot-v3/build/mainboard/jetway/j7f2//home/corey/coreboot/coreboot-v3/arch/x86/via/c7.o',<br>
>>>> needed by `/home/corey/coreboot/coreboot-v3/build/coreboot.stage2'.  Stop.<br>
>>>><br>
>>>> I'm no expert on makefiles, so I could use a hand. I've tried to locate<br>
>>>> where/how geodelx/cpu.c is built, but it doesn't seem that it ever is,<br>
>>>> adding a #error to the file still lets alix1c and other geodelx boards build<br>
>>>> without error.<br>
>>>><br>
>>>> Other then that, things are looking VERY promising, I'm finishing up a<br>
>>>> few things I put off before, and once the CPU driver is running I'd like to<br>
>>>> try booting a kernel :)<br>
>>>><br>
>>>> Thanks,<br>
>>>> Corey<br>
>>>><br>
>>> I agree with Ron.  You're doing good work and we could help you easier if<br>
>>> we commit it.  I added one more line to your Makefile and it tries to build<br>
>>> the file.<br>
>>><br>
>>> I think the c7.c file should move, but for now it helps.<br>
>>><br>
>>> Thanks,<br>
>>> Myles<br>
>>><br>
>>><br>
>> Ok, adding it there work. But why does it work there but nowhere else? I'll<br>
>> try to make a commit within the hour, I've just got to split the c7/cn700<br>
>> stuff off from the other hacking I've been doing.<br>
>><br>
>><br>
><br>
> self-acked and committed, feel free to move c7.c anywhere you think it<br>
> belongs ;) I've self-acked for expediency, I'm about to brave a snowstorm to<br>
> get to school for a final, and won't be back for probably 4 hours or so. I<br>
> also updated the epia-cn dts, so it should be in the same state as the j7f2<br>
> if you fix the superio stuff.<br>
><br>
<br>
</div></div>Sorry I didn't find time to review this sooner. Is it intentional that<br>
one of the files written by you had a (C) coresystems?</blockquote><div><br>Yes - those files are pretty much direct copies from v2 code, just with the v3 driver struct, x_t -> struct x changes, and include files fixed.<br>
<br>-Corey<br></div></div><br>