Re: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread Grant Likely
On Mon, Jun 30, 2008 at 02:00:12PM -0600, John Linn wrote: > > > >Since xps2_recv() and xps2_send() are used by the sxps2_* routines and > >by the irq handler, they should be moved to about them in this file. > > > > OK, keeping them close makes maintenance easier I guess. > Oops, there is a typ

RE: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread John Linn
Thanks Grant. Sounds like there's a few things that not everyone does the same based on comments from others. In general, all pretty easy changes that in general make sense. Thanks, John >On Mon, Jun 30, 2008 at 07:24:48AM -0700, John Linn wrote: >> Added a new driver for Xilinx XPS PS2 IP. Thi

RE: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread John Linn
werpc: Xilinx: PS2: Added new XPS PS2 driver >>>>> "John" == John Linn <[EMAIL PROTECTED]> writes: > > Added a new driver for Xilinx XPS PS2 IP. This driver is > > a flat driver to better match the Linux driver pattern. > >This should proba

Re: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread Grant Likely
On Mon, Jun 30, 2008 at 02:10:23PM -0400, Dmitry Torokhov wrote: > Hi Grant, John, > > On Mon, Jun 30, 2008 at 11:16:28AM -0600, Grant Likely wrote: > > On Mon, Jun 30, 2008 at 07:24:48AM -0700, John Linn wrote: > > > +/* > > > + * sxps2_write() sends a byte out through the PS/2 interface. > > >

Re: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread Dmitry Torokhov
Hi Grant, John, On Mon, Jun 30, 2008 at 11:16:28AM -0600, Grant Likely wrote: > On Mon, Jun 30, 2008 at 07:24:48AM -0700, John Linn wrote: > > +config SERIO_XILINX_XPS_PS2 > > + tristate "Xilinx XPS PS/2 Controller Support" > > + help > > + This driver supports XPS PS/2 IP from Xilinx EDK.

Re: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread Grant Likely
On Mon, Jun 30, 2008 at 07:24:48AM -0700, John Linn wrote: > Added a new driver for Xilinx XPS PS2 IP. This driver is > a flat driver to better match the Linux driver pattern. > > Signed-off-by: Sadanand <[EMAIL PROTECTED]> > Signed-off-by: John Linn <[EMAIL PROTECTED]> I don't know much about th

RE: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread John Linn
Thanks Peter, will do. -Original Message- From: Peter Korsgaard [mailto:[EMAIL PROTECTED] On Behalf Of Peter Korsgaard Sent: Monday, June 30, 2008 8:45 AM To: John Linn Cc: linuxppc-dev@ozlabs.org; Sadanand Mutyala Subject: Re: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

Re: [PATCH] powerpc: Xilinx: PS2: Added new XPS PS2 driver

2008-06-30 Thread Peter Korsgaard
> "John" == John Linn <[EMAIL PROTECTED]> writes: > Added a new driver for Xilinx XPS PS2 IP. This driver is > a flat driver to better match the Linux driver pattern. This should probably go to the linux-input@ list as well. > Signed-off-by: Sadanand <[EMAIL PROTECTED]> > Signed-off-by: