[PATCH 06/14] drivers/firmware/sdei: Drop redundant error message in sdei_probe()

Gavin Shan gshan at redhat.com
Mon Jul 6 01:47:24 EDT 2020


This drops the redundant error message in sdei_probe() because the
situation can be indicated by the error number from next error
message.

The function scoped variables are reorder according to their
importance by the way.

Signed-off-by: Gavin Shan <gshan at redhat.com>
---
 drivers/firmware/arm_sdei.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/firmware/arm_sdei.c b/drivers/firmware/arm_sdei.c
index ca0e3f5eb907..28bc712ba71d 100644
--- a/drivers/firmware/arm_sdei.c
+++ b/drivers/firmware/arm_sdei.c
@@ -935,17 +935,15 @@ int sdei_unregister_ghes(struct ghes *ghes)
 
 static int sdei_probe(struct platform_device *pdev)
 {
-	int err;
-	u64 ver = 0;
 	int conduit;
+	u64 ver = 0;
+	int err;
 
 	conduit = acpi_psci_use_hvc() ? SMCCC_CONDUIT_HVC : SMCCC_CONDUIT_SMC;
 	sdei_firmware_call = acpi_psci_use_hvc() ?
 			     &sdei_smccc_hvc : &sdei_smccc_smc;
 
 	err = sdei_api_get_version(&ver);
-	if (err == -EOPNOTSUPP)
-		pr_err("advertised but not implemented in platform firmware\n");
 	if (err) {
 		pr_err("Failed to get SDEI version: %d\n", err);
 		sdei_mark_interface_broken();
-- 
2.23.0




More information about the linux-arm-kernel mailing list