* [PATCH] INPUT:Change a more clearly name for the input handlers' table
@ 2011-04-08 12:23 wanlong.gao
2011-04-12 6:24 ` Dmitry Torokhov
0 siblings, 1 reply; 3+ messages in thread
From: wanlong.gao @ 2011-04-08 12:23 UTC (permalink / raw)
To: dmitry.torokhov; +Cc: linux-input, linux-kernel, Wanlong Gao
From: Wanlong Gao <wanlong.gao@gmail.com>
Change the input handlers' table name to input_handler_table for
making more clearly sense .
Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
---
drivers/input/input.c | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/input/input.c b/drivers/input/input.c
index d6e8bd8..00511dc 100644
--- a/drivers/input/input.c
+++ b/drivers/input/input.c
@@ -45,7 +45,7 @@ static LIST_HEAD(input_handler_list);
*/
static DEFINE_MUTEX(input_mutex);
-static struct input_handler *input_table[8];
+static struct input_handler *input_handler_table[8];
static inline int is_event_supported(unsigned int code,
unsigned long *bm, unsigned int max)
@@ -1893,11 +1893,11 @@ int input_register_handler(struct input_handler *handler)
INIT_LIST_HEAD(&handler->h_list);
if (handler->fops != NULL) {
- if (input_table[handler->minor >> 5]) {
+ if (input_handler_table[handler->minor >> 5]) {
retval = -EBUSY;
goto out;
}
- input_table[handler->minor >> 5] = handler;
+ input_handler_table[handler->minor >> 5] = handler;
}
list_add_tail(&handler->node, &input_handler_list);
@@ -1933,7 +1933,7 @@ void input_unregister_handler(struct input_handler *handler)
list_del_init(&handler->node);
if (handler->fops != NULL)
- input_table[handler->minor >> 5] = NULL;
+ input_handler_table[handler->minor >> 5] = NULL;
input_wakeup_procfs_readers();
@@ -2062,7 +2062,7 @@ static int input_open_file(struct inode *inode, struct file *file)
return err;
/* No load-on-demand here? */
- handler = input_table[iminor(inode) >> 5];
+ handler = input_handler_table[iminor(inode) >> 5];
if (handler)
new_fops = fops_get(handler->fops);
--
1.7.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] INPUT:Change a more clearly name for the input handlers' table
2011-04-08 12:23 [PATCH] INPUT:Change a more clearly name for the input handlers' table wanlong.gao
@ 2011-04-12 6:24 ` Dmitry Torokhov
2011-04-12 12:01 ` wanlong gao
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Torokhov @ 2011-04-12 6:24 UTC (permalink / raw)
To: wanlong.gao; +Cc: linux-input, linux-kernel
On Fri, Apr 08, 2011 at 08:23:44PM +0800, wanlong.gao@gmail.com wrote:
> From: Wanlong Gao <wanlong.gao@gmail.com>
>
> Change the input handlers' table name to input_handler_table for
> making more clearly sense .
>
Sorry but I think the old name is fine. I'd rather see this table
removed altogether and character devices pushed down into input handlers
themselves.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] INPUT:Change a more clearly name for the input handlers' table
2011-04-12 6:24 ` Dmitry Torokhov
@ 2011-04-12 12:01 ` wanlong gao
0 siblings, 0 replies; 3+ messages in thread
From: wanlong gao @ 2011-04-12 12:01 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: linux-input, linux-kernel
On 4/12/11, Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:
> On Fri, Apr 08, 2011 at 08:23:44PM +0800, wanlong.gao@gmail.com wrote:
>> From: Wanlong Gao <wanlong.gao@gmail.com>
>>
>> Change the input handlers' table name to input_handler_table for
>> making more clearly sense .
>>
>
> Sorry but I think the old name is fine. I'd rather see this table
> removed altogether and character devices pushed down into input handlers
> themselves.
If so , it will be a really hard work .
Thanks a lot.
>
> Thanks.
>
> --
> Dmitry
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-04-12 12:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-08 12:23 [PATCH] INPUT:Change a more clearly name for the input handlers' table wanlong.gao
2011-04-12 6:24 ` Dmitry Torokhov
2011-04-12 12:01 ` wanlong gao
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).