<html><head><style type="text/css"><!-- DIV {margin:0px;} --></style></head><body><div style="font-family:times new roman,new york,times,serif;font-size:12pt"><div>Its OK, I'm flexible. Do whatever everyone decides is right, I just need to move on to other things.<br><br>Thanks,<br>Dan Lykowski<br></div><div style="font-family: times new roman,new york,times,serif; font-size: 12pt;"><br><div style="font-family: arial,helvetica,sans-serif; font-size: 13px;"><font size="2" face="Tahoma"><hr size="1"><b><span style="font-weight: bold;">From:</span></b> Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net><br><b><span style="font-weight: bold;">To:</span></b> "Li, Maggie" <Maggie.Li@amd.com><br><b><span style="font-weight: bold;">Cc:</span></b> Marc Jones <marcj303@gmail.com>; Dan Lykowski <engineerguy3737@yahoo.com>; coreboot@coreboot.org<br><b><span style="font-weight: bold;">Sent:</span></b> Tuesday, February 3, 2009 8:20:17
 PM<br><b><span style="font-weight: bold;">Subject:</span></b> Re: [coreboot] SB600 HDA can't find codec fix<br></font><br>
Hi Dan,<br><br>sorry for chasing you around in circles on that issue.<br><br>On 04.02.2009 05:14, Li, Maggie wrote:<br>> Dan <br>><br>> I think you made a mistake. It should be “sm_dev = dev_find_device(PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_ATI_SB600_SM, 0);”, not “sm_dev = pci_locate_device(PCI_ID(0x1002, 0x4385), 0);” in your file.<br>><br>> In my last letter, there is a writing error. We should say that dev_find_device is more reliable than dev_find_slot<br>>   <br><br>I believe dev_find_device is indeed best. Anyway, I think I can make<br>that change locally and commit if you're OK with that.<br><br>Regards,<br>Carl-Daniel<br><br>> -----Original Message-----<br>> From: Carl-Daniel Hailfinger [mailto:<a ymailto="mailto:c-d.hailfinger.devel.2006@gmx.net" href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>] <br>> Sent: Monday, February 02, 2009 10:26 AM<br>> To: Li, Maggie<br>>
 Cc: Dan Lykowski; Marc Jones; <a ymailto="mailto:coreboot@coreboot.org" href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>> Subject: Re: [coreboot] SB600 HDA can't find codec fix<br>><br>> Hi Maggie,<br>><br>> while your point about pci_find_device is valid, I think that<br>> dev_find_device is the function Dan should choose. It allows us to<br>> address the PCI devices even if their bus locations are shifted which is<br>> possible with some AMD chipsets and may also be true for boards with<br>> multiple chipsets.<br>> In general, we want to use functions which can deal with non-default bus<br>> topologies.<br>>   <br>-- <br><a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br><br><br>--<br>coreboot mailing list: <a ymailto="mailto:coreboot@coreboot.org" href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br><a
 href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a></div></div></div><br>



      </body></html>