Messages in this thread | | | Date | Fri, 18 Mar 2022 11:40:55 -0600 | From | Keith Busch <> | Subject | Re: [PATCH] nvmet: remove redundant assignment after left shift |
| |
On Fri, Mar 18, 2022 at 01:30:14AM +0000, Colin Ian King wrote: > The left shift is followed by a re-assignment back to cc_css, > the assignment is redundant. Fix this by replacing the <<= > operator with << instead. > > Cleans up clang scan build warning: > drivers/nvme/target/core.c:1124:10: warning: Although the value > stored to 'cc_css' is used in the enclosing expression, the > value is never actually read from 'cc_css' [deadcode.DeadStores]
Looks good.
Reviewed-by: Keith Busch <kbusch@kernel.org>
| |