[coreboot] SB600 HDA can't find codec fix
Li, Maggie
Maggie.Li at amd.com
Mon Feb 2 03:22:35 CET 2009
Stefan
I agree with you.
Best regards
Maggie li
-----Original Message-----
From: Stefan Reinauer [mailto:stepan at coresystems.de]
Sent: Monday, February 02, 2009 10:18 AM
To: Li, Maggie
Cc: Dan Lykowski; Marc Jones; Carl-Daniel Hailfinger; coreboot at coreboot.org
Subject: Re: [coreboot] SB600 HDA can't find codec fix
Li, Maggie wrote:
>
> Dan,
>
>
>
> I read the RRG spec, just as you said, you should use 0xF in file
> sb600_hda.c to get the current audio codec, that is to say,
>
> dword &= 0xF;
> if (!dword)
> goto no_codec;
>
>
>
> pci_locate_device is really should be used in early setup. Device_t is
> the type of u32 at that time. After this stage, you should use
> dev_find_slot. In file sb600_sata.c, “/* sm_dev =
> pci_locate_device(PCI_ID(0x1002, 0x4385), 0) */ “ gives you
> misunderstanding about how to get the SMBus and it should be removed.
> You can submit a patch about sata and I will ack it. Thanks.
>
Somewhat related, I think the
device_t azalia_dev = dev_find_slot(0, PCI_DEVFN(0x14, 2));
should be passed from hda_init() instead, because the device is already
known there.
Stefan
--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
More information about the coreboot
mailing list