src/sb: Use 'print("%s...", __func__)'
Change-Id: Ie0d845d3e501ed5ebeef1997944445d31768e410
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39373
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
diff --git a/src/southbridge/amd/agesa/hudson/hudson.c b/src/southbridge/amd/agesa/hudson/hudson.c
index 3f04987..3609314 100644
--- a/src/southbridge/amd/agesa/hudson/hudson.c
+++ b/src/southbridge/amd/agesa/hudson/hudson.c
@@ -39,7 +39,7 @@
void hudson_enable(struct device *dev)
{
- printk(BIOS_DEBUG, "hudson_enable()\n");
+ printk(BIOS_DEBUG, "%s()\n", __func__);
switch (dev->path.pci.devfn) {
case PCI_DEVFN(0x14, 5):
if (dev->enabled == 0) {
diff --git a/src/southbridge/amd/agesa/hudson/smi.c b/src/southbridge/amd/agesa/hudson/smi.c
index 567a3f8..9e6db34 100644
--- a/src/southbridge/amd/agesa/hudson/smi.c
+++ b/src/southbridge/amd/agesa/hudson/smi.c
@@ -12,7 +12,7 @@
void smm_setup_structures(void *gnvs, void *tcg, void *smi1)
{
- printk(BIOS_DEBUG, "smm_setup_structures STUB!!!\n");
+ printk(BIOS_DEBUG, "%s STUB!!!\n", __func__);
}
/** Set the EOS bit and enable SMI generation from southbridge */
diff --git a/src/southbridge/amd/cimx/sb800/late.c b/src/southbridge/amd/cimx/sb800/late.c
index cf84b21..1cf3ae8 100644
--- a/src/southbridge/amd/cimx/sb800/late.c
+++ b/src/southbridge/amd/cimx/sb800/late.c
@@ -108,7 +108,7 @@
static void lpc_init(struct device *dev)
{
- printk(BIOS_DEBUG, "SB800 - Late.c - lpc_init - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Late.c - %s - Start.\n", __func__);
cmos_check_update_date();
@@ -122,7 +122,7 @@
setup_i8259(); /* Initialize i8259 pic */
setup_i8254(); /* Initialize i8254 timers */
- printk(BIOS_DEBUG, "SB800 - Late.c - lpc_init - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Late.c - %s - End.\n", __func__);
}
unsigned long acpi_fill_mcfg(unsigned long current)
diff --git a/src/southbridge/amd/cimx/sb800/lpc.c b/src/southbridge/amd/cimx/sb800/lpc.c
index 9517d95..a082e0c 100644
--- a/src/southbridge/amd/cimx/sb800/lpc.c
+++ b/src/southbridge/amd/cimx/sb800/lpc.c
@@ -11,7 +11,7 @@
{
struct resource *res;
- printk(BIOS_DEBUG, "SB800 - Lpc.c - lpc_read_resources - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Lpc.c - %s - Start.\n", __func__);
/* Get the normal pci resources of this device */
pci_dev_read_resources(dev); /* We got one for APIC, or one more for TRAP */
@@ -37,14 +37,14 @@
res->flags = IORESOURCE_MEM | IORESOURCE_ASSIGNED | IORESOURCE_FIXED;
compact_resources(dev);
- printk(BIOS_DEBUG, "SB800 - Lpc.c - lpc_read_resources - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Lpc.c - %s - End.\n", __func__);
}
void lpc_set_resources(struct device *dev)
{
struct resource *res;
- printk(BIOS_DEBUG, "SB800 - Lpc.c - lpc_set_resources - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Lpc.c - %s - Start.\n", __func__);
/* Special case. SPI Base Address. The SpiRomEnable should STAY set. */
res = find_resource(dev, 2);
@@ -52,7 +52,7 @@
pci_dev_set_resources(dev);
- printk(BIOS_DEBUG, "SB800 - Lpc.c - lpc_set_resources - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Lpc.c - %s - End.\n", __func__);
}
/**
@@ -68,7 +68,7 @@
int var_num = 0;
u16 reg_var[3];
- printk(BIOS_DEBUG, "SB800 - Lpc.c - lpc_enable_childrens_resources - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Lpc.c - %s - Start.\n", __func__);
reg = pci_read_config32(dev, 0x44);
reg_x = pci_read_config32(dev, 0x48);
@@ -166,5 +166,5 @@
//pci_write_config16(dev, 0x64, reg_var[0]); //cause filo can not find sata
break;
}
- printk(BIOS_DEBUG, "SB800 - Lpc.c - lpc_enable_childrens_resources - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Lpc.c - %s - End.\n", __func__);
}
diff --git a/src/southbridge/amd/cimx/sb800/smbus.c b/src/southbridge/amd/cimx/sb800/smbus.c
index bc9d921..86bde26 100644
--- a/src/southbridge/amd/cimx/sb800/smbus.c
+++ b/src/southbridge/amd/cimx/sb800/smbus.c
@@ -50,11 +50,11 @@
u8 byte;
if (smbus_wait_until_ready(smbus_io_base) < 0) {
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_recv_byte - smbus not ready.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - smbus not ready.\n", __func__);
return -2; /* not ready */
}
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_recv_byte - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - Start.\n", __func__);
/* set the device I'm talking to */
outb(((device & 0x7f) << 1) | 1, smbus_io_base + SMBHSTADDR);
@@ -71,7 +71,7 @@
/* read results of transaction */
byte = inb(smbus_io_base + SMBHSTCMD);
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_recv_byte - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - End.\n", __func__);
return byte;
}
@@ -80,11 +80,11 @@
u8 byte;
if (smbus_wait_until_ready(smbus_io_base) < 0) {
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_send_byte - smbus not ready.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - smbus not ready.\n", __func__);
return -2; /* not ready */
}
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_send_byte - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - Start.\n", __func__);
/* set the command... */
outb(val, smbus_io_base + SMBHSTCMD);
@@ -101,7 +101,7 @@
return -3; /* timeout or error */
}
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_send_byte - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - End.\n", __func__);
return 0;
}
@@ -110,11 +110,11 @@
u8 byte;
if (smbus_wait_until_ready(smbus_io_base) < 0) {
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_read_byte - smbus not ready.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - smbus not ready.\n", __func__);
return -2; /* not ready */
}
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_read_byte - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - Start.\n", __func__);
/* set the command/address... */
outb(address & 0xff, smbus_io_base + SMBHSTCMD);
@@ -134,7 +134,7 @@
/* read results of transaction */
byte = inb(smbus_io_base + SMBHSTDAT0);
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_read_byte - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - End.\n", __func__);
return byte;
}
@@ -143,11 +143,11 @@
u8 byte;
if (smbus_wait_until_ready(smbus_io_base) < 0) {
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_write_byte - smbus not ready.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - smbus not ready.\n", __func__);
return -2; /* not ready */
}
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_write_byte - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - Start.\n", __func__);
/* set the command/address... */
outb(address & 0xff, smbus_io_base + SMBHSTCMD);
@@ -167,7 +167,7 @@
return -3; /* timeout or error */
}
- printk(BIOS_DEBUG, "SB800 - Smbus.c - do_smbus_write_byte - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - End.\n", __func__);
return 0;
}
@@ -175,7 +175,7 @@
{
u32 tmp;
- printk(BIOS_DEBUG, "SB800 - Smbus.c - alink_ab_indx - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - Start.\n", __func__);
outl((reg_space & 0x7) << 29 | reg_addr, AB_INDX);
tmp = inl(AB_DATA);
/* rpr 4.2
@@ -191,14 +191,14 @@
outl((reg_space & 0x7) << 29 | reg_addr, AB_INDX); /* probably we don't have to do it again. */
outl(tmp, AB_DATA);
outl(0, AB_INDX);
- printk(BIOS_DEBUG, "SB800 - Smbus.c - alink_ab_indx - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - End.\n", __func__);
}
void alink_rc_indx(u32 reg_space, u32 reg_addr, u32 port, u32 mask, u32 val)
{
u32 tmp;
- printk(BIOS_DEBUG, "SB800 - Smbus.c - alink_rc_indx - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - Start.\n", __func__);
outl((reg_space & 0x7) << 29 | (port & 3) << 24 | reg_addr, AB_INDX);
tmp = inl(AB_DATA);
/* rpr 4.2
@@ -214,7 +214,7 @@
outl((reg_space & 0x7) << 29 | (port & 3) << 24 | reg_addr, AB_INDX); /* probably we don't have to do it again. */
outl(tmp, AB_DATA);
outl(0, AB_INDX);
- printk(BIOS_DEBUG, "SB800 - Smbus.c - alink_rc_indx - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - End.\n", __func__);
}
/* space = 0: AX_INDXC, AX_DATAC
@@ -224,7 +224,7 @@
{
u32 tmp;
- printk(BIOS_DEBUG, "SB800 - Smbus.c - alink_ax_indx - Start.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - Start.\n", __func__);
/* read axindc to tmp */
outl(space << 29 | space << 3 | 0x30, AB_INDX);
outl(axindc, AB_DATA);
@@ -243,5 +243,5 @@
outl(space << 29 | space << 3 | 0x34, AB_INDX);
outl(tmp, AB_DATA);
outl(0, AB_INDX);
- printk(BIOS_DEBUG, "SB800 - Smbus.c - alink_ax_indx - End.\n");
+ printk(BIOS_DEBUG, "SB800 - Smbus.c - %s - End.\n", __func__);
}
diff --git a/src/southbridge/amd/pi/hudson/hudson.c b/src/southbridge/amd/pi/hudson/hudson.c
index 63fe473..852144b 100644
--- a/src/southbridge/amd/pi/hudson/hudson.c
+++ b/src/southbridge/amd/pi/hudson/hudson.c
@@ -26,7 +26,7 @@
void hudson_enable(struct device *dev)
{
- printk(BIOS_DEBUG, "hudson_enable()\n");
+ printk(BIOS_DEBUG, "%s()\n", __func__);
switch (dev->path.pci.devfn) {
case PCI_DEVFN(0x14, 7): /* SD */
if (dev->enabled == 0) {
diff --git a/src/southbridge/amd/pi/hudson/smi.c b/src/southbridge/amd/pi/hudson/smi.c
index 567a3f8..9e6db34 100644
--- a/src/southbridge/amd/pi/hudson/smi.c
+++ b/src/southbridge/amd/pi/hudson/smi.c
@@ -12,7 +12,7 @@
void smm_setup_structures(void *gnvs, void *tcg, void *smi1)
{
- printk(BIOS_DEBUG, "smm_setup_structures STUB!!!\n");
+ printk(BIOS_DEBUG, "%s STUB!!!\n", __func__);
}
/** Set the EOS bit and enable SMI generation from southbridge */