From e89d444043b63529fa7d5a54e821ea491da006c2 Mon Sep 17 00:00:00 2001 From: Richard Spiegel Date: Fri, 8 Dec 2017 07:52:42 -0700 Subject: soc/amd/stoneyridge: Remove "\t" from name table Remove "\t" from name strings in soc/amd/stoneyridge/southbridge.c array irq_association[], and change the print string in soc/amd/common/amd_pci_util.c that use the names from "%s" to "%-20s". This sets a fixed field of 20 characters for the string name, allowing for variable length to the names (up to 20 characters), thus saving memory space used by the strings. BUG=b:70344551 TEST=Build and boot, record output of irq routing and verify alignment. Change-Id: I92dfac9b64932fb0cd3359abd4d1aac651535f1a Signed-off-by: Richard Spiegel Reviewed-on: https://review.coreboot.org/22785 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/soc/amd/common/block/pci/amd_pci_util.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/soc/amd/common') diff --git a/src/soc/amd/common/block/pci/amd_pci_util.c b/src/soc/amd/common/block/pci/amd_pci_util.c index d30adf4134..c5659cbc79 100644 --- a/src/soc/amd/common/block/pci/amd_pci_util.c +++ b/src/soc/amd/common/block/pci/amd_pci_util.c @@ -70,7 +70,7 @@ void write_pci_int_table(void) /* PIC IRQ routine */ printk(BIOS_DEBUG, "PCI_INTR tables: Writing registers C00/C01 for" " PCI IRQ routing:\n" - "\tPCI_INTR_INDEX\t\tPIC mode" + "PCI_INTR_INDEX\tname\t\t PIC mode" "\tAPIC mode\n"); /* * Iterate table idx_name, indexes outside the table are ignored @@ -81,7 +81,7 @@ void write_pci_int_table(void) for (i = 0 ; i < limit; i++) { byte = idx_name[i].index; write_pci_int_idx(byte, 0, (u8) picr_data_ptr[byte]); - printk(BIOS_DEBUG, "\t0x%02X %s\t0x%02X\t\t", + printk(BIOS_DEBUG, "0x%02X\t\t%-20s 0x%02X\t", byte, idx_name[i].name, read_pci_int_idx(byte, 0)); write_pci_int_idx(byte, 1, (u8) intr_data_ptr[byte]); -- cgit v1.2.3