diff mbox series

[v2] scsi: add missing MODULE_DESCRIPTION() macros

Message ID 20240608-md-drivers-scsi-v2-1-d00d652e5d34@quicinc.com (mailing list archive)
State Superseded
Headers show
Series [v2] scsi: add missing MODULE_DESCRIPTION() macros | expand

Commit Message

Jeff Johnson June 8, 2024, 3:33 p.m. UTC 
On x86, make allmodconfig && make W=1 C=1 reports:
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/scsi_common.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/advansys.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/BusLogic.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/aha1740.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/isci/isci.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/elx/efct.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/atp870u.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/ppa.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/imm.o

Add all missing invocations of the MODULE_DESCRIPTION() macro.

This updates all files which have a MODULE_LICENSE() but which do not
have a MODULE_DESCRIPTION(), even ones which did not produce the x86
allmodconfig warnings.

Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
---
Changes in v2:
- Updated descriptions of atari_scsi.c, g_NCR5380.c, mac_scsi.c per Finn Thain
  & Michael Schmitz
- Removed unnecessary modifications to initio.c and sr.c
- Link to v1: https://lore.kernel.org/r/20240607-md-drivers-scsi-v1-1-17ae31cc4fe5@quicinc.com
---
 drivers/scsi/BusLogic.c             | 1 +
 drivers/scsi/advansys.c             | 1 +
 drivers/scsi/aha1542.c              | 1 +
 drivers/scsi/aha1740.c              | 1 +
 drivers/scsi/atari_scsi.c           | 1 +
 drivers/scsi/atp870u.c              | 1 +
 drivers/scsi/elx/efct/efct_driver.c | 1 +
 drivers/scsi/g_NCR5380.c            | 1 +
 drivers/scsi/imm.c                  | 1 +
 drivers/scsi/isci/init.c            | 1 +
 drivers/scsi/mac_scsi.c             | 1 +
 drivers/scsi/pcmcia/aha152x_stub.c  | 1 +
 drivers/scsi/ppa.c                  | 1 +
 drivers/scsi/scsi_common.c          | 1 +
 drivers/scsi/sun3_scsi.c            | 1 +
 15 files changed, 15 insertions(+)


---
base-commit: 19ca0d8a433ff37018f9429f7e7739e9f3d3d2b4
change-id: 20240607-md-drivers-scsi-e3364073e9b9

Comments

Finn Thain June 8, 2024, 11:43 p.m. UTC | #1 
On Sat, 8 Jun 2024, Jeff Johnson wrote:

> On x86, make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/scsi_common.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/advansys.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/BusLogic.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/aha1740.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/isci/isci.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/elx/efct.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/atp870u.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/ppa.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/imm.o
> 
> Add all missing invocations of the MODULE_DESCRIPTION() macro.
> 
> This updates all files which have a MODULE_LICENSE() but which do not
> have a MODULE_DESCRIPTION(), even ones which did not produce the x86
> allmodconfig warnings.
> 
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>

Acked-by: Finn Thain <fthain@linux-m68k.org>

