RE: [PATCH 0/5] bfa: fix W=1 build warnings

2016-08-10 Thread Sudarsana Kalluru
Gurumurthy ; Sudarsana Kalluru Cc: James E . J . Bottomley ; Martin K . Petersen ; linux-scsi ; linux-kernel ; Baoyou Xie ; Arnd Bergmann Subject: [PATCH 0/5] bfa: fix W=1 build warnings I tried to build an allmodconfig kernel with W=1, and the bfa driver stuck out for having the most warnings

RE: [PATCH 1/5] bfa: mark symbols static where possible

2016-08-10 Thread Sudarsana Kalluru
*bfad_im_scsi_transport_template; extern struct scsi_transport_template *bfad_im_scsi_vport_transport_template; Thanks, Sudarsana -Original Message- From: Arnd Bergmann [mailto:a...@arndb.de] Sent: 02 August 2016 20:53 To: Anil Gurumurthy ; Sudarsana Kalluru Cc: James E . J . Bottomley ; Martin K

RE: [patch] bfa: clean up some bounds checking

2016-07-01 Thread Sudarsana Kalluru
Both the approaches (i.e., using a loop or the strchr()) look fine to me. Thanks, Sudarsana -Original Message- From: walter harms [mailto:wha...@bfs.de] Sent: 16 June 2016 17:50 To: Dan Carpenter Cc: Anil Gurumurthy ; Sudarsana Kalluru ; James E.J. Bottomley ; Martin K. Petersen

RE: [PATCH 5/11] drivers/scsi/bfa/bfad_bsg.c: Remove useless return variables

2014-06-10 Thread Sudarsana Kalluru
Thanks for the patch. Acked-by: Sudarsana Kalluru -Original Message- From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi-ow...@vger.kernel.org] On Behalf Of Peter Senna Tschudin Sent: 31 May 2014 18:44 To: Anil Gurumurthy Cc: kernel-janit...@vger.kernel.org; Sudarsana Kalluru