diff mbox series

driver/usb MODULE_DESCRIPTION entries to usb modules

Message ID k9XZapJHJVXoO82YNf3IlaNfFbMsR4tzD22KaFWD_x_WL5ksH5DzaCy1knWtCBjSfPWExoNX0f3t4lRVU0UhNiDt4nkxU2juWmEkC4x7P3A=@protonmail.com (mailing list archive)
State Superseded
Headers show
Series driver/usb MODULE_DESCRIPTION entries to usb modules | expand

Commit Message

Rob Gill June 19, 2020, 11:18 a.m. UTC
The user tool modinfo is used to get information on kernel modules, including a
description where it is available.

This patch adds a brief MODULE_DESCRIPTION to the following modules in driver/usb.

isight_firmware
mxuport
u_ether
uas

Signed-off-by: Rob Gill <rrobgill@protonmail.com>
---
 drivers/usb/gadget/function/u_ether.c | 1 +
 drivers/usb/misc/isight_firmware.c    | 1 +
 drivers/usb/serial/mxuport.c          | 1 +
 drivers/usb/storage/uas.c             | 1 +
 4 files changed, 4 insertions(+)

--
2.17.1



Sent with ProtonMail Secure Email.

Comments

Greg KH June 19, 2020, 12:13 p.m. UTC | #1
On Fri, Jun 19, 2020 at 11:18:31AM +0000, Rob Gill wrote:
> The user tool modinfo is used to get information on kernel modules, including a
> description where it is available.
> 
> This patch adds a brief MODULE_DESCRIPTION to the following modules in driver/usb.
> 
> isight_firmware
> mxuport
> u_ether
> uas
> 
> Signed-off-by: Rob Gill <rrobgill@protonmail.com>
> ---
>  drivers/usb/gadget/function/u_ether.c | 1 +
>  drivers/usb/misc/isight_firmware.c    | 1 +
>  drivers/usb/serial/mxuport.c          | 1 +
>  drivers/usb/storage/uas.c             | 1 +
>  4 files changed, 4 insertions(+)
> 
> diff --git a/drivers/usb/gadget/function/u_ether.c b/drivers/usb/gadget/function/u_ether.c
> index fbe96ef1a..579ab41c6 100644
> --- a/drivers/usb/gadget/function/u_ether.c
> +++ b/drivers/usb/gadget/function/u_ether.c
> @@ -1180,3 +1180,4 @@ EXPORT_SYMBOL_GPL(gether_disconnect);
> 
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("David Brownell");
> +MODULE_DESCRIPTION("Ethernet-over-USB link layer utilities for Gadget stack");

Is this really "utilities", or just the driver itself for a usb ethernet
gadget?  I know you took this from the top of the file, but is that
still correct?





> diff --git a/drivers/usb/misc/isight_firmware.c b/drivers/usb/misc/isight_firmware.c
> index 4d30095d6..607c90c22 100644
> --- a/drivers/usb/misc/isight_firmware.c
> +++ b/drivers/usb/misc/isight_firmware.c
> @@ -129,3 +129,4 @@ module_usb_driver(isight_firmware_driver);
> 
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Matthew Garrett <mjg@redhat.com>");
> +MODULE_DESCRIPTION("USB isight camera firmware");

This isn't the firmware itself, but a driver to load the firmware.  So
maybe "USB isight camera firmware loader"?

thanks,

greg k-h
diff mbox series

Patch

diff --git a/drivers/usb/gadget/function/u_ether.c b/drivers/usb/gadget/function/u_ether.c
index fbe96ef1a..579ab41c6 100644
--- a/drivers/usb/gadget/function/u_ether.c
+++ b/drivers/usb/gadget/function/u_ether.c
@@ -1180,3 +1180,4 @@  EXPORT_SYMBOL_GPL(gether_disconnect);

 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("David Brownell");
+MODULE_DESCRIPTION("Ethernet-over-USB link layer utilities for Gadget stack");
diff --git a/drivers/usb/misc/isight_firmware.c b/drivers/usb/misc/isight_firmware.c
index 4d30095d6..607c90c22 100644
--- a/drivers/usb/misc/isight_firmware.c
+++ b/drivers/usb/misc/isight_firmware.c
@@ -129,3 +129,4 @@  module_usb_driver(isight_firmware_driver);

 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Matthew Garrett <mjg@redhat.com>");
+MODULE_DESCRIPTION("USB isight camera firmware");
diff --git a/drivers/usb/serial/mxuport.c b/drivers/usb/serial/mxuport.c
index 2513ee902..c1d70d528 100644
--- a/drivers/usb/serial/mxuport.c
+++ b/drivers/usb/serial/mxuport.c
@@ -1323,3 +1323,4 @@  module_usb_serial_driver(serial_drivers, mxuport_idtable);
 MODULE_AUTHOR("Andrew Lunn <andrew@lunn.ch>");
 MODULE_AUTHOR("<support@moxa.com>");
 MODULE_LICENSE("GPL");
+MODULE_DESCRIPTION("MOXA UPort series driver");
diff --git a/drivers/usb/storage/uas.c b/drivers/usb/storage/uas.c
index d59207111..7f59748c4 100644
--- a/drivers/usb/storage/uas.c
+++ b/drivers/usb/storage/uas.c
@@ -1272,3 +1272,4 @@  MODULE_LICENSE("GPL");
 MODULE_IMPORT_NS(USB_STORAGE);
 MODULE_AUTHOR(
 	"Hans de Goede <hdegoede@redhat.com>, Matthew Wilcox and Sarah Sharp");
+MODULE_DESCRIPTION("USB attached SCSI");