[coreboot] [PATCH] sb600: don't load verb for codec

[Bao, Zheng]

Don't load verb in BIOS stage. It is not BIOS's responsibility.
And we can not have verb for every single codec.

Signed-off-by: Zheng Bao <zheng....@amd.com>

Index: src/southbridge/amd/sb600/sb600_hda.c
===================================================================
--- src/southbridge/amd/sb600/sb600_hda.c       (revision 5184)
+++ src/southbridge/amd/sb600/sb600_hda.c       (working copy)
@@ -90,88 +90,10 @@
        return 0;
 }
 
-static u32 cim_verb_data[] = {
-       0x01471c10,
-       0x01471d40,
-       0x01471e01,
-       0x01471f01,
-/* 1 */
-       0x01571c12,
-       0x01571d10,
-       0x01571e01,
-       0x01571f01,
-/* 2 */
-       0x01671c11,
-       0x01671d60,
-       0x01671e01,
-       0x01671f01,
-/* 3 */
-       0x01771c14,
-       0x01771d20,
-       0x01771e01,
-       0x01771f01,
-/* 4 */
-       0x01871c30,
-       0x01871d90,
-       0x01871ea1,
-       0x01871f01,
-/* 5 */
-       0x01971cf0,
-       0x01971d11,
-       0x01971e11,
-       0x01971f41,
-/* 6 */
-       0x01a71c80,
-       0x01a71d30,
-       0x01a71e81,
-       0x01a71f01,
-/* 7 */
-       0x01b71cf0,
-       0x01b71d11,
-       0x01b71e11,
-       0x01b71f41,
-/* 8 */
-       0x01c71cf0,
-       0x01c71d11,
-       0x01c71e11,
-       0x01c71f41,
-/* 9 */
-       0x01d71cf0,
-       0x01d71d11,
-       0x01d71e11,
-       0x01d71f41,
-/* 10 */
-       0x01e71c50,
-       0x01e71d11,
-       0x01e71e44,
-       0x01e71f01,
-/* 11 */
-       0x01f71c60,
-       0x01f71d61,
-       0x01f71ec4,
-       0x01f71f01,
-};
-static u32 find_verb(u32 viddid, u32 ** verb)
-{
-       device_t azalia_dev = dev_find_slot(0, PCI_DEVFN(0x14, 2));
-       struct southbridge_amd_sb600_config *cfg =
-           (struct southbridge_amd_sb600_config
*)azalia_dev->chip_info;
-       printk_debug("Dev=%s\n", dev_path(azalia_dev));
-       printk_debug("Default viddid=%x\n", cfg->hda_viddid);
-       printk_debug("Reading viddid=%x\n", viddid);
-       if (!cfg)
-               return 0;
-       if (viddid != cfg->hda_viddid)
-               return 0;
-       *verb = (u32 *) cim_verb_data;
-       return sizeof(cim_verb_data) / sizeof(u32);
-}
-
 /**
  *  Wait 50usec for for the codec to indicate it is ready
  *  no response would imply that the codec is non-operative
  */
-
 static int wait_for_ready(u8 *base)
 {
        /* Use a 50 usec timeout - the Linux kernel uses the
@@ -194,7 +116,6 @@
  *  the previous command.  No response would imply that the code
  *  is non-operative
  */
-
 static int wait_for_valid(u8 *base)
 {
        /* Use a 50 usec timeout - the Linux kernel uses the
@@ -215,9 +136,6 @@
 static void codec_init(u8 * base, int addr)
 {
        u32 dword;
-       u32 *verb;
-       u32 verb_size;
-       int i;
 
        /* 1 */
        if (wait_for_ready(base) == -1)
@@ -232,26 +150,7 @@
        dword = read32(base + 0x64);
 
        /* 2 */
-       printk_debug("codec viddid: %08x\n", dword);
-       verb_size = find_verb(dword, &verb);
-
-       if (!verb_size) {
-               printk_debug("No verb!\n");
-               return;
-       }
-
-       printk_debug("verb_size: %d\n", verb_size);
-       /* 3 */
-       for (i = 0; i < verb_size; i++) {
-               if (wait_for_ready(base) == -1)
-                       return;
-
-               write32(base + 0x60, verb[i]);
-
-               if (wait_for_valid(base) == -1)
-                       return;
-       }
-       printk_debug("verb loaded!\n");
+       printk_debug("%x(th) codec viddid: %08x\n", addr, dword);
 }
 
 static void codecs_init(u8 * base, u32 codec_mask)

sb600_dont_load_verb.patch
Description: sb600_dont_load_verb.patch

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot