x86: Silence some logging statements
authorSimon Glass <sjg@chromium.org>
Wed, 4 Nov 2020 16:57:39 +0000 (09:57 -0700)
committerBin Meng <bmeng.cn@gmail.com>
Fri, 6 Nov 2020 01:51:32 +0000 (09:51 +0800)
Quite a few log_info() calls are included in the x86 code which should use
log_debug() instead. Convert them to reduce unwanted output.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
arch/x86/cpu/apollolake/fsp_s.c
arch/x86/cpu/intel_common/intel_opregion.c
arch/x86/cpu/intel_common/itss.c
arch/x86/lib/acpi_nhlt.c
arch/x86/lib/fsp/fsp_graphics.c

index 715ceab..2881880 100644 (file)
@@ -116,10 +116,10 @@ static int set_power_limits(struct udevice *dev)
 
        /* Program package power limits in RAPL MSR */
        msr_write(MSR_PKG_POWER_LIMIT, limit);
-       log_info("RAPL PL1 %d.%dW\n", tdp / power_unit,
-                100 * (tdp % power_unit) / power_unit);
-       log_info("RAPL PL2 %d.%dW\n", pl2_val / power_unit,
-                100 * (pl2_val % power_unit) / power_unit);
+       log_debug("RAPL PL1 %d.%dW\n", tdp / power_unit,
+                 100 * (tdp % power_unit) / power_unit);
+       log_debug("RAPL PL2 %d.%dW\n", pl2_val / power_unit,
+                 100 * (pl2_val % power_unit) / power_unit);
 
        /*
         * Sett RAPL MMIO register for Power limits. RAPL driver is using MSR
index c95ae04..1eed21d 100644 (file)
@@ -42,7 +42,7 @@ static int locate_vbt(char **vbtp, int *sizep)
                return -EINVAL;
        }
 
-       log_info("Found a VBT of %u bytes\n", size);
+       log_debug("Found a VBT of %u bytes\n", size);
        *sizep = size;
        *vbtp = vbt_data;
 
index 53dd09d..de17b93 100644 (file)
@@ -67,7 +67,7 @@ static int snapshot_polarities(struct udevice *dev)
        reg_start = start / IRQS_PER_IPC;
        reg_end = DIV_ROUND_UP(end, IRQS_PER_IPC);
 
-       log_info("ITSS IRQ Polarities snapshot %p\n", priv->irq_snapshot);
+       log_debug("ITSS IRQ Polarities snapshot %p\n", priv->irq_snapshot);
        for (i = reg_start; i < reg_end; i++) {
                uint reg = PCR_ITSS_IPC0_CONF + sizeof(u32) * i;
 
@@ -89,11 +89,11 @@ static void show_polarities(struct udevice *dev, const char *msg)
 {
        int i;
 
-       log_info("ITSS IRQ Polarities %s:\n", msg);
+       log_debug("ITSS IRQ Polarities %s:\n", msg);
        for (i = 0; i < NUM_IPC_REGS; i++) {
                uint reg = PCR_ITSS_IPC0_CONF + sizeof(u32) * i;
 
-               log_info("IPC%d: 0x%08x\n", i, pcr_read32(dev, reg));
+               log_debug("IPC%d: 0x%08x\n", i, pcr_read32(dev, reg));
        }
 }
 
@@ -115,7 +115,7 @@ static int restore_polarities(struct udevice *dev)
               sizeof(priv->irq_snapshot));
 
        show_polarities(dev, "Before");
-       log_info("priv->irq_snapshot %p\n", priv->irq_snapshot);
+       log_debug("priv->irq_snapshot %p\n", priv->irq_snapshot);
 
        reg_start = start / IRQS_PER_IPC;
        reg_end = DIV_ROUND_UP(end, IRQS_PER_IPC);
index 0e8e669..6c8cd83 100644 (file)
@@ -390,7 +390,7 @@ int nhlt_serialise_oem_overrides(struct acpi_ctx *ctx, struct nhlt *nhlt,
        size_t oem_table_id_len;
        int ret;
 
-       log_info("ACPI:    * NHLT\n");
+       log_debug("ACPI:    * NHLT\n");
        sz = nhlt_current_size(nhlt);
 
        /* Create header */
index 858d794..6534b66 100644 (file)
@@ -139,7 +139,7 @@ static int fsp_video_acpi_write_tables(const struct udevice *dev,
        struct igd_opregion *opregion;
        int ret;
 
-       printf("ACPI:    * IGD OpRegion\n");
+       log_debug("ACPI:    * IGD OpRegion\n");
        opregion = (struct igd_opregion *)ctx->current;
 
        ret = intel_gma_init_igd_opregion((struct udevice *)dev, opregion);