All of lore.kernel.org
 help / color / mirror / Atom feed
* [TRIVIAL PATCH 0/2] Fix warnings in kernel build
@ 2013-08-26 17:06 Ewan D. Milne
  2013-08-26 17:06 ` [TRIVIAL PATCH 1/2] scsi: Add missing @sdev parameter description to scsi_report_sense() Ewan D. Milne
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Ewan D. Milne @ 2013-08-26 17:06 UTC (permalink / raw)
  To: linux-scsi

From: "Ewan D. Milne" <emilne@redhat.com>

The patch to add uevents for certain Unit Attention codes
had a couple of problems which were found by a kbuild test robot.
Sorry about that, I will be more careful next time.

Ewan D. Milne (2):
  scsi: Add missing @sdev parameter description to scsi_report_sense()
  scsi: Remove unused variable "flags" from scsi_report_sense()

 drivers/scsi/scsi_error.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
1.7.11.7


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

* [TRIVIAL PATCH 1/2] scsi: Add missing @sdev parameter description to scsi_report_sense()
  2013-08-26 17:06 [TRIVIAL PATCH 0/2] Fix warnings in kernel build Ewan D. Milne
@ 2013-08-26 17:06 ` Ewan D. Milne
  2013-08-26 17:06 ` [TRIVIAL PATCH 2/2] scsi: Remove unused variable "flags" from scsi_report_sense() Ewan D. Milne
  2013-08-26 17:53 ` [TRIVIAL PATCH 0/2] Fix warnings in kernel build James Bottomley
  2 siblings, 0 replies; 4+ messages in thread
From: Ewan D. Milne @ 2013-08-26 17:06 UTC (permalink / raw)
  To: linux-scsi

From: "Ewan D. Milne" <emilne@redhat.com>

Signed-off-by: Ewan D. Milne <emilne@redhat.com>
---
 drivers/scsi/scsi_error.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
index 598afd9..cf6fd20 100644
--- a/drivers/scsi/scsi_error.c
+++ b/drivers/scsi/scsi_error.c
@@ -241,6 +241,7 @@ static void scsi_report_lun_change(struct scsi_device *sdev)
 /**
  * scsi_report_sense - Examine scsi sense information and log messages for
  *		       certain conditions, also issue uevents for some of them.
+ * @sdev:	Device reporting sense information
  * @sshdr:	sshdr to be examined
  */
 static void scsi_report_sense(struct scsi_device *sdev,
-- 
1.7.11.7


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

* [TRIVIAL PATCH 2/2] scsi: Remove unused variable "flags" from scsi_report_sense()
  2013-08-26 17:06 [TRIVIAL PATCH 0/2] Fix warnings in kernel build Ewan D. Milne
  2013-08-26 17:06 ` [TRIVIAL PATCH 1/2] scsi: Add missing @sdev parameter description to scsi_report_sense() Ewan D. Milne
@ 2013-08-26 17:06 ` Ewan D. Milne
  2013-08-26 17:53 ` [TRIVIAL PATCH 0/2] Fix warnings in kernel build James Bottomley
  2 siblings, 0 replies; 4+ messages in thread
From: Ewan D. Milne @ 2013-08-26 17:06 UTC (permalink / raw)
  To: linux-scsi

From: "Ewan D. Milne" <emilne@redhat.com>

Signed-off-by: Ewan D. Milne <emilne@redhat.com>
---
 drivers/scsi/scsi_error.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
index cf6fd20..7f95236 100644
--- a/drivers/scsi/scsi_error.c
+++ b/drivers/scsi/scsi_error.c
@@ -248,7 +248,6 @@ static void scsi_report_sense(struct scsi_device *sdev,
 			      struct scsi_sense_hdr *sshdr)
 {
 	enum scsi_device_event evt_type = SDEV_EVT_MAXBITS;	/* i.e. none */
-	unsigned long flags;
 
 	if (sshdr->sense_key == UNIT_ATTENTION) {
 		if (sshdr->asc == 0x3f && sshdr->ascq == 0x03) {
-- 
1.7.11.7


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

* Re: [TRIVIAL PATCH 0/2] Fix warnings in kernel build
  2013-08-26 17:06 [TRIVIAL PATCH 0/2] Fix warnings in kernel build Ewan D. Milne
  2013-08-26 17:06 ` [TRIVIAL PATCH 1/2] scsi: Add missing @sdev parameter description to scsi_report_sense() Ewan D. Milne
  2013-08-26 17:06 ` [TRIVIAL PATCH 2/2] scsi: Remove unused variable "flags" from scsi_report_sense() Ewan D. Milne
@ 2013-08-26 17:53 ` James Bottomley
  2 siblings, 0 replies; 4+ messages in thread
From: James Bottomley @ 2013-08-26 17:53 UTC (permalink / raw)
  To: Ewan D. Milne; +Cc: linux-scsi

On Mon, 2013-08-26 at 13:06 -0400, Ewan D. Milne wrote:
> From: "Ewan D. Milne" <emilne@redhat.com>
> 
> The patch to add uevents for certain Unit Attention codes
> had a couple of problems which were found by a kbuild test robot.
> Sorry about that, I will be more careful next time.

I already did both of these based on Fengguang's checker results ...

James



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

end of thread, other threads:[~2013-08-26 17:53 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-26 17:06 [TRIVIAL PATCH 0/2] Fix warnings in kernel build Ewan D. Milne
2013-08-26 17:06 ` [TRIVIAL PATCH 1/2] scsi: Add missing @sdev parameter description to scsi_report_sense() Ewan D. Milne
2013-08-26 17:06 ` [TRIVIAL PATCH 2/2] scsi: Remove unused variable "flags" from scsi_report_sense() Ewan D. Milne
2013-08-26 17:53 ` [TRIVIAL PATCH 0/2] Fix warnings in kernel build James Bottomley

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.