Re: [PATCH v2] scsi: add missing MODULE_DESCRIPTION() macros

From: Jeff Johnson
Date: Mon Jun 10 2024 - 10:57:55 EST


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