Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1721786 > unrolled thread

[PATCH] i40iw: make some structures const

Started byBhumika Goyal <bhumirks@gmail.com>
First post2017-08-28 18:30 +0200
Last post2017-08-29 15:30 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] i40iw: make some structures const Bhumika Goyal <bhumirks@gmail.com> - 2017-08-28 18:30 +0200
    Re: [PATCH] i40iw: make some structures const Doug Ledford <dledford@redhat.com> - 2017-08-29 01:20 +0200
    Re: [PATCH] i40iw: make some structures const Shiraz Saleem <shiraz.saleem@intel.com> - 2017-08-29 15:30 +0200

#1721786 — [PATCH] i40iw: make some structures const

FromBhumika Goyal <bhumirks@gmail.com>
Date2017-08-28 18:30 +0200
Subject[PATCH] i40iw: make some structures const
Message-ID<ujv5f-7QO-3@gated-at.bofh.it>
Make some structures const as they are only used during a copy
operation.

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
 drivers/infiniband/hw/i40iw/i40iw_uk.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/infiniband/hw/i40iw/i40iw_uk.c b/drivers/infiniband/hw/i40iw/i40iw_uk.c
index 1060725..0aadb7a 100644
--- a/drivers/infiniband/hw/i40iw/i40iw_uk.c
+++ b/drivers/infiniband/hw/i40iw/i40iw_uk.c
@@ -912,7 +912,7 @@ enum i40iw_status_code i40iw_get_wqe_shift(u32 wqdepth, u32 sge, u32 inline_data
 	return 0;
 }
 
-static struct i40iw_qp_uk_ops iw_qp_uk_ops = {
+static const struct i40iw_qp_uk_ops iw_qp_uk_ops = {
 	.iw_qp_post_wr = i40iw_qp_post_wr,
 	.iw_qp_ring_push_db = i40iw_qp_ring_push_db,
 	.iw_rdma_write = i40iw_rdma_write,
@@ -926,14 +926,14 @@ enum i40iw_status_code i40iw_get_wqe_shift(u32 wqdepth, u32 sge, u32 inline_data
 	.iw_post_nop = i40iw_nop
 };
 
-static struct i40iw_cq_ops iw_cq_ops = {
+static const struct i40iw_cq_ops iw_cq_ops = {
 	.iw_cq_request_notification = i40iw_cq_request_notification,
 	.iw_cq_poll_completion = i40iw_cq_poll_completion,
 	.iw_cq_post_entries = i40iw_cq_post_entries,
 	.iw_cq_clean = i40iw_clean_cq
 };
 
-static struct i40iw_device_uk_ops iw_device_uk_ops = {
+static const struct i40iw_device_uk_ops iw_device_uk_ops = {
 	.iwarp_cq_uk_init = i40iw_cq_uk_init,
 	.iwarp_qp_uk_init = i40iw_qp_uk_init,
 };
-- 
1.9.1

[toc] | [next] | [standalone]


#1722050

FromDoug Ledford <dledford@redhat.com>
Date2017-08-29 01:20 +0200
Message-ID<ujBu2-3iy-19@gated-at.bofh.it>
In reply to#1721786
On Mon, 2017-08-28 at 21:51 +0530, Bhumika Goyal wrote:
> Make some structures const as they are only used during a copy
> operation.
> 
> Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>

Thanks, applied.

-- 
Doug Ledford <dledford@redhat.com>
    GPG KeyID: B826A3330E572FDD
    Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

[toc] | [prev] | [next] | [standalone]


#1722492

FromShiraz Saleem <shiraz.saleem@intel.com>
Date2017-08-29 15:30 +0200
Message-ID<ujOKD-3av-27@gated-at.bofh.it>
In reply to#1721786
On Mon, Aug 28, 2017 at 09:51:23PM +0530, Bhumika Goyal wrote:
> Make some structures const as they are only used during a copy
> operation.
> 
> Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>

Thanks!

Acked-by: Shiraz Saleem <shiraz.saleem@intel.com>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web