platform/x86/amd/hsmp: improve the error log
authorSuma Hegde <suma.hegde@amd.com>
Tue, 10 Oct 2023 12:03:10 +0000 (12:03 +0000)
committerIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Thu, 12 Oct 2023 13:31:52 +0000 (16:31 +0300)
1. Change print message during platform init to a more meaningful
   clear message.
2. Return the error code returned by hsmp_test() itself, rather then
   returning a common EOPNOTSUPP error.

Signed-off-by: Suma Hegde <suma.hegde@amd.com>
Reviewed-by: Naveen Krishna Chatradhi <nchatrad@amd.com>
Link: https://lore.kernel.org/r/20231010120310.3464066-3-suma.hegde@amd.com
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
drivers/platform/x86/amd/hsmp.c

index af8adfbd936946c65a43fed4b1040588d11341da..b056a5c8654a2613561707ec3972e899592d233b 100644 (file)
@@ -564,10 +564,10 @@ static int __init hsmp_plt_init(void)
        for (i = 0; i < plat_dev.num_sockets; i++) {
                ret = hsmp_test(i, 0xDEADBEEF);
                if (ret) {
-                       pr_err("HSMP is not supported on Fam:%x model:%x\n",
+                       pr_err("HSMP test message failed on Fam:%x model:%x\n",
                               boot_cpu_data.x86, boot_cpu_data.x86_model);
-                       pr_err("Or Is HSMP disabled in BIOS ?\n");
-                       return -EOPNOTSUPP;
+                       pr_err("Is HSMP disabled in BIOS ?\n");
+                       return ret;
                }
        }