Commit d55215c5 authored Jun 03, 2016 by Bart Van Assche Committed by Doug Ledford Jun 06, 2016

IB/hfi1: Use bit 0 instead of bit 1

The first argument of test_bit() and clear_bit() is a bit number and
not a bitmask. Hence change that first argument from (1 << 0) into 0.
This patch avoids that smatch reports the following warnings:

user_sdma.c:1059: sdma_cache_evict() warn: test_bit() takes a bit number
user_sdma.c:1590: sdma_rb_remove() warn: test_bit() takes a bit number

Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
Cc: Mike Marciniszyn <mike.marciniszyn@intel.com>
Cc: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>

parent 48a0cc13

drivers/infiniband/hw/hfi1/user_sdma.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -183,7 +183,7 @@ struct user_sdma_iovec {
		struct sdma_mmu_node *node;
		};

		#define SDMA_CACHE_NODE_EVICT BIT(0)
		#define SDMA_CACHE_NODE_EVICT 0

		struct sdma_mmu_node {
		struct mmu_rb_node rb;