Message ID | 1476954305-576-7-git-send-email-sumit.saxena@broadcom.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers | show |
On Thu, Oct 20, 2016 at 02:05:04AM -0700, Sumit Saxena wrote: > CC: stable@vger.kernel.org > Signed-off-by: Sumit Saxena <sumit.saxena@broadcom.com> > Reviewed-by: Hannes Reinecke <hare@suse.com> > Reviewed-by: Tomas Henzl <thenzl@redhat.com> > --- > drivers/scsi/megaraid/megaraid_sas_fusion.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) I know I reject patches without any changelog text in them, hopefully the scsi maintainers also do... -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, 2016-10-20 at 11:15 +0200, Greg KH wrote: > On Thu, Oct 20, 2016 at 02:05:04AM -0700, Sumit Saxena wrote: > > CC: stable@vger.kernel.org > > Signed-off-by: Sumit Saxena <sumit.saxena@broadcom.com> > > Reviewed-by: Hannes Reinecke <hare@suse.com> > > Reviewed-by: Tomas Henzl <thenzl@redhat.com> > > --- > > drivers/scsi/megaraid/megaraid_sas_fusion.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > I know I reject patches without any changelog text in them, hopefully > the scsi maintainers also do... It depends: for complex patches needing detailed explanations, yes. However, this is a four line change for which the subject seems adequately descriptive, so this particular one seems fine. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/scsi/megaraid/megaraid_sas_fusion.c b/drivers/scsi/megaraid/megaraid_sas_fusion.c index 2e61306..24778ba 100644 --- a/drivers/scsi/megaraid/megaraid_sas_fusion.c +++ b/drivers/scsi/megaraid/megaraid_sas_fusion.c @@ -2005,6 +2005,8 @@ static void megasas_build_ld_nonrw_fusion(struct megasas_instance *instance, io_request->DevHandle = pd_sync->seq[pd_index].devHandle; pRAID_Context->regLockFlags |= (MR_RL_FLAGS_SEQ_NUM_ENABLE|MR_RL_FLAGS_GRANT_DESTINATION_CUDA); + pRAID_Context->Type = MPI2_TYPE_CUDA; + pRAID_Context->nseg = 0x1; } else if (fusion->fast_path_io) { pRAID_Context->VirtualDiskTgtId = cpu_to_le16(device_id); pRAID_Context->configSeqNum = 0; @@ -2040,12 +2042,10 @@ static void megasas_build_ld_nonrw_fusion(struct megasas_instance *instance, pRAID_Context->timeoutValue = cpu_to_le16((os_timeout_value > timeout_limit) ? timeout_limit : os_timeout_value); - if (fusion->adapter_type == INVADER_SERIES) { - pRAID_Context->Type = MPI2_TYPE_CUDA; - pRAID_Context->nseg = 0x1; + if (fusion->adapter_type == INVADER_SERIES) io_request->IoFlags |= cpu_to_le16(MPI25_SAS_DEVICE0_FLAGS_ENABLED_FAST_PATH); - } + cmd->request_desc->SCSIIO.RequestFlags = (MPI2_REQ_DESCRIPT_FLAGS_FP_IO << MEGASAS_REQ_DESCRIPT_FLAGS_TYPE_SHIFT);