On Wed, 2012-08-22 at 16:55 +0530, sreekanth.reddy@xxxxxxx 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.reddy@xxxxxxx>
> Cc: <stable@xxxxxxxxxx>
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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
