linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] i40iw: remove unused 'timeval' struct member
@ 2017-11-27 11:38 Arnd Bergmann
  2017-11-27 11:38 ` [PATCH 2/2] nes: " Arnd Bergmann
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Arnd Bergmann @ 2017-11-27 11:38 UTC (permalink / raw)
  To: Faisal Latif, Shiraz Saleem, Doug Ledford, Jason Gunthorpe
  Cc: y2038, Arnd Bergmann, Mustafa Ismail, Henry Orosco,
	Tatyana Nikolova, linux-rdma, linux-kernel

There is a stale entry in i40iw_cm_tcp_context that apparently
was copied from the 'nes' driver but never used in i40iw.
I'm trying to kill off all users of timeval as part of the
y2038-safety work, so let's just remove this one.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/infiniband/hw/i40iw/i40iw_cm.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/infiniband/hw/i40iw/i40iw_cm.h b/drivers/infiniband/hw/i40iw/i40iw_cm.h
index 0d5840d2c4fc..1577608fc51a 100644
--- a/drivers/infiniband/hw/i40iw/i40iw_cm.h
+++ b/drivers/infiniband/hw/i40iw/i40iw_cm.h
@@ -276,8 +276,6 @@ struct i40iw_cm_tcp_context {
 	u32 mss;
 	u8 snd_wscale;
 	u8 rcv_wscale;
-
-	struct timeval sent_ts;
 };
 
 enum i40iw_cm_listener_state {
-- 
2.9.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] nes: remove unused 'timeval' struct member
  2017-11-27 11:38 [PATCH 1/2] i40iw: remove unused 'timeval' struct member Arnd Bergmann
@ 2017-11-27 11:38 ` Arnd Bergmann
  2017-11-28 18:37 ` [PATCH 1/2] i40iw: " Shiraz Saleem
  2017-12-11 23:38 ` [1/2] " Jason Gunthorpe
  2 siblings, 0 replies; 4+ messages in thread
From: Arnd Bergmann @ 2017-11-27 11:38 UTC (permalink / raw)
  To: Faisal Latif, Doug Ledford, Jason Gunthorpe
  Cc: y2038, Arnd Bergmann, linux-rdma, linux-kernel

There is a stale entry in nes_cm_tcp_context that has apparently
never been used in mainline linux.

I'm trying to kill off all users of timeval as part of the
y2038-safety work, so let's just remove this one.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/infiniband/hw/nes/nes_cm.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/infiniband/hw/nes/nes_cm.h b/drivers/infiniband/hw/nes/nes_cm.h
index d827d03e3941..2cf2641867d8 100644
--- a/drivers/infiniband/hw/nes/nes_cm.h
+++ b/drivers/infiniband/hw/nes/nes_cm.h
@@ -279,7 +279,6 @@ struct nes_cm_tcp_context {
 	u8  rcv_wscale;
 
 	struct nes_cm_tsa_context tsa_cntxt;
-	struct timeval            sent_ts;
 };
 
 
-- 
2.9.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] i40iw: remove unused 'timeval' struct member
  2017-11-27 11:38 [PATCH 1/2] i40iw: remove unused 'timeval' struct member Arnd Bergmann
  2017-11-27 11:38 ` [PATCH 2/2] nes: " Arnd Bergmann
@ 2017-11-28 18:37 ` Shiraz Saleem
  2017-12-11 23:38 ` [1/2] " Jason Gunthorpe
  2 siblings, 0 replies; 4+ messages in thread
From: Shiraz Saleem @ 2017-11-28 18:37 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Latif, Faisal, Doug Ledford, Jason Gunthorpe, y2038, Ismail,
	Mustafa, Orosco, Henry, Nikolova, Tatyana E, linux-rdma,
	linux-kernel

On Mon, Nov 27, 2017 at 04:38:09AM -0700, Arnd Bergmann wrote:
> There is a stale entry in i40iw_cm_tcp_context that apparently
> was copied from the 'nes' driver but never used in i40iw.
> I'm trying to kill off all users of timeval as part of the
> y2038-safety work, so let's just remove this one.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---

Thanks!

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [1/2] i40iw: remove unused 'timeval' struct member
  2017-11-27 11:38 [PATCH 1/2] i40iw: remove unused 'timeval' struct member Arnd Bergmann
  2017-11-27 11:38 ` [PATCH 2/2] nes: " Arnd Bergmann
  2017-11-28 18:37 ` [PATCH 1/2] i40iw: " Shiraz Saleem
@ 2017-12-11 23:38 ` Jason Gunthorpe
  2 siblings, 0 replies; 4+ messages in thread
From: Jason Gunthorpe @ 2017-12-11 23:38 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Faisal Latif, Shiraz Saleem, Doug Ledford, y2038, Mustafa Ismail,
	Henry Orosco, Tatyana Nikolova, linux-rdma, linux-kernel

On Mon, Nov 27, 2017 at 12:38:09PM +0100, Arnd Bergmann wrote:
> There is a stale entry in i40iw_cm_tcp_context that apparently
> was copied from the 'nes' driver but never used in i40iw.
> I'm trying to kill off all users of timeval as part of the
> y2038-safety work, so let's just remove this one.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Acked-by: Shiraz Saleem <shiraz.saleem@intel.com>
>  drivers/infiniband/hw/i40iw/i40iw_cm.h | 2 --
>  1 file changed, 2 deletions(-)

Applied both of these patches in the series to for-next

Thanks,
Jason

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-12-11 23:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-27 11:38 [PATCH 1/2] i40iw: remove unused 'timeval' struct member Arnd Bergmann
2017-11-27 11:38 ` [PATCH 2/2] nes: " Arnd Bergmann
2017-11-28 18:37 ` [PATCH 1/2] i40iw: " Shiraz Saleem
2017-12-11 23:38 ` [1/2] " Jason Gunthorpe

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).