On Wed, 2012-08-22 at 16:55 +0530, sreekanth.re...@lsi.com wrote:
> Fix for issue - Unable to boot from the SAS drive connected to HBA
> 
> This patch checks whether HBA is SAS2008 B0 controller.
> if it is a SAS2008 B0 controller then it use IO-APIC interrupt instead of 
> MSIX,
> as SAS2008 B0 controller doesn't support MSIX interrupts.
> 
> Signed-off-by: Sreekanth Reddy <sreekanth.re...@lsi.com>
> Cc: <sta...@kernel.org>

Checkpatch, next time, please!  This has a nasty case of spaces
converted to tabs:

ERROR: code indent should use tabs where possible
#24: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1213:
+           if it is SAS2008 B0 controller use IO-APIC instead of MSIX
*/$

ERROR: code indent should use tabs where possible
#25: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1214:
+        if (ioc->pdev->device == MPI2_MFGPAGE_DEVID_SAS2008 &&$

WARNING: please, no spaces at the start of a line
#25: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1214:
+        if (ioc->pdev->device == MPI2_MFGPAGE_DEVID_SAS2008 &&$

ERROR: code indent should use tabs where possible
#26: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1215:
+            ioc->pdev->revision == 0x01) {$

WARNING: please, no spaces at the start of a line
#26: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1215:
+            ioc->pdev->revision == 0x01) {$

ERROR: code indent should use tabs where possible
#27: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1216:
+                return -EINVAL;$

WARNING: please, no spaces at the start of a line
#27: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1216:
+                return -EINVAL;$

ERROR: code indent should use tabs where possible
#28: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1217:
+        }$

WARNING: please, no spaces at the start of a line
#28: FILE: drivers/scsi/mpt2sas/mpt2sas_base.c:1217:
+        }$

total: 5 errors, 4 warnings, 13 lines checked

NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch
or
      scripts/cleanfile

James


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to