diff mbox series

RDMA/hfi1: remove redundant assignment to pointer ppd

Message ID 20231023141733.667807-1-colin.i.king@gmail.com (mailing list archive)
State Accepted
Headers show
Series RDMA/hfi1: remove redundant assignment to pointer ppd | expand

Commit Message

Colin Ian King Oct. 23, 2023, 2:17 p.m. UTC
Pointer ppd is being assigned a value in a for-loop however it
is never read. The assignment is redundant and can be removed.
Cleans up clang scan build warning:

drivers/infiniband/hw/hfi1/init.c:1030:3: warning: Value stored
to 'ppd' is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
 drivers/infiniband/hw/hfi1/init.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Leon Romanovsky Oct. 24, 2023, 3:16 p.m. UTC | #1
On Mon, 23 Oct 2023 15:17:33 +0100, Colin Ian King wrote:
> Pointer ppd is being assigned a value in a for-loop however it
> is never read. The assignment is redundant and can be removed.
> Cleans up clang scan build warning:
> 
> drivers/infiniband/hw/hfi1/init.c:1030:3: warning: Value stored
> to 'ppd' is never read [deadcode.DeadStores]
> 
> [...]

Applied, thanks!

[1/1] RDMA/hfi1: remove redundant assignment to pointer ppd
      https://git.kernel.org/rdma/rdma/c/b55706366c5ed6

Best regards,
diff mbox series

Patch

diff --git a/drivers/infiniband/hw/hfi1/init.c b/drivers/infiniband/hw/hfi1/init.c
index 5ce2215e09c2..cbac4a442d9e 100644
--- a/drivers/infiniband/hw/hfi1/init.c
+++ b/drivers/infiniband/hw/hfi1/init.c
@@ -1027,7 +1027,6 @@  static void shutdown_device(struct hfi1_devdata *dd)
 	msix_clean_up_interrupts(dd);
 
 	for (pidx = 0; pidx < dd->num_pports; ++pidx) {
-		ppd = dd->pport + pidx;
 		for (i = 0; i < dd->num_rcv_contexts; i++) {
 			rcd = hfi1_rcd_get_by_index(dd, i);
 			hfi1_rcvctrl(dd, HFI1_RCVCTRL_TAILUPD_DIS |