Thanks.
Hannes Reinecke June 10, 2024, 6:10 a.m. UTC | #2 
On 6/8/24 17:33, Jeff Johnson wrote:
> On x86, make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/scsi_common.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/advansys.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/BusLogic.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/aha1740.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/isci/isci.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/elx/efct.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/atp870u.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/ppa.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/scsi/imm.o
> 
> Add all missing invocations of the MODULE_DESCRIPTION() macro.
> 
> This updates all files which have a MODULE_LICENSE() but which do not
> have a MODULE_DESCRIPTION(), even ones which did not produce the x86
> allmodconfig warnings.
> 
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
> ---
> Changes in v2:
> - Updated descriptions of atari_scsi.c, g_NCR5380.c, mac_scsi.c per Finn Thain
>    & Michael Schmitz
> - Removed unnecessary modifications to initio.c and sr.c
> - Link to v1: https://lore.kernel.org/r/20240607-md-drivers-scsi-v1-1-17ae31cc4fe5@quicinc.com
> ---
>   drivers/scsi/BusLogic.c             | 1 +
>   drivers/scsi/advansys.c             | 1 +
>   drivers/scsi/aha1542.c              | 1 +
>   drivers/scsi/aha1740.c              | 1 +
>   drivers/scsi/atari_scsi.c           | 1 +
>   drivers/scsi/atp870u.c              | 1 +
>   drivers/scsi/elx/efct/efct_driver.c | 1 +
>   drivers/scsi/g_NCR5380.c            | 1 +
>   drivers/scsi/imm.c                  | 1 +
>   drivers/scsi/isci/init.c            | 1 +
>   drivers/scsi/mac_scsi.c             | 1 +
>   drivers/scsi/pcmcia/aha152x_stub.c  | 1 +
>   drivers/scsi/ppa.c                  | 1 +
>   drivers/scsi/scsi_common.c          | 1 +
>   drivers/scsi/sun3_scsi.c            | 1 +
>   15 files changed, 15 insertions(+)
> 
> diff --git a/drivers/scsi/BusLogic.c b/drivers/scsi/BusLogic.c
> index 72ceaf650b0d..2135a2b3e2d0 100644
> --- a/drivers/scsi/BusLogic.c
> +++ b/drivers/scsi/BusLogic.c
> @@ -78,6 +78,7 @@ static struct blogic_drvr_options blogic_drvr_options[BLOGIC_MAX_ADAPTERS];
>     BusLogic can be assigned a string by insmod.
>   */
>   
> +MODULE_DESCRIPTION("BusLogic MultiMaster and FlashPoint SCSI Host Adapter driver");
>   MODULE_LICENSE("GPL");
>   #ifdef MODULE
>   static char *BusLogic;
> diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c
> index ab066bb27a57..fd4fcb37863d 100644
> --- a/drivers/scsi/advansys.c
> +++ b/drivers/scsi/advansys.c
> @@ -11545,6 +11545,7 @@ static void __exit advansys_exit(void)
>   module_init(advansys_init);
>   module_exit(advansys_exit);
>   
> +MODULE_DESCRIPTION("AdvanSys SCSI Adapter driver");
>   MODULE_LICENSE("GPL");
>   MODULE_FIRMWARE("advansys/mcode.bin");
>   MODULE_FIRMWARE("advansys/3550.bin");
> diff --git a/drivers/scsi/aha1542.c b/drivers/scsi/aha1542.c
> index 9503996c6325..add10098a569 100644
> --- a/drivers/scsi/aha1542.c
> +++ b/drivers/scsi/aha1542.c
> @@ -1009,6 +1009,7 @@ static int aha1542_biosparam(struct scsi_device *sdev,
>   
>   	return 0;
>   }
> +MODULE_DESCRIPTION("Adaptec AHA-1542 SCSI host adapter driver");
>   MODULE_LICENSE("GPL");

Please add a newline before the MODULE_DESCRIPTION line.

>   
>   static int aha1542_init_cmd_priv(struct Scsi_Host *shost, struct scsi_cmnd *cmd)
> diff --git a/drivers/scsi/aha1740.c b/drivers/scsi/aha1740.c
> index 3d18945abaf7..be7ebbbb9ba8 100644
> --- a/drivers/scsi/aha1740.c
> +++ b/drivers/scsi/aha1740.c
> @@ -681,4 +681,5 @@ static __exit void aha1740_exit (void)
>   module_init (aha1740_init);
>   module_exit (aha1740_exit);
>   
> +MODULE_DESCRIPTION("Adaptec AHA1740 SCSI host adapter driver");
>   MODULE_LICENSE("GPL");
> diff --git a/drivers/scsi/atari_scsi.c b/drivers/scsi/atari_scsi.c
> index 742625ac7d99..98a1b966a0b0 100644
> --- a/drivers/scsi/atari_scsi.c
> +++ b/drivers/scsi/atari_scsi.c
> @@ -894,4 +894,5 @@ static struct platform_driver atari_scsi_driver __refdata = {
>   module_platform_driver_probe(atari_scsi_driver, atari_scsi_probe);
>   
>   MODULE_ALIAS("platform:" DRV_MODULE_NAME);
> +MODULE_DESCRIPTION("Atari TT/Falcon NCR5380 SCSI driver");
>   MODULE_LICENSE("GPL");
> diff --git a/drivers/scsi/atp870u.c b/drivers/scsi/atp870u.c
> index 2a748af269c2..3f006b2f39a6 100644
> --- a/drivers/scsi/atp870u.c
> +++ b/drivers/scsi/atp870u.c
> @@ -1724,6 +1724,7 @@ static void atp870u_remove (struct pci_dev *pdev)
>   	atp870u_free_tables(pshost);
>   	scsi_host_put(pshost);
>   }
> +MODULE_DESCRIPTION("ACARD SCSI host adapter driver");
>   MODULE_LICENSE("GPL");

