diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-11-16 16:08:34 +0100 |
|---|---|---|
| committer | Moyster <oysterized@gmail.com> | 2017-06-17 15:49:07 +0200 |
| commit | d73062b98c447e5651b98e6d4b46ad40fd4e7f05 (patch) | |
| tree | c19e674d4eec3fba6a6c381d83c7bd5fb4146e9d /kernel/workqueue.c | |
| parent | 4b4e264ccb1d17e42e29f16f84f77b8f522b8d72 (diff) | |
scsi: mvsas: fix command_active typo
commit af15769ffab13d777e55fdef09d0762bf0c249c4 upstream.
gcc-7 notices that the condition in mvs_94xx_command_active looks
suspicious:
drivers/scsi/mvsas/mv_94xx.c: In function 'mvs_94xx_command_active':
drivers/scsi/mvsas/mv_94xx.c:671:15: error: '<<' in boolean context, did you mean '<' ? [-Werror=int-in-bool-context]
This was introduced when the mv_printk() statement got added, and leads
to the condition being ignored. This is probably harmless.
Changing '&&' to '&' makes the code look reasonable, as we check the
command bit before setting and printing it.
Fixes: a4632aae8b66 ("[SCSI] mvsas: Add new macros and functions")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Willy Tarreau <w@1wt.eu>
Diffstat (limited to 'kernel/workqueue.c')
0 files changed, 0 insertions, 0 deletions