platform/chrome: cros_ec_proto: don't show MKBP version if unsupported
authorTzung-Bi Shih <tzungbi@kernel.org>
Thu, 9 Jun 2022 08:49:49 +0000 (08:49 +0000)
committerTzung-Bi Shih <tzungbi@kernel.org>
Fri, 10 Jun 2022 02:31:43 +0000 (02:31 +0000)
It wrongly showed the following message when it doesn't support MKBP:
"MKBP support version 4294967295".

Fix it.

Reviewed-by: Guenter Roeck <groeck@chromium.org>
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Link: https://lore.kernel.org/r/20220609084957.3684698-14-tzungbi@kernel.org
drivers/platform/chrome/cros_ec_proto.c

index 60b8ef980b8af7a36c13269ec056465aca2d2edf..630d7ee5392a423d4dd67803b24c356f4691b750 100644 (file)
@@ -512,13 +512,13 @@ int cros_ec_query_all(struct cros_ec_device *ec_dev)
        ret = cros_ec_get_host_command_version_mask(ec_dev,
                                                    EC_CMD_GET_NEXT_EVENT,
                                                    &ver_mask);
-       if (ret < 0 || ver_mask == 0)
+       if (ret < 0 || ver_mask == 0) {
                ec_dev->mkbp_event_supported = 0;
-       else
+       } else {
                ec_dev->mkbp_event_supported = fls(ver_mask);
 
-       dev_dbg(ec_dev->dev, "MKBP support version %u\n",
-               ec_dev->mkbp_event_supported - 1);
+               dev_dbg(ec_dev->dev, "MKBP support version %u\n", ec_dev->mkbp_event_supported - 1);
+       }
 
        /* Probe if host sleep v1 is supported for S0ix failure detection. */
        ret = cros_ec_get_host_command_version_mask(ec_dev,