2012-06-10 00:59:47

by Jesper Juhl

[permalink] [raw]
Subject: [PATCH] [SCSI] qla2xxx: Get rid of unneeded includes of version.h

Signed-off-by: Jesper Juhl <[email protected]>
---
drivers/scsi/qla2xxx/qla_target.c | 1 -
drivers/scsi/qla2xxx/qla_target.h | 2 --
2 files changed, 3 deletions(-)

diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c
index 04f80eb..f7a49e4 100644
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -26,7 +26,6 @@
#include <linux/module.h>
#include <linux/init.h>
#include <linux/types.h>
-#include <linux/version.h>
#include <linux/blkdev.h>
#include <linux/interrupt.h>
#include <linux/pci.h>
diff --git a/drivers/scsi/qla2xxx/qla_target.h b/drivers/scsi/qla2xxx/qla_target.h
index 9ec19bc..7df4df5 100644
--- a/drivers/scsi/qla2xxx/qla_target.h
+++ b/drivers/scsi/qla2xxx/qla_target.h
@@ -919,8 +919,6 @@ struct qla_tgt_srr_ctio {
#define QLA_TGT_XMIT_STATUS 2
#define QLA_TGT_XMIT_ALL (QLA_TGT_XMIT_STATUS|QLA_TGT_XMIT_DATA)

-#include <linux/version.h>
-
extern struct qla_tgt_data qla_target;
/*
* Internal function prototypes
--
1.7.10.4


--
Jesper Juhl <[email protected]> http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.


2012-06-10 07:18:59

by Sachin Kamat

[permalink] [raw]
Subject: Re: [PATCH] [SCSI] qla2xxx: Get rid of unneeded includes of version.h

Hi Jesper,

I have already submitted a similar patch for this:

http://www.spinics.net/lists/linux-scsi/msg59815.html



On 10 June 2012 06:29, Jesper Juhl <[email protected]> wrote:
> Signed-off-by: Jesper Juhl <[email protected]>
> ---
> ?drivers/scsi/qla2xxx/qla_target.c | ? ?1 -
> ?drivers/scsi/qla2xxx/qla_target.h | ? ?2 --
> ?2 files changed, 3 deletions(-)
>
> diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c
> index 04f80eb..f7a49e4 100644
> --- a/drivers/scsi/qla2xxx/qla_target.c
> +++ b/drivers/scsi/qla2xxx/qla_target.c
> @@ -26,7 +26,6 @@
> ?#include <linux/module.h>
> ?#include <linux/init.h>
> ?#include <linux/types.h>
> -#include <linux/version.h>
> ?#include <linux/blkdev.h>
> ?#include <linux/interrupt.h>
> ?#include <linux/pci.h>
> diff --git a/drivers/scsi/qla2xxx/qla_target.h b/drivers/scsi/qla2xxx/qla_target.h
> index 9ec19bc..7df4df5 100644
> --- a/drivers/scsi/qla2xxx/qla_target.h
> +++ b/drivers/scsi/qla2xxx/qla_target.h
> @@ -919,8 +919,6 @@ struct qla_tgt_srr_ctio {
> ?#define QLA_TGT_XMIT_STATUS ? ? ? ? ? ?2
> ?#define QLA_TGT_XMIT_ALL ? ? ? ? ? ? ? (QLA_TGT_XMIT_STATUS|QLA_TGT_XMIT_DATA)
>
> -#include <linux/version.h>
> -
> ?extern struct qla_tgt_data qla_target;
> ?/*
> ?* Internal function prototypes
> --
> 1.7.10.4
>
>
> --
> Jesper Juhl <[email protected]> ? ? ? http://www.chaosbits.net/
> Don't top-post http://www.catb.org/jargon/html/T/top-post.html
> Plain text mails only, please.
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at ?http://www.tux.org/lkml/



--
With warm regards,
Sachin

2012-06-11 20:39:22

by Jesper Juhl

[permalink] [raw]
Subject: Re: [PATCH] [SCSI] qla2xxx: Get rid of unneeded includes of version.h

On Sun, 10 Jun 2012, Sachin Kamat wrote:

> Hi Jesper,
>
> I have already submitted a similar patch for this:
>
> http://www.spinics.net/lists/linux-scsi/msg59815.html
>
Ok. Cool.

Please consider my patch an ACK of yours then :-)


--
Jesper Juhl <[email protected]> http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.