Re: [PATCH 02/29] qla2xxx: Add support for ISP8044.

2013-08-28 Thread James Bottomley
On Tue, 2013-08-27 at 01:37 -0400, Saurav Kashyap wrote: From: Atul Deshmukh atul.deshm...@qlogic.com You seem to have stopped running your stuff through checkpatch: WARNING: please, no spaces at the start of a line #317: FILE: drivers/scsi/qla2xxx/qla_gbl.h:479: +uint8_t *, uint32_t,

Re: [PATCH 02/29] qla2xxx: Add support for ISP8044.

2013-08-28 Thread Saurav Kashyap
Hi James, Comments inline.. On Tue, 2013-08-27 at 01:37 -0400, Saurav Kashyap wrote: From: Atul Deshmukh atul.deshm...@qlogic.com You seem to have stopped running your stuff through checkpatch: SK We did run all the patches through checkpatch before submission, did see the warnings and was