Dmitry Torokhov wrote: > >> The original logic for putting next to the keyboard.c was because they >> are both fighting for the same HW. Please let me know if this changes >> your mind, else I will go ahead and move the keyboard implementation >> into the kernel/debug/kdb. >> >> > > It does not behause in 2.6 the driver talking to the i8042 is called > i8042.c and it is located in drivers/input/serio. > drivers/char/keyboard.c does not talk directly to hardware anymore and > pc_keyb.c is long gone. > > Given the fact that your code does not implement a character device I do > not see any reason for it to live in drivers/char. >
I buy your logic. I have moved the kdb_keyboard.c to kernel/debug/kdb/kdb_keyboard.c. Can I assume we have reached closure on this patch for the short term? Eventually I would like to to get rid of kdb_keyboard.c entirely and replace it with some kind of way to atomically poll the input devices, but this is longer term. Thanks, Jason. ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ Kgdb-bugreport mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
