Revert "ACPI: Suppress misleading SPCR console message when SPCR table is absent"

This reverts commit bad3fa2fb9206f4dcec6ddef094ec2fbf6e8dcb2.

Commit bad3fa2fb920 ("ACPI: Suppress misleading SPCR console message
when SPCR table is absent") mistakenly assumes acpi_parse_spcr()
returning 0 to indicate a failure to parse SPCR. While addressing the
resultant incorrect logging it was deemed that dropping the message is
a better approach as it is not particularly useful.

Roll back the commit introducing the bug as a step towards dropping
the log message.

Link: https://lore.kernel.org/all/aQN0YWUYaPYWpgJM@willie-the-truck/
Signed-off-by: Punit Agrawal <punit.agrawal@oss.qualcomm.com>
Signed-off-by: Will Deacon <will@kernel.org>

diff --git a/arch/arm64/kernel/acpi.c b/arch/arm64/kernel/acpi.c
index 7aca29e1d30b2499fb9f276e8fde25f1fc9846e5..fd164e8a35b2d20718877bd39474202cf635ebd5 100644 (file)
--- a/arch/arm64/kernel/acpi.c
+++ b/arch/arm64/kernel/acpi.c
@@ -197,8 +197,6 @@ out:
  */
 void __init acpi_boot_table_init(void)
 {
-       int ret;
-
        /*
         * Enable ACPI instead of device tree unless
         * - ACPI has been disabled explicitly (acpi=off), or
@@ -252,12 +250,10 @@ done:
                 * behaviour, use acpi=nospcr to disable console in ACPI SPCR
                 * table as default serial console.
                 */
-               ret = acpi_parse_spcr(earlycon_acpi_spcr_enable,
+               acpi_parse_spcr(earlycon_acpi_spcr_enable,
                        !param_acpi_nospcr);
-               if (!ret || param_acpi_nospcr || !IS_ENABLED(CONFIG_ACPI_SPCR_TABLE))
-                       pr_info("Use ACPI SPCR as default console: No\n");
-               else
-                       pr_info("Use ACPI SPCR as default console: Yes\n");
+               pr_info("Use ACPI SPCR as default console: %s\n",
+                               param_acpi_nospcr ? "No" : "Yes");
 
                if (IS_ENABLED(CONFIG_ACPI_BGRT))
                        acpi_table_parse(ACPI_SIG_BGRT, acpi_parse_bgrt);