[S] Change in coreboot[master]: sb/intel/i82371eb: Streamline IDE debug messages

[Keith Hui (Code Review)]

Keith Hui has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/73331 )

Change subject: sb/intel/i82371eb: Streamline IDE debug messages
......................................................................

sb/intel/i82371eb: Streamline IDE debug messages

Debug messages shown during IDE initialization are streamlined as
follows:

"Primary IDE interface" (and similar) are shortened to
"Primary interface".
We don't need to see "IDE" twice as messages are already prefixed.

Refactor "IDE: (Primary) IDE interface: (on)" into
"IDE: (Primary interface): (on)" to allow compiler to deduplicate
component strings, also used later in messages re UDMA/33.

This reduces uncompressed string size by 32 bytes and allows ramstage
to compress a wee bit better.

Change-Id: I16f5c2b3775c5a73b83d83817d7075e944089a12
Signed-off-by: Keith Hui 
---
M src/southbridge/intel/i82371eb/ide.c
1 file changed, 30 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/31/73331/1

diff --git a/src/southbridge/intel/i82371eb/ide.c b/src/southbridge/intel/i82371eb/ide.c
index 9ea71db..5a2f190 100644
--- a/src/southbridge/intel/i82371eb/ide.c
+++ b/src/southbridge/intel/i82371eb/ide.c
@@ -32,14 +32,14 @@
    reg16 = pci_read_config16(dev, IDETIM_PRI);
    reg16 = ONOFF(conf->ide0_enable, reg16, IDE_DECODE_ENABLE);
    pci_write_config16(dev, IDETIM_PRI, reg16);
-	printk(BIOS_DEBUG, "IDE: %s IDE interface: %s\n", "Primary",
+	printk(BIOS_DEBUG, "IDE: %s: %s\n", "Primary interface",
    	     conf->ide0_enable ? "on" : "off");

/* Enable/disable the secondary IDE interface. */
    reg16 = pci_read_config16(dev, IDETIM_SEC);
    reg16 = ONOFF(conf->ide1_enable, reg16, IDE_DECODE_ENABLE);
    pci_write_config16(dev, IDETIM_SEC, reg16);
-	printk(BIOS_DEBUG, "IDE: %s IDE interface: %s\n", "Secondary",
+	printk(BIOS_DEBUG, "IDE: %s: %s\n", "Secondary interface",
    	     conf->ide1_enable ? "on" : "off");

/* Enable access to the legacy IDE ports (both primary and secondary),
@@ -81,10 +81,10 @@
    	pci_write_config8(dev, UDMACTL, reg8);

printk(BIOS_DEBUG, "IDE: %s, drive %d: UDMA/33: %s\n",
-			     "Primary IDE interface", 0,
+			     "Primary interface", 0,
    		     conf->ide0_drive0_udma33_enable ? "on" : "off");
    	printk(BIOS_DEBUG, "IDE: %s, drive %d: UDMA/33: %s\n",
-			     "Primary IDE interface", 1,
+			     "Primary interface", 1,
    		     conf->ide0_drive1_udma33_enable ? "on" : "off");
    }

@@ -96,10 +96,10 @@
    	pci_write_config8(dev, UDMACTL, reg8);

printk(BIOS_DEBUG, "IDE: %s, drive %d: UDMA/33: %s\n",
-			     "Secondary IDE interface", 0,
+			     "Secondary interface", 0,
    		     conf->ide1_drive0_udma33_enable ? "on" : "off");
    	printk(BIOS_DEBUG, "IDE: %s, drive %d: UDMA/33: %s\n",
-			     "Secondary IDE interface", 1,
+			     "Secondary interface", 1,
    		     conf->ide1_drive1_udma33_enable ? "on" : "off");
    }
 }