On 9/1/2010 10:53 AM, rajeev wrote: >>> +struct spear_kbd { >>> >> + struct input_dev *input; >>> >> + void __iomem *io_base; /* Keyboard Base Address */ >>> >> + struct clk *clk; >>> >> + int *keymap; >> > >> > You need a copy of keymap here so that userspace can modify it safely >> > via EVIOCSKEYCODE. >> > > We have one copy of struct spear_kbd per device structure. I think it > will be fine if we change this structure only. And so don't need to copy > another structure in driver. > Sorry!! I wanted to say "keymap array" not "struct spear_kbd" in above explanation. rajeev. -- 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