scsi: fix operator precedence warning
authorRandy Dunlap <[email protected]>
Thu, 22 Apr 2010 18:02:14 +0000 (11:02 -0700)
committerLinus Torvalds <[email protected]>
Thu, 22 Apr 2010 18:11:02 +0000 (11:11 -0700)
Fix operator precedence warning (from sparse), which results in the
data value always being 0:

drivers/scsi/qla4xxx/ql4_mbx.c:470:66: warning: right shift by bigger than source value

Signed-off-by: Randy Dunlap <[email protected]>
Acked-by: Ravi Anand <[email protected]>
Cc: David C Somayajulu <[email protected]>
Cc: Karen Higgins <[email protected]>
Cc: Vikas Chaudhary <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
drivers/scsi/qla4xxx/ql4_mbx.c

index 09d6d4b76f39536512b3578142b26a1a0f27bcf8..caeb7d10ae0450b1cb8227f7f8c9fbc50b3f3664 100644 (file)
@@ -467,7 +467,7 @@ int qla4xxx_get_fwddb_entry(struct scsi_qla_host *ha,
        if (conn_err_detail)
                *conn_err_detail = mbox_sts[5];
        if (tcp_source_port_num)
-               *tcp_source_port_num = (uint16_t) mbox_sts[6] >> 16;
+               *tcp_source_port_num = (uint16_t) (mbox_sts[6] >> 16);
        if (connection_id)
                *connection_id = (uint16_t) mbox_sts[6] & 0x00FF;
        status = QLA_SUCCESS;