diff mbox series

rdma/cxgb4: Remove a set-but-not-used variable

Message ID 20180731150815.14947-1-bart.vanassche@wdc.com (mailing list archive)
State Accepted
Delegated to: Jason Gunthorpe
Headers show
Series rdma/cxgb4: Remove a set-but-not-used variable | expand

Commit Message

Bart Van Assche July 31, 2018, 3:08 p.m. UTC
This patch avoids that the following warning is reported when building with
W=1:

drivers/infiniband/hw/cxgb4/cm.c:1860:5: warning: variable 'status' set but not used [-Wunused-but-set-variable]
  u8 status;
     ^~~~~~

Fixes: 6a0b6174d35a ("rdma/cxgb4: Add support for kernel mode SRQ's")
Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Cc: Steve Wise <swise@opengridcomputing.com>
Cc: Raju Rangoju <rajur@chelsio.com>
---
 drivers/infiniband/hw/cxgb4/cm.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Steve Wise July 31, 2018, 3:14 p.m. UTC | #1
On 7/31/2018 10:08 AM, Bart Van Assche wrote:
> This patch avoids that the following warning is reported when building with
> W=1:
>
> drivers/infiniband/hw/cxgb4/cm.c:1860:5: warning: variable 'status' set but not used [-Wunused-but-set-variable]
>   u8 status;
>      ^~~~~~
>
> Fixes: 6a0b6174d35a ("rdma/cxgb4: Add support for kernel mode SRQ's")
> Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
> Cc: Steve Wise <swise@opengridcomputing.com>
> Cc: Raju Rangoju <rajur@chelsio.com>

Thanks Bart,

Acked-by: Steve Wise <swise@opengridcomputing.com>

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jason Gunthorpe July 31, 2018, 10:59 p.m. UTC | #2
On Tue, Jul 31, 2018 at 08:08:15AM -0700, Bart Van Assche wrote:
> This patch avoids that the following warning is reported when building with
> W=1:
> 
> drivers/infiniband/hw/cxgb4/cm.c:1860:5: warning: variable 'status' set but not used [-Wunused-but-set-variable]
>   u8 status;
>      ^~~~~~
> 
> Fixes: 6a0b6174d35a ("rdma/cxgb4: Add support for kernel mode SRQ's")
> Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
> Cc: Steve Wise <swise@opengridcomputing.com>
> Cc: Raju Rangoju <rajur@chelsio.com>
> Acked-by: Steve Wise <swise@opengridcomputing.com>
> ---
>  drivers/infiniband/hw/cxgb4/cm.c | 2 --
>  1 file changed, 2 deletions(-)

Applied to for-next

Thanks,
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c
index 54f7fbef7880..d7cfa38baad2 100644
--- a/drivers/infiniband/hw/cxgb4/cm.c
+++ b/drivers/infiniband/hw/cxgb4/cm.c
@@ -1857,10 +1857,8 @@  static void complete_cached_srq_buffers(struct c4iw_ep *ep, u32 srqidx_status)
 {
 	enum chip_type adapter_type;
 	u32 srqidx;
-	u8 status;
 
 	adapter_type = ep->com.dev->rdev.lldi.adapter_type;
-	status = ABORT_RSS_STATUS_G(be32_to_cpu(srqidx_status));
 	srqidx = ABORT_RSS_SRQIDX_G(be32_to_cpu(srqidx_status));
 
 	/*