Currently, logical and is being used instead of *bitwise* and.

Fix this by using a proper bitwise and operator.

Addresses-Coverity-ID: 1468455 ("Logical vs. bitwise operator")
Fixes: 64f7c494a3c0 ("typec: tcpm: Add support for sink PPS related
messages")
Signed-off-by: Gustavo A. R. Silva <gust...@embeddedor.com>
---
 drivers/usb/typec/tcpm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/typec/tcpm.c b/drivers/usb/typec/tcpm.c
index 1ee259b..7ee417a 100644
--- a/drivers/usb/typec/tcpm.c
+++ b/drivers/usb/typec/tcpm.c
@@ -1772,7 +1772,7 @@ static void tcpm_pd_ext_msg_request(struct tcpm_port 
*port,
        enum pd_ext_msg_type type = pd_header_type_le(msg->header);
        unsigned int data_size = 
pd_ext_header_data_size_le(msg->ext_msg.header);
 
-       if (!(msg->ext_msg.header && PD_EXT_HDR_CHUNKED)) {
+       if (!(msg->ext_msg.header & PD_EXT_HDR_CHUNKED)) {
                tcpm_log(port, "Unchunked extended messages unsupported");
                return;
        }
-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" 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