Messages in this thread | | | From | Chaitanya Kulkarni <> | Subject | Re: [PATCH] nvmet: remove redundant assignment after left shift | Date | Mon, 21 Mar 2022 07:17:02 +0000 |
| |
On 3/17/22 18:30, 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] > > Signed-off-by: Colin Ian King <colin.i.king@gmail.com> >
Thanks, looks good.
Reviewed-by: Chaitanya Kulkarni <kch@nvidia.com>
-ck
| |