The advent of DIF Type 2 devices exposed these missing break statements. Signed-off-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx> --- diff --git a/include/scsi/scsi_host.h b/include/scsi/scsi_host.h index 6e728b1..7220125 100644 --- a/include/scsi/scsi_host.h +++ b/include/scsi/scsi_host.h @@ -801,12 +801,15 @@ static inline unsigned int scsi_host_dif_capable(struct Scsi_Host *shost, unsign case 1: if (shost->prot_capabilities & SHOST_DIF_TYPE1_PROTECTION) return target_type; + break; case 2: if (shost->prot_capabilities & SHOST_DIF_TYPE2_PROTECTION) return target_type; + break; case 3: if (shost->prot_capabilities & SHOST_DIF_TYPE3_PROTECTION) return target_type; + break; } return 0; -- 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
