diff mbox

[2/2] ipr: Enable SIS pipe commands for SIS-32 devices.

Message ID 1439995626-16202-2-git-send-email-krisman@linux.vnet.ibm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Gabriel Krisman Bertazi Aug. 19, 2015, 2:47 p.m. UTC
Remove unnecessary check that disabled SIS pipe commands for SIS-32
devices.  This change was sufficient to enable raw mode and send SIS
pipe commands for a 57B3 device.

Fixes: f8ee25d7d239 ("ipr: AF DASD raw mode implementation in ipr driver")
Signed-off-by: Gabriel Krisman Bertazi <krisman@linux.vnet.ibm.com>
---
 drivers/scsi/ipr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Brian King Aug. 20, 2015, 8:28 p.m. UTC | #1
On 08/19/2015 09:47 AM, Gabriel Krisman Bertazi wrote:
> Remove unnecessary check that disabled SIS pipe commands for SIS-32
> devices.  This change was sufficient to enable raw mode and send SIS
> pipe commands for a 57B3 device.
> 
> Fixes: f8ee25d7d239 ("ipr: AF DASD raw mode implementation in ipr driver")
> Signed-off-by: Gabriel Krisman Bertazi <krisman@linux.vnet.ibm.com>
> ---

Thanks!

Acked-by: Brian King <brking@linux.vnet.ibm.com>
wenxiong@linux.vnet.ibm.com Aug. 27, 2015, 7:28 p.m. UTC | #2
Gabriel Krisman Bertazi <krisman <at> linux.vnet.ibm.com> writes:

> 
> Remove unnecessary check that disabled SIS pipe commands for SIS-32
> devices.  This change was sufficient to enable raw mode and send SIS
> pipe commands for a 57B3 device.
> 

Reviewed-by: Wen Xiong <wenxiong@linux.vnet.ibm.com>

Thanks,

Wendy


--
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 mbox

Patch

diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c
index db0c2cf..b19719f 100644
--- a/drivers/scsi/ipr.c
+++ b/drivers/scsi/ipr.c
@@ -4554,7 +4554,7 @@  static ssize_t ipr_store_raw_mode(struct device *dev,
 	spin_lock_irqsave(ioa_cfg->host->host_lock, lock_flags);
 	res = (struct ipr_resource_entry *)sdev->hostdata;
 	if (res) {
-		if (ioa_cfg->sis64 && ipr_is_af_dasd_device(res)) {
+		if (ipr_is_af_dasd_device(res)) {
 			res->raw_mode = simple_strtoul(buf, NULL, 10);
 			len = strlen(buf);
 			if (res->sdev)