Again, missing newline.

>   
>   static const struct scsi_host_template atp870u_template = {
> diff --git a/drivers/scsi/elx/efct/efct_driver.c b/drivers/scsi/elx/efct/efct_driver.c
> index 49fd2cfed70c..55d2301bfd7d 100644
> --- a/drivers/scsi/elx/efct/efct_driver.c
> +++ b/drivers/scsi/elx/efct/efct_driver.c
> @@ -778,5 +778,6 @@ static void __exit efct_exit(void)
>   module_init(efct_init);
>   module_exit(efct_exit);
>   MODULE_VERSION(EFCT_DRIVER_VERSION);
> +MODULE_DESCRIPTION("Emulex Fibre Channel Target driver");
>   MODULE_LICENSE("GPL");
>   MODULE_AUTHOR("Broadcom");
> diff --git a/drivers/scsi/g_NCR5380.c b/drivers/scsi/g_NCR5380.c
> index f6305e3e60f4..270eae7ac427 100644
> --- a/drivers/scsi/g_NCR5380.c
> +++ b/drivers/scsi/g_NCR5380.c
> @@ -110,6 +110,7 @@ module_param_array(card, int, NULL, 0);
>   MODULE_PARM_DESC(card, "card type (0=NCR5380, 1=NCR53C400, 2=NCR53C400A, 3=DTC3181E, 4=HP C2502)");
>   
>   MODULE_ALIAS("g_NCR5380_mmio");
> +MODULE_DESCRIPTION("Generic NCR5380/NCR53C400 SCSI driver");
>   MODULE_LICENSE("GPL");
>   
>   static void g_NCR5380_trigger_irq(struct Scsi_Host *instance)
> diff --git a/drivers/scsi/imm.c b/drivers/scsi/imm.c
> index 21339da505f1..6e779bb14d98 100644
> --- a/drivers/scsi/imm.c
> +++ b/drivers/scsi/imm.c
> @@ -1279,4 +1279,5 @@ static struct parport_driver imm_driver = {
>   };
>   module_parport_driver(imm_driver);
>   
> +MODULE_DESCRIPTION("IOMEGA MatchMaker parallel port SCSI host adapter driver");
>   MODULE_LICENSE("GPL");
> diff --git a/drivers/scsi/isci/init.c b/drivers/scsi/isci/init.c
> index de2aefcf2089..d31884f82f2a 100644
> --- a/drivers/scsi/isci/init.c
> +++ b/drivers/scsi/isci/init.c
> @@ -758,6 +758,7 @@ static __exit void isci_exit(void)
>   	sas_release_transport(isci_transport_template);
>   }
>   
> +MODULE_DESCRIPTION("Intel(R) C600 Series Chipset SAS Controller driver");
>   MODULE_LICENSE("Dual BSD/GPL");
>   MODULE_FIRMWARE(ISCI_FW_NAME);
>   module_init(isci_init);
> diff --git a/drivers/scsi/mac_scsi.c b/drivers/scsi/mac_scsi.c
> index a402c4dc4645..53ee8f84d094 100644
> --- a/drivers/scsi/mac_scsi.c
> +++ b/drivers/scsi/mac_scsi.c
> @@ -550,4 +550,5 @@ static struct platform_driver mac_scsi_driver __refdata = {
>   module_platform_driver_probe(mac_scsi_driver, mac_scsi_probe);
>   
>   MODULE_ALIAS("platform:" DRV_MODULE_NAME);
> +MODULE_DESCRIPTION("Macintosh NCR5380 SCSI driver");
>   MODULE_LICENSE("GPL");
> diff --git a/drivers/scsi/pcmcia/aha152x_stub.c b/drivers/scsi/pcmcia/aha152x_stub.c
> index 6a6621728c69..1b54ba51a485 100644
> --- a/drivers/scsi/pcmcia/aha152x_stub.c
> +++ b/drivers/scsi/pcmcia/aha152x_stub.c
> @@ -75,6 +75,7 @@ module_param(synchronous, int, 0);
>   module_param(reset_delay, int, 0);
>   module_param(ext_trans, int, 0);
>   
> +MODULE_DESCRIPTION("Adaptec AHA152X-compatible PCMCIA SCSI card driver");
>   MODULE_LICENSE("Dual MPL/GPL");
>   
>   /*====================================================================*/
> diff --git a/drivers/scsi/ppa.c b/drivers/scsi/ppa.c
> index 8300f0bdddb3..2d9fcc45ad85 100644
> --- a/drivers/scsi/ppa.c
> +++ b/drivers/scsi/ppa.c
> @@ -1155,4 +1155,5 @@ static struct parport_driver ppa_driver = {
>   };
>   module_parport_driver(ppa_driver);
>   
> +MODULE_DESCRIPTION("IOMEGA PPA3 parallel port SCSI host adapter driver");
>   MODULE_LICENSE("GPL");
> diff --git a/drivers/scsi/scsi_common.c b/drivers/scsi/scsi_common.c
> index 9c14fdf61037..04749fde1636 100644
> --- a/drivers/scsi/scsi_common.c
> +++ b/drivers/scsi/scsi_common.c
> @@ -12,6 +12,7 @@
>   #include <asm/unaligned.h>
>   #include <scsi/scsi_common.h>
>   
> +MODULE_DESCRIPTION("SCSI functions used by both the initiator and the target code");
>   MODULE_LICENSE("GPL v2");
>   
>   /* Command group 3 is reserved and should never be used.  */
> diff --git a/drivers/scsi/sun3_scsi.c b/drivers/scsi/sun3_scsi.c
> index 4a8cc2e8238e..f51702893306 100644
> --- a/drivers/scsi/sun3_scsi.c
> +++ b/drivers/scsi/sun3_scsi.c
> @@ -666,4 +666,5 @@ static struct platform_driver sun3_scsi_driver = {
>   module_platform_driver_probe(sun3_scsi_driver, sun3_scsi_probe);
>   
>   MODULE_ALIAS("platform:" DRV_MODULE_NAME);
> +MODULE_DESCRIPTION("Sun3 NCR5380 SCSI controller driver");
>   MODULE_LICENSE("GPL");
> 
> ---
> base-commit: 19ca0d8a433ff37018f9429f7e7739e9f3d3d2b4
> change-id: 20240607-md-drivers-scsi-e3364073e9b9
> 
Cheers,

Hannes
Jeff Johnson June 10, 2024, 2:57 p.m. UTC | #3 
On 6/9/2024 11:10 PM, Hannes Reinecke wrote:
> On 6/8/24 17:33, Jeff Johnson wrote:
[...]
>> diff --git a/drivers/scsi/aha1542.c b/drivers/scsi/aha1542.c
>> index 9503996c6325..add10098a569 100644
>> --- a/drivers/scsi/aha1542.c
>> +++ b/drivers/scsi/aha1542.c
>> @@ -1009,6 +1009,7 @@ static int aha1542_biosparam(struct scsi_device *sdev,
>>   
>>   	return 0;
>>   }
>> +MODULE_DESCRIPTION("Adaptec AHA-1542 SCSI host adapter driver");
>>   MODULE_LICENSE("GPL");
> 
> Please add a newline before the MODULE_DESCRIPTION line.
> 
>>   
>>   static int aha1542_init_cmd_priv(struct Scsi_Host *shost, struct scsi_cmnd *cmd)
[...]
>> --- a/drivers/scsi/atp870u.c
>> +++ b/drivers/scsi/atp870u.c
>> @@ -1724,6 +1724,7 @@ static void atp870u_remove (struct pci_dev *pdev)
>>   	atp870u_free_tables(pshost);
>>   	scsi_host_put(pshost);
>>   }
>> +MODULE_DESCRIPTION("ACARD SCSI host adapter driver");
>>   MODULE_LICENSE("GPL");
> 
> Again, missing newline.

I'll update these in v2
diff mbox series

Patch

diff --git a/drivers/scsi/BusLogic.c b/drivers/scsi/BusLogic.c
index 72ceaf650b0d..2135a2b3e2d0 100644
--- a/drivers/scsi/BusLogic.c
+++ b/drivers/scsi/BusLogic.c
@@ -78,6 +78,7 @@  static struct blogic_drvr_options blogic_drvr_options[BLOGIC_MAX_ADAPTERS];
   BusLogic can be assigned a string by insmod.
 */
 
+MODULE_DESCRIPTION("BusLogic MultiMaster and FlashPoint SCSI Host Adapter driver");
 MODULE_LICENSE("GPL");
 #ifdef MODULE
 static char *BusLogic;
diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c
index ab066bb27a57..fd4fcb37863d 100644
--- a/drivers/scsi/advansys.c
+++ b/drivers/scsi/advansys.c
@@ -11545,6 +11545,7 @@  static void __exit advansys_exit(void)
 module_init(advansys_init);
 module_exit(advansys_exit);
 
+MODULE_DESCRIPTION("AdvanSys SCSI Adapter driver");
 MODULE_LICENSE("GPL");
 MODULE_FIRMWARE("advansys/mcode.bin");
 MODULE_FIRMWARE("advansys/3550.bin");
diff --git a/drivers/scsi/aha1542.c b/drivers/scsi/aha1542.c
index 9503996c6325..add10098a569 100644
--- a/drivers/scsi/aha1542.c
+++ b/drivers/scsi/aha1542.c
@@ -1009,6 +1009,7 @@  static int aha1542_biosparam(struct scsi_device *sdev,
 
 	return 0;
 }
+MODULE_DESCRIPTION("Adaptec AHA-1542 SCSI host adapter driver");
 MODULE_LICENSE("GPL");
 
 static int aha1542_init_cmd_priv(struct Scsi_Host *shost, struct scsi_cmnd *cmd)
diff --git a/drivers/scsi/aha1740.c b/drivers/scsi/aha1740.c
index 3d18945abaf7..be7ebbbb9ba8 100644
--- a/drivers/scsi/aha1740.c
+++ b/drivers/scsi/aha1740.c
@@ -681,4 +681,5 @@  static __exit void aha1740_exit (void)
 module_init (aha1740_init);
 module_exit (aha1740_exit);
 
+MODULE_DESCRIPTION("Adaptec AHA1740 SCSI host adapter driver");
 MODULE_LICENSE("GPL");
diff --git a/drivers/scsi/atari_scsi.c b/drivers/scsi/atari_scsi.c
index 742625ac7d99..98a1b966a0b0 100644
--- a/drivers/scsi/atari_scsi.c
+++ b/drivers/scsi/atari_scsi.c
@@ -894,4 +894,5 @@  static struct platform_driver atari_scsi_driver __refdata = {
 module_platform_driver_probe(atari_scsi_driver, atari_scsi_probe);
 
 MODULE_ALIAS("platform:" DRV_MODULE_NAME);
+MODULE_DESCRIPTION("Atari TT/Falcon NCR5380 SCSI driver");
 MODULE_LICENSE("GPL");
diff --git a/drivers/scsi/atp870u.c b/drivers/scsi/atp870u.c
index 2a748af269c2..3f006b2f39a6 100644
--- a/drivers/scsi/atp870u.c
+++ b/drivers/scsi/atp870u.c
@@ -1724,6 +1724,7 @@  static void atp870u_remove (struct pci_dev *pdev)
 	atp870u_free_tables(pshost);
 	scsi_host_put(pshost);
 }
+MODULE_DESCRIPTION("ACARD SCSI host adapter driver");
 MODULE_LICENSE("GPL");
 
 static const struct scsi_host_template atp870u_template = {
diff --git a/drivers/scsi/elx/efct/efct_driver.c b/drivers/scsi/elx/efct/efct_driver.c
index 49fd2cfed70c..55d2301bfd7d 100644
--- a/drivers/scsi/elx/efct/efct_driver.c
+++ b/drivers/scsi/elx/efct/efct_driver.c
@@ -778,5 +778,6 @@  static void __exit efct_exit(void)
 module_init(efct_init);
 module_exit(efct_exit);
 MODULE_VERSION(EFCT_DRIVER_VERSION);
+MODULE_DESCRIPTION("Emulex Fibre Channel Target driver");
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Broadcom");
diff --git a/drivers/scsi/g_NCR5380.c b/drivers/scsi/g_NCR5380.c
index f6305e3e60f4..270eae7ac427 100644
--- a/drivers/scsi/g_NCR5380.c
+++ b/drivers/scsi/g_NCR5380.c
@@ -110,6 +110,7 @@  module_param_array(card, int, NULL, 0);
 MODULE_PARM_DESC(card, "card type (0=NCR5380, 1=NCR53C400, 2=NCR53C400A, 3=DTC3181E, 4=HP C2502)");
 
 MODULE_ALIAS("g_NCR5380_mmio");
+MODULE_DESCRIPTION("Generic NCR5380/NCR53C400 SCSI driver");
 MODULE_LICENSE("GPL");
 
 static void g_NCR5380_trigger_irq(struct Scsi_Host *instance)
diff --git a/drivers/scsi/imm.c b/drivers/scsi/imm.c
index 21339da505f1..6e779bb14d98 100644
--- a/drivers/scsi/imm.c
+++ b/drivers/scsi/imm.c
@@ -1279,4 +1279,5 @@  static struct parport_driver imm_driver = {
 };
 module_parport_driver(imm_driver);
 
+MODULE_DESCRIPTION("IOMEGA MatchMaker parallel port SCSI host adapter driver");
 MODULE_LICENSE("GPL");
diff --git a/drivers/scsi/isci/init.c b/drivers/scsi/isci/init.c
index de2aefcf2089..d31884f82f2a 100644
--- a/drivers/scsi/isci/init.c
+++ b/drivers/scsi/isci/init.c
@@ -758,6 +758,7 @@  static __exit void isci_exit(void)
 	sas_release_transport(isci_transport_template);
 }
 
+MODULE_DESCRIPTION("Intel(R) C600 Series Chipset SAS Controller driver");
 MODULE_LICENSE("Dual BSD/GPL");
 MODULE_FIRMWARE(ISCI_FW_NAME);
 module_init(isci_init);
diff --git a/drivers/scsi/mac_scsi.c b/drivers/scsi/mac_scsi.c
index a402c4dc4645..53ee8f84d094 100644
--- a/drivers/scsi/mac_scsi.c
+++ b/drivers/scsi/mac_scsi.c
@@ -550,4 +550,5 @@  static struct platform_driver mac_scsi_driver __refdata = {
 module_platform_driver_probe(mac_scsi_driver, mac_scsi_probe);
 
 MODULE_ALIAS("platform:" DRV_MODULE_NAME);
+MODULE_DESCRIPTION("Macintosh NCR5380 SCSI driver");
 MODULE_LICENSE("GPL");
diff --git a/drivers/scsi/pcmcia/aha152x_stub.c b/drivers/scsi/pcmcia/aha152x_stub.c
index 6a6621728c69..1b54ba51a485 100644
--- a/drivers/scsi/pcmcia/aha152x_stub.c
+++ b/drivers/scsi/pcmcia/aha152x_stub.c
@@ -75,6 +75,7 @@  module_param(synchronous, int, 0);
 module_param(reset_delay, int, 0);
 module_param(ext_trans, int, 0);
 
+MODULE_DESCRIPTION("Adaptec AHA152X-compatible PCMCIA SCSI card driver");
 MODULE_LICENSE("Dual MPL/GPL");
 
 /*====================================================================*/
diff --git a/drivers/scsi/ppa.c b/drivers/scsi/ppa.c
index 8300f0bdddb3..2d9fcc45ad85 100644
--- a/drivers/scsi/ppa.c
+++ b/drivers/scsi/ppa.c
@@ -1155,4 +1155,5 @@  static struct parport_driver ppa_driver = {
 };
 module_parport_driver(ppa_driver);
 
+MODULE_DESCRIPTION("IOMEGA PPA3 parallel port SCSI host adapter driver");
 MODULE_LICENSE("GPL");
diff --git a/drivers/scsi/scsi_common.c b/drivers/scsi/scsi_common.c
index 9c14fdf61037..04749fde1636 100644
--- a/drivers/scsi/scsi_common.c
+++ b/drivers/scsi/scsi_common.c
@@ -12,6 +12,7 @@ 
 #include <asm/unaligned.h>
 #include <scsi/scsi_common.h>
 
+MODULE_DESCRIPTION("SCSI functions used by both the initiator and the target code");
 MODULE_LICENSE("GPL v2");
 
 /* Command group 3 is reserved and should never be used.  */
diff --git a/drivers/scsi/sun3_scsi.c b/drivers/scsi/sun3_scsi.c
index 4a8cc2e8238e..f51702893306 100644
--- a/drivers/scsi/sun3_scsi.c
+++ b/drivers/scsi/sun3_scsi.c
@@ -666,4 +666,5 @@  static struct platform_driver sun3_scsi_driver = {
 module_platform_driver_probe(sun3_scsi_driver, sun3_scsi_probe);
 
 MODULE_ALIAS("platform:" DRV_MODULE_NAME);
+MODULE_DESCRIPTION("Sun3 NCR5380 SCSI controller driver");
 MODULE_LICENSE("GPL");