Hi Thomas,Hi Dmitry,
On Thu, Oct 08, 2009 at 02:59:12PM +0800, Thomas Chou wrote:
This patch adds a new SERIO driver to support the Altera UniversityThank you for the patch, it looks like it is reasonable written although
Program PS/2 controller.
it should do request_mem_region for the IO memory region it tries to
remap and also IO addresses should not be cast to unsigned int but
rather 'void __iomem *'. I also don;t see the reason for it to depend on
EMBEDDED since the things that depend on EMBEDDED are usually features
that are used almost by everyone and only in case of embedded arch you
may want to turn them off to save some memory.
I also prefer even static functions to have the driver name as their
prefix - this way if I see a backtrace I know exactly which module is
involved.
I made a small patch on top of yours, please give it a try and if it did
not break anything then I will fold it all together and queue for
2.6.33.
Thanks!