linux-kernel - Re: [PATCH] HID: add chardev to propagate special events of Roccat hardware to userland

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LNX.2.00.1005071305350.9849@pobox.suse.cz>
Date:	Fri, 7 May 2010 13:10:09 +0200 (CEST)
From:	Jiri Kosina <jkosina@...e.cz>
To:	Stefan Achatz <erazor_de@...rs.sourceforge.net>
Cc:	Stephane Chatty <chatty@...c.fr>,
	Jussi Kivilinna <jussi.kivilinna@...et.fi>, wylda@...ny.cz,
	Jerome Vidal <jerom3@...e.fr>, Tejun Heo <tj@...nel.org>,
	linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] HID: add chardev to propagate special events of Roccat
 hardware to userland

On Thu, 6 May 2010, Stefan Achatz wrote:

> This Patch adds a chardev that is used to report special events
> like request for on-screen-display of actual mouse-setting informations,
> request for execution of macro sequences not stored in mouse.
> The device is as generic as possible so that the functionality is usable
> by all (kone and upcomming) roccat device drivers.

Hi Stefan,

what userspace application is supposed to process these events?

> Also small improvements in kone driver are implemented as understanding
> of the device and kernel programming increases.

It would be nice to separate those out to standalone patches, please.

> --- a/drivers/hid/hid-roccat-kone.c
> +++ b/drivers/hid/hid-roccat-kone.c
> @@ -37,6 +37,7 @@
>  #include <linux/module.h>
>  #include <linux/slab.h>
>  #include "hid-ids.h"
> +#include "hid-roccat.h"
>  #include "hid-roccat-kone.h"
>  
>  static void kone_set_settings_checksum(struct kone_settings *settings)
> @@ -630,7 +631,7 @@ static ssize_t kone_sysfs_set_startup_profile(struct device *dev,
>  static ssize_t kone_sysfs_show_driver_version(struct device *dev,
>  		struct device_attribute *attr, char *buf)
>  {
> -	return snprintf(buf, PAGE_SIZE, DRIVER_VERSION "\n");
> +	return snprintf(buf, PAGE_SIZE, ROCCAT_KONE_DRIVER_VERSION "\n");

Do we really want to keep this kind of driver versioning? Git provided 
much more fine-graned versioning anyway. What is it needed for?

> new file mode 100644
> index 0000000..e05d48e
> --- /dev/null
> +++ b/drivers/hid/hid-roccat.c
> @@ -0,0 +1,428 @@
> +/*
> + * Roccat driver for Linux
> + *
> + * Copyright (c) 2010 Stefan Achatz <erazor_de@...rs.sourceforge.net>
> + */
> +
> +/*
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License as published by the Free
> + * Software Foundation; either version 2 of the License, or (at your option)
> + * any later version.
> + */
> +
> +/*
> + * Module roccat is a char device used to report special events of roccat
> + * hardware to userland. These events include requests for on-screen-display of
> + * profile or dpi settings or requests for execution of macro sequences that are
> + * not stored in device. The information in these events depends on hid device
> + * implementation and contains data that is not available in a single hid event
> + * or else hidraw could have been used.
> + * It is inspired by hidraw, but uses only one circular buffer for all readers.

This information could maybe go into the kernel changelog as well. For 
someone not familiar with the driver layout, the difference between 
'roccat' and 'roccat-kone' might be very difficult to understand.

Thanks,

-- 
Jiri Kosina
SUSE Labs, Novell Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/