diff mbox series

[11/11] tcm_qla2xxx: Minimize #include directives

Message ID 20190404194448.121489-12-bvanassche@acm.org (mailing list archive)
State Accepted
Headers show
Series qla2xxx patches for kernel v5.2 | expand

Commit Message

Bart Van Assche April 4, 2019, 7:44 p.m. UTC
Only include those header files that are needed by the code in this source
file.

Cc: Himanshu Madhani <hmadhani@marvell.com>
Cc: Giridhar Malavali <gmalavali@marvell.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
 drivers/scsi/qla2xxx/tcm_qla2xxx.c | 7 -------
 1 file changed, 7 deletions(-)

Comments

Himanshu Madhani April 8, 2019, 4:47 p.m. UTC | #1
On 4/4/19, 12:45 PM, "Bart Van Assche" <bvanassche@acm.org> wrote:

    External Email
    
    ----------------------------------------------------------------------
    Only include those header files that are needed by the code in this source
    file.
    
    Cc: Himanshu Madhani <hmadhani@marvell.com>
    Cc: Giridhar Malavali <gmalavali@marvell.com>
    Signed-off-by: Bart Van Assche <bvanassche@acm.org>
    ---
     drivers/scsi/qla2xxx/tcm_qla2xxx.c | 7 -------
     1 file changed, 7 deletions(-)
    
    diff --git a/drivers/scsi/qla2xxx/tcm_qla2xxx.c b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
    index 8a3075d17c63..013f94abe477 100644
    --- a/drivers/scsi/qla2xxx/tcm_qla2xxx.c
    +++ b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
    @@ -24,22 +24,15 @@
     
     
     #include <linux/module.h>
    -#include <linux/moduleparam.h>
     #include <linux/utsname.h>
    -#include <linux/vmalloc.h>
    -#include <linux/init.h>
     #include <linux/list.h>
     #include <linux/slab.h>
    -#include <linux/kthread.h>
     #include <linux/types.h>
     #include <linux/string.h>
     #include <linux/configfs.h>
     #include <linux/ctype.h>
     #include <asm/unaligned.h>
    -#include <scsi/scsi.h>
     #include <scsi/scsi_host.h>
    -#include <scsi/scsi_device.h>
    -#include <scsi/scsi_cmnd.h>
     #include <target/target_core_base.h>
     #include <target/target_core_fabric.h>
     
    -- 
    2.21.0.196.g041f5ea1cf98
    
    
 Thanks for the patch.

Acked-by: Himanshu Madhani <hmadhani@marvell.com>
diff mbox series

Patch

diff --git a/drivers/scsi/qla2xxx/tcm_qla2xxx.c b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
index 8a3075d17c63..013f94abe477 100644
--- a/drivers/scsi/qla2xxx/tcm_qla2xxx.c
+++ b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
@@ -24,22 +24,15 @@ 
 
 
 #include <linux/module.h>
-#include <linux/moduleparam.h>
 #include <linux/utsname.h>
-#include <linux/vmalloc.h>
-#include <linux/init.h>
 #include <linux/list.h>
 #include <linux/slab.h>
-#include <linux/kthread.h>
 #include <linux/types.h>
 #include <linux/string.h>
 #include <linux/configfs.h>
 #include <linux/ctype.h>
 #include <asm/unaligned.h>
-#include <scsi/scsi.h>
 #include <scsi/scsi_host.h>
-#include <scsi/scsi_device.h>
-#include <scsi/scsi_cmnd.h>
 #include <target/target_core_base.h>
 #include <target/target_core_fabric.h>