From d425ddd10518b38fa107d3278cc483c036445a40 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Sat, 5 Dec 2020 18:22:58 +0100 Subject: azalia: Make `azalia_find_verb` parametric Allow to specify which table should the verb list be read from. Change-Id: Id1bc40c4364cda848f416bad9eeab1b8ca3e9512 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/48350 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Rudolph --- src/device/azalia_device.c | 12 ++++++------ src/include/device/azalia_device.h | 2 +- src/southbridge/intel/bd82x6x/azalia.c | 2 +- src/southbridge/intel/i82801gx/azalia.c | 2 +- src/southbridge/intel/i82801ix/azalia.c | 2 +- src/southbridge/intel/i82801jx/azalia.c | 2 +- src/southbridge/intel/ibexpeak/azalia.c | 2 +- 7 files changed, 12 insertions(+), 12 deletions(-) diff --git a/src/device/azalia_device.c b/src/device/azalia_device.c index 440b6fb83d..283d2f0fa0 100644 --- a/src/device/azalia_device.c +++ b/src/device/azalia_device.c @@ -85,17 +85,17 @@ no_codec: return 0; } -u32 azalia_find_verb(u32 viddid, const u32 **verb) +u32 azalia_find_verb(const u32 *verb_table, u32 verb_table_bytes, u32 viddid, const u32 **verb) { int idx = 0; - while (idx < (cim_verb_data_size / sizeof(u32))) { - u32 verb_size = 4 * cim_verb_data[idx + 2]; // in u32 - if (cim_verb_data[idx] != viddid) { + while (idx < (verb_table_bytes / sizeof(u32))) { + u32 verb_size = 4 * verb_table[idx + 2]; // in u32 + if (verb_table[idx] != viddid) { idx += verb_size + 3; // skip verb + header continue; } - *verb = &cim_verb_data[idx + 3]; + *verb = &verb_table[idx + 3]; return verb_size; } @@ -179,7 +179,7 @@ static void codec_init(struct device *dev, u8 *base, int addr) /* 2 */ reg32 = read32(base + HDA_IR_REG); printk(BIOS_DEBUG, "azalia_audio: codec viddid: %08x\n", reg32); - verb_size = azalia_find_verb(reg32, &verb); + verb_size = azalia_find_verb(cim_verb_data, cim_verb_data_size, reg32, &verb); if (!verb_size) { printk(BIOS_DEBUG, "azalia_audio: No verb!\n"); diff --git a/src/include/device/azalia_device.h b/src/include/device/azalia_device.h index 4e0674e2e3..194572578f 100644 --- a/src/include/device/azalia_device.h +++ b/src/include/device/azalia_device.h @@ -19,7 +19,7 @@ #define HDA_ICII_VALID (1 << 1) int azalia_set_bits(void *port, u32 mask, u32 val); -u32 azalia_find_verb(u32 viddid, const u32 **verb); +u32 azalia_find_verb(const u32 *verb_table, u32 verb_table_bytes, u32 viddid, const u32 **verb); void azalia_audio_init(struct device *dev); extern struct device_operations default_azalia_audio_ops; diff --git a/src/southbridge/intel/bd82x6x/azalia.c b/src/southbridge/intel/bd82x6x/azalia.c index 136d6f2f43..fd0852828b 100644 --- a/src/southbridge/intel/bd82x6x/azalia.c +++ b/src/southbridge/intel/bd82x6x/azalia.c @@ -113,7 +113,7 @@ static void codec_init(struct device *dev, u8 *base, int addr) /* 2 */ reg32 = read32(base + HDA_IR_REG); printk(BIOS_DEBUG, "Azalia: codec viddid: %08x\n", reg32); - verb_size = azalia_find_verb(reg32, &verb); + verb_size = azalia_find_verb(cim_verb_data, cim_verb_data_size, reg32, &verb); if (!verb_size) { printk(BIOS_DEBUG, "Azalia: No verb!\n"); diff --git a/src/southbridge/intel/i82801gx/azalia.c b/src/southbridge/intel/i82801gx/azalia.c index 71f7647ac1..31ff159d64 100644 --- a/src/southbridge/intel/i82801gx/azalia.c +++ b/src/southbridge/intel/i82801gx/azalia.c @@ -111,7 +111,7 @@ static void codec_init(struct device *dev, u8 *base, int addr) /* 2 */ reg32 = read32(base + HDA_IR_REG); printk(BIOS_DEBUG, "Azalia: codec viddid: %08x\n", reg32); - verb_size = azalia_find_verb(reg32, &verb); + verb_size = azalia_find_verb(cim_verb_data, cim_verb_data_size, reg32, &verb); if (!verb_size) { printk(BIOS_DEBUG, "Azalia: No verb!\n"); diff --git a/src/southbridge/intel/i82801ix/azalia.c b/src/southbridge/intel/i82801ix/azalia.c index 5df3213905..2d8e6d4cba 100644 --- a/src/southbridge/intel/i82801ix/azalia.c +++ b/src/southbridge/intel/i82801ix/azalia.c @@ -111,7 +111,7 @@ static void codec_init(struct device *dev, u8 *base, int addr) /* 2 */ reg32 = read32(base + HDA_IR_REG); printk(BIOS_DEBUG, "Azalia: codec viddid: %08x\n", reg32); - verb_size = azalia_find_verb(reg32, &verb); + verb_size = azalia_find_verb(cim_verb_data, cim_verb_data_size, reg32, &verb); if (!verb_size) { printk(BIOS_DEBUG, "Azalia: No verb!\n"); diff --git a/src/southbridge/intel/i82801jx/azalia.c b/src/southbridge/intel/i82801jx/azalia.c index 89e6a8c9f3..3ac0a4a470 100644 --- a/src/southbridge/intel/i82801jx/azalia.c +++ b/src/southbridge/intel/i82801jx/azalia.c @@ -111,7 +111,7 @@ static void codec_init(struct device *dev, u8 *base, int addr) /* 2 */ reg32 = read32(base + HDA_IR_REG); printk(BIOS_DEBUG, "Azalia: codec viddid: %08x\n", reg32); - verb_size = azalia_find_verb(reg32, &verb); + verb_size = azalia_find_verb(cim_verb_data, cim_verb_data_size, reg32, &verb); if (!verb_size) { printk(BIOS_DEBUG, "Azalia: No verb!\n"); diff --git a/src/southbridge/intel/ibexpeak/azalia.c b/src/southbridge/intel/ibexpeak/azalia.c index 0ded4d3f50..2510996bf8 100644 --- a/src/southbridge/intel/ibexpeak/azalia.c +++ b/src/southbridge/intel/ibexpeak/azalia.c @@ -109,7 +109,7 @@ static void codec_init(struct device *dev, u8 *base, int addr) /* 2 */ reg32 = read32(base + HDA_IR_REG); printk(BIOS_DEBUG, "Azalia: codec viddid: %08x\n", reg32); - verb_size = azalia_find_verb(reg32, &verb); + verb_size = azalia_find_verb(cim_verb_data, cim_verb_data_size, reg32, &verb); if (!verb_size) { printk(BIOS_DEBUG, "Azalia: No verb!\n"); -- cgit v1.2.3