From: Roland Dreier <roland@xxxxxxxxxxxxxxx> There's no need for the #ifdef protection when building into the kernel, and in fact we need the module_init() for the initialization function to be called. Signed-off-by: Roland Dreier <roland@xxxxxxxxxxxxxxx> --- drivers/target/tcm_fc/tfc_conf.c | 2 -- drivers/target/tcm_qla2xxx/tcm_qla2xxx_configfs.c | 2 -- drivers/target/tcm_vhost/tcm_vhost_configfs.c | 2 -- 3 files changed, 0 insertions(+), 6 deletions(-) diff --git a/drivers/target/tcm_fc/tfc_conf.c b/drivers/target/tcm_fc/tfc_conf.c index 3291c12..a5c7128 100644 --- a/drivers/target/tcm_fc/tfc_conf.c +++ b/drivers/target/tcm_fc/tfc_conf.c @@ -656,9 +656,7 @@ static void __exit ft_exit(void) synchronize_rcu(); } -#ifdef MODULE MODULE_DESCRIPTION("FC TCM fabric driver " FT_VERSION); MODULE_LICENSE("GPL"); module_init(ft_init); module_exit(ft_exit); -#endif /* MODULE */ diff --git a/drivers/target/tcm_qla2xxx/tcm_qla2xxx_configfs.c b/drivers/target/tcm_qla2xxx/tcm_qla2xxx_configfs.c index 1fa0de0..8b810da 100644 --- a/drivers/target/tcm_qla2xxx/tcm_qla2xxx_configfs.c +++ b/drivers/target/tcm_qla2xxx/tcm_qla2xxx_configfs.c @@ -1427,9 +1427,7 @@ static void __exit tcm_qla2xxx_exit(void) tcm_qla2xxx_deregister_configfs(); } -#ifdef MODULE MODULE_DESCRIPTION("TCM QLA2XXX series NPIV enabled fabric driver"); MODULE_LICENSE("GPL"); module_init(tcm_qla2xxx_init); module_exit(tcm_qla2xxx_exit); -#endif diff --git a/drivers/target/tcm_vhost/tcm_vhost_configfs.c b/drivers/target/tcm_vhost/tcm_vhost_configfs.c index 1d5d03a..c281a7e 100644 --- a/drivers/target/tcm_vhost/tcm_vhost_configfs.c +++ b/drivers/target/tcm_vhost/tcm_vhost_configfs.c @@ -636,9 +636,7 @@ static void tcm_vhost_exit(void) vhost_scsi_deregister(); }; -#ifdef MODULE MODULE_DESCRIPTION("TCM_VHOST series fabric driver"); MODULE_LICENSE("GPL"); module_init(tcm_vhost_init); module_exit(tcm_vhost_exit); -#endif -- 1.7.5.4 -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html