[coreboot-gerrit] New patch to review for coreboot: nb/amd/amdht: Use variable for function name

[Paul Menzel (paulepanter@users.sourceforge.net)]

Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18556

-gerrit

commit 6143dac757ba17f2c3550ae9cbd9f9b491b41f3f
Author: Paul Menzel <pmenzel at molgen.mpg.de>
Date:   Fri Mar 3 14:41:33 2017 +0100

    nb/amd/amdht: Use variable for function name
    
    One very long line has to be wrapped to be shorter than 80 characters to
    satisfy the lint scripts.
    
    Change-Id: Ie98eff360ebc5b68ce496edc15eb2d9fddcac868
    Signed-off-by: Paul Menzel <pmenzel at molgen.mpg.de>
---
 src/northbridge/amd/amdht/ht_wrapper.c | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/northbridge/amd/amdht/ht_wrapper.c b/src/northbridge/amd/amdht/ht_wrapper.c
index 8babb47..f4e8337 100644
--- a/src/northbridge/amd/amdht/ht_wrapper.c
+++ b/src/northbridge/amd/amdht/ht_wrapper.c
@@ -110,7 +110,7 @@ static void AMD_CB_EventNotify (u8 evtClass, u16 event, const u8 *pEventData0)
 	uint8_t log_level;
 	uint8_t dump_event_detail;
 
-	printk(BIOS_DEBUG, "AMD_CB_EventNotify(): ");
+	printk(BIOS_DEBUG, "%s: ", __func__);
 
 	/* Decode event */
 	dump_event_detail = 1;
@@ -232,7 +232,7 @@ void amd_ht_init(struct sys_info *sysinfo)
 {
 
 	if (!sysinfo) {
-		printk(BIOS_DEBUG, "Skipping amd_ht_init()\n");
+		printk(BIOS_DEBUG, "Skipping %s\n", __func__);
 		return;
 	}
 
@@ -256,9 +256,9 @@ void amd_ht_init(struct sys_info *sysinfo)
 		&sysinfo->ht_link_cfg // struct ht_link_config*
 	};
 
-	printk(BIOS_DEBUG, "Enter amd_ht_init()\n");
+	printk(BIOS_DEBUG, "Enter %s\n", __func__);
 	amdHtInitialize(&ht_wrapper);
-	printk(BIOS_DEBUG, "Exit amd_ht_init()\n");
+	printk(BIOS_DEBUG, "Exit %s\n", __func__);
 }
 
 /**
@@ -268,7 +268,7 @@ void amd_ht_init(struct sys_info *sysinfo)
  *
  */
 void amd_ht_fixup(struct sys_info *sysinfo) {
-	printk(BIOS_DEBUG, "amd_ht_fixup()\n");
+	printk(BIOS_DEBUG, "%s\n", __func__);
 	if (IS_ENABLED(CONFIG_CPU_AMD_MODEL_10XXX)) {
 		uint8_t rev_gte_d = 0;
 		uint8_t fam15h = 0;
@@ -307,7 +307,11 @@ void amd_ht_fixup(struct sys_info *sysinfo) {
 				for (node = 0; node < node_count; node++) {
 					f3xe8 = pci_read_config32(NODE_PCI(node, 3), 0xe8);
 					uint8_t internal_node_number = ((f3xe8 & 0xc0000000) >> 30);
-					printk(BIOS_DEBUG, "amd_ht_fixup(): node %d (internal node ID %d): disabling defective HT link", node, internal_node_number);
+					printk(BIOS_DEBUG,
+					       "%s: node %d (internal node "
+					       "ID %d): disabling defective "
+					       "HT link", __func__, node,
+					       internal_node_number);
 					if (internal_node_number == 0) {
 						uint8_t package_link_3_connected = pci_read_config32(NODE_PCI(node, 0), (fam15h)?0x98:0xd8) & 0x1;
 						printk(BIOS_DEBUG, " (L3 connected: %d)\n", package_link_3_connected);