It means firmware attempts to immediately process or launch the capsule
when flags in capsule header is not set. Moreover, reset is not needed
in this case. Current code will output log to indicate reset.

This patch adds a branch to avoid reset log output when the flags is not
set.

Cc: Joey Zheng <yu.zh...@hxt-semitech.com>
Signed-off-by: Shunyong Yang <shunyong.y...@hxt-semitech.com>
---
 drivers/firmware/efi/capsule-loader.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/drivers/firmware/efi/capsule-loader.c 
b/drivers/firmware/efi/capsule-loader.c
index e456f4602df1..a63b8e5bde23 100644
--- a/drivers/firmware/efi/capsule-loader.c
+++ b/drivers/firmware/efi/capsule-loader.c
@@ -134,10 +134,15 @@ static ssize_t efi_capsule_submit_update(struct 
capsule_info *cap_info)
 
        /* Indicate capsule binary uploading is done */
        cap_info->index = NO_FURTHER_WRITE_ACTION;
-       pr_info("Successfully upload capsule file with reboot type '%s'\n",
-               !cap_info->reset_type ? "RESET_COLD" :
-               cap_info->reset_type == 1 ? "RESET_WARM" :
-               "RESET_SHUTDOWN");
+
+       if (cap_info->header.flags)
+               pr_info("Successfully upload capsule file with reboot type 
'%s'\n",
+                       !cap_info->reset_type ? "RESET_COLD" :
+                       cap_info->reset_type == 1 ? "RESET_WARM" :
+                       "RESET_SHUTDOWN");
+       else
+               pr_info("Successfully upload, process and launch capsule 
file\n");
+
        return 0;
 }
 
-- 
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to