All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO
@ 2017-11-08  8:38 ` Dan Carpenter
  0 siblings, 0 replies; 6+ messages in thread
From: Dan Carpenter @ 2017-11-08  8:38 UTC (permalink / raw)
  To: Sathya Prakash, Suganath Prabu Subramani
  Cc: Chaitra P B, James E.J. Bottomley, Martin K. Petersen,
	MPT-FusionLinux.pdl, linux-scsi, kernel-janitors

pr_info() has a KERN_INFO already so the second KERN_INFO isn't needed.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
index 3a9438a1704e..b258f210120a 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
@@ -8683,7 +8684,7 @@ _scsih_mark_responding_pcie_device(struct MPT3SAS_ADAPTER *ioc,
 
 			if (pcie_device->handle = pcie_device_pg0->DevHandle)
 				goto out;
-			pr_info(KERN_INFO "\thandle changed from(0x%04x)!!!\n",
+			pr_info("\thandle changed from(0x%04x)!!!\n",
 			    pcie_device->handle);
 			pcie_device->handle = pcie_device_pg0->DevHandle;
 			if (sas_target_priv_data)

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO
@ 2017-11-08  8:38 ` Dan Carpenter
  0 siblings, 0 replies; 6+ messages in thread
From: Dan Carpenter @ 2017-11-08  8:38 UTC (permalink / raw)
  To: Sathya Prakash, Suganath Prabu Subramani
  Cc: Chaitra P B, James E.J. Bottomley, Martin K. Petersen,
	MPT-FusionLinux.pdl, linux-scsi, kernel-janitors

pr_info() has a KERN_INFO already so the second KERN_INFO isn't needed.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
index 3a9438a1704e..b258f210120a 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
@@ -8683,7 +8684,7 @@ _scsih_mark_responding_pcie_device(struct MPT3SAS_ADAPTER *ioc,
 
 			if (pcie_device->handle == pcie_device_pg0->DevHandle)
 				goto out;
-			pr_info(KERN_INFO "\thandle changed from(0x%04x)!!!\n",
+			pr_info("\thandle changed from(0x%04x)!!!\n",
 			    pcie_device->handle);
 			pcie_device->handle = pcie_device_pg0->DevHandle;
 			if (sas_target_priv_data)

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* RE: [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO
  2017-11-08  8:38 ` Dan Carpenter
@ 2017-11-08 16:03   ` Sathya Prakash Veerichetty
  -1 siblings, 0 replies; 6+ messages in thread
From: Sathya Prakash Veerichetty @ 2017-11-08 16:03 UTC (permalink / raw)
  To: Dan Carpenter, Suganath Prabu Subramani
  Cc: Chaitra Basappa, James E.J. Bottomley, Martin K. Petersen,
	PDL-MPT-FUSIONLINUX, linux-scsi, kernel-janitors

Thanks.
Acked-by: Sathya Prakash Veerichetty <sathya.prakash@broadcom.com>

-----Original Message-----
From: Dan Carpenter [mailto:dan.carpenter@oracle.com]
Sent: Wednesday, November 8, 2017 1:38 AM
To: Sathya Prakash; Suganath Prabu Subramani
Cc: Chaitra P B; James E.J. Bottomley; Martin K. Petersen;
MPT-FusionLinux.pdl@broadcom.com; linux-scsi@vger.kernel.org;
kernel-janitors@vger.kernel.org
Subject: [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO

pr_info() has a KERN_INFO already so the second KERN_INFO isn't needed.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
index 3a9438a1704e..b258f210120a 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
@@ -8683,7 +8684,7 @@ _scsih_mark_responding_pcie_device(struct
MPT3SAS_ADAPTER *ioc,

 			if (pcie_device->handle =
pcie_device_pg0->DevHandle)
 				goto out;
-			pr_info(KERN_INFO "\thandle changed
from(0x%04x)!!!\n",
+			pr_info("\thandle changed from(0x%04x)!!!\n",
 			    pcie_device->handle);
 			pcie_device->handle = pcie_device_pg0->DevHandle;
 			if (sas_target_priv_data)

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* RE: [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO
@ 2017-11-08 16:03   ` Sathya Prakash Veerichetty
  0 siblings, 0 replies; 6+ messages in thread
From: Sathya Prakash Veerichetty @ 2017-11-08 16:03 UTC (permalink / raw)
  To: Dan Carpenter, Suganath Prabu Subramani
  Cc: Chaitra Basappa, James E.J. Bottomley, Martin K. Petersen,
	PDL-MPT-FUSIONLINUX, linux-scsi, kernel-janitors

Thanks.
Acked-by: Sathya Prakash Veerichetty <sathya.prakash@broadcom.com>

-----Original Message-----
From: Dan Carpenter [mailto:dan.carpenter@oracle.com]
Sent: Wednesday, November 8, 2017 1:38 AM
To: Sathya Prakash; Suganath Prabu Subramani
Cc: Chaitra P B; James E.J. Bottomley; Martin K. Petersen;
MPT-FusionLinux.pdl@broadcom.com; linux-scsi@vger.kernel.org;
kernel-janitors@vger.kernel.org
Subject: [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO

pr_info() has a KERN_INFO already so the second KERN_INFO isn't needed.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
index 3a9438a1704e..b258f210120a 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
@@ -8683,7 +8684,7 @@ _scsih_mark_responding_pcie_device(struct
MPT3SAS_ADAPTER *ioc,

 			if (pcie_device->handle ==
pcie_device_pg0->DevHandle)
 				goto out;
-			pr_info(KERN_INFO "\thandle changed
from(0x%04x)!!!\n",
+			pr_info("\thandle changed from(0x%04x)!!!\n",
 			    pcie_device->handle);
 			pcie_device->handle = pcie_device_pg0->DevHandle;
 			if (sas_target_priv_data)

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO
  2017-11-08  8:38 ` Dan Carpenter
@ 2017-11-08 23:18   ` Martin K. Petersen
  -1 siblings, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2017-11-08 23:18 UTC (permalink / raw)
  To: Dan Carpenter
  Cc: Sathya Prakash, Suganath Prabu Subramani, Chaitra P B,
	James E.J. Bottomley, Martin K. Petersen, MPT-FusionLinux.pdl,
	linux-scsi, kernel-janitors


Dan,

> pr_info() has a KERN_INFO already so the second KERN_INFO isn't needed.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Applied to 4.15/scsi-queue.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO
@ 2017-11-08 23:18   ` Martin K. Petersen
  0 siblings, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2017-11-08 23:18 UTC (permalink / raw)
  To: Dan Carpenter
  Cc: Sathya Prakash, Suganath Prabu Subramani, Chaitra P B,
	James E.J. Bottomley, Martin K. Petersen, MPT-FusionLinux.pdl,
	linux-scsi, kernel-janitors


Dan,

> pr_info() has a KERN_INFO already so the second KERN_INFO isn't needed.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Applied to 4.15/scsi-queue.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-11-08 23:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-08  8:38 [patch 2/2] scsi: mpt3sas: remove a stray KERN_INFO Dan Carpenter
2017-11-08  8:38 ` Dan Carpenter
2017-11-08 16:03 ` Sathya Prakash Veerichetty
2017-11-08 16:03   ` Sathya Prakash Veerichetty
2017-11-08 23:18 ` Martin K. Petersen
2017-11-08 23:18   ` Martin K. Petersen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.