RDMA/qedr: Endianness warnings cleanup
authorAlok Prasad <palok@marvell.com>
Thu, 1 Oct 2020 10:09:59 +0000 (10:09 +0000)
committerJason Gunthorpe <jgg@nvidia.com>
Thu, 1 Oct 2020 23:50:37 +0000 (20:50 -0300)
Making a change to fix following sparse warnings reported by kbuild bot.

 CHECK   drivers/infiniband/hw/qedr/verbs.c

drivers/infiniband/hw/qedr/verbs.c:3872:59: warning: incorrect type in assignment (different base types)
drivers/infiniband/hw/qedr/verbs.c:3872:59:    expected restricted __le32 [usertype] sge_prod
drivers/infiniband/hw/qedr/verbs.c:3872:59:    got unsigned int [usertype] sge_prod
drivers/infiniband/hw/qedr/verbs.c:3875:59: warning: incorrect type in assignment (different base types)
drivers/infiniband/hw/qedr/verbs.c:3875:59:    expected restricted __le32 [usertype] wqe_prod
drivers/infiniband/hw/qedr/verbs.c:3875:59:    got unsigned int [usertype] wqe_prod

Link: https://lore.kernel.org/r/20201001100959.19940-1-palok@marvell.com
Reported-by: kbuild test robot <lkp@intel.com>
Fixes: acca72e2b031 ("RDMA/qedr: SRQ's bug fixes")
Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
Signed-off-by: Michal Kalderon <mkalderon@marvell.com>
Signed-off-by: Alok Prasad <palok@marvell.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/qedr/verbs.c

index 23559f1fe96e1c7c3b6ffc135c386f588d22d704..b5603b3ed6a4ebc1bd359e4d9c2ae82ad215bffe 100644 (file)
@@ -3861,10 +3861,10 @@ int qedr_post_srq_recv(struct ib_srq *ibsrq, const struct ib_recv_wr *wr,
                 * in first 4 bytes and need to update WQE producer in
                 * next 4 bytes.
                 */
-               srq->hw_srq.virt_prod_pair_addr->sge_prod = hw_srq->sge_prod;
+               srq->hw_srq.virt_prod_pair_addr->sge_prod = cpu_to_le32(hw_srq->sge_prod);
                /* Make sure sge producer is updated first */
                dma_wmb();
-               srq->hw_srq.virt_prod_pair_addr->wqe_prod = hw_srq->wqe_prod;
+               srq->hw_srq.virt_prod_pair_addr->wqe_prod = cpu_to_le32(hw_srq->wqe_prod);
 
                wr = wr->next;
        }