On Tue, Jun 29, 2010 at 07:06:24PM +0400, Dmitriy Vasil'ev wrote: > I am sorry, I am not a kernel programmer, I just share my code. > I think, kernel maintainers must heck any code before applying it to kernel. > > I have found of the problem: > my editor uses \r\n as end of line symbol. > original code uses \n as end of line > > diff utlility could be more flexible.... > The diff utility does not know on which OS the resulting file will be used so, by default, it reports everything that was changed. And it was a good thing it showd all these differences as we do not really need DOS format in kernel code. > 1. I do not know how many people use encoder without button. > Probably, need correct this code to disable using the button and > free a irq, if button do not uses. > > 2. I do not know what code inside the kernel use the rotary encoder. > If this code there is, probably it need correct these code. Just grep for it. > > My patch, check it please before applying to kernel: > I will assume that it was Outlook that butchered all indentation, but no, it does not apply... It also does not seem to follow our kernel style (and please do not consider us too picky, but we do request everyone follow the coding style so that review process is easier and we can concentrate on more important details). Anyway, I think that mapping rotary actions on EV_KEY does make sense and would be nice to have; the separate gpio/irq for press button is better handled as a separate input device by gpio_keys driver. Thanks. -- Dmitry -- 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
