* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox