Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-28 Thread Richard Röjfors
Richard Röjfors wrote: Ben Dooks wrote: On Sun, Jan 24, 2010 at 06:33:20PM +0100, Richard R?jfors wrote: + +#define XIIC_MSB_OFFSET 0 +#define XIIC_REG_OFFSET (0x100+XIIC_MSB_OFFSET) givevn you're running everything through indirect read/write calls, how about doing it in there? Could do,

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-26 Thread Ben Dooks
On Sun, Jan 24, 2010 at 06:33:20PM +0100, Richard R?jfors wrote: Hi Ben, Thanks for the feedback! Comments below... On 1/24/10 4:49 PM, Ben Dooks wrote: On Tue, Jan 19, 2010 at 05:01:16PM +0100, Richard R?jfors wrote: This patch adds support for the Xilinx XPS IIC Bus Interface. The

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-26 Thread Ben Dooks
On Sun, Jan 24, 2010 at 07:15:26PM +0100, Wolfram Sang wrote: + * i2c-xiic.c the directory path to the file would have been nice. Will update Ben, are you sure? Those filenames (especially with directories) easily go stale when reorganizing the tree. I said nice, not esential.

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-26 Thread Richard Röjfors
Ben Dooks wrote: On Sun, Jan 24, 2010 at 06:33:20PM +0100, Richard R?jfors wrote: + +#define XIIC_MSB_OFFSET 0 +#define XIIC_REG_OFFSET (0x100+XIIC_MSB_OFFSET) givevn you're running everything through indirect read/write calls, how about doing it in there? Could do, bad thing is that it

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-24 Thread Ben Dooks
On Tue, Jan 19, 2010 at 05:01:16PM +0100, Richard R?jfors wrote: This patch adds support for the Xilinx XPS IIC Bus Interface. The driver uses the dynamic mode, supporting to put several I2C messages in the FIFO to reduce the number of interrupts. It has the same feature as ocores, it can

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-24 Thread Richard Röjfors
Hi Ben, Thanks for the feedback! Comments below... On 1/24/10 4:49 PM, Ben Dooks wrote: On Tue, Jan 19, 2010 at 05:01:16PM +0100, Richard R?jfors wrote: This patch adds support for the Xilinx XPS IIC Bus Interface. The driver uses the dynamic mode, supporting to put several I2C messages in

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-24 Thread Wolfram Sang
+ * i2c-xiic.c the directory path to the file would have been nice. Will update Ben, are you sure? Those filenames (especially with directories) easily go stale when reorganizing the tree. -- Pengutronix e.K. | Wolfram Sang| Industrial Linux

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-24 Thread Jean Delvare
On Sun, 24 Jan 2010 19:15:26 +0100, Wolfram Sang wrote: + * i2c-xiic.c the directory path to the file would have been nice. Will update Ben, are you sure? Those filenames (especially with directories) easily go stale when reorganizing the tree. FWIW, I agree with Wolfram on

Re: [PATCH RESEND 1/1] i2c: Add support for Xilinx XPS IIC Bus Interface

2010-01-24 Thread Peter Korsgaard
Wolfram == Wolfram Sang w.s...@pengutronix.de writes: + * i2c-xiic.c the directory path to the file would have been nice. Will update Wolfram Ben, are you sure? Those filenames (especially with Wolfram directories) easily go stale when reorganizing the tree. Exactly, that's