Re: [PATCH 2/2] HID: wacom: Add module description

[Jiri Kosina <jkosina@xxxxxxx>]

On Wed, 28 Mar 2012, Przemo Firszt wrote:

> Add description for hid-wacom module: "Driver for Wacom Graphire Bluetooth
> and Wacom Intuos4 WL"
> 
> Signed-off-by: Przemo Firszt <przemo@xxxxxxxxx>
> ---
>  drivers/hid/hid-wacom.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/hid/hid-wacom.c b/drivers/hid/hid-wacom.c
> index f130f40..fb0fe16 100644
> --- a/drivers/hid/hid-wacom.c
> +++ b/drivers/hid/hid-wacom.c
> @@ -687,5 +687,5 @@ static void __exit wacom_exit(void)
>  
>  module_init(wacom_init);
>  module_exit(wacom_exit);
> +MODULE_DESCRIPTION("Driver for Wacom Graphire Bluetooth and Wacom Intuos4 WL");

I am a little bit unsure about having the information about supported 
models duplicate in Kconfig and MODULE_DESCRIPTION() ... it's just adding 
more places that we will forget to update once support for a new device 
line is added.

Applied, but not fully convinced.

Thanks,

-- 
Jiri Kosina
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html