Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-06-10 Thread Wolfram Sang
> > > + /* Synchronization & notification */ > > > + spinlock_t lock; > > > > Why the lock? The core has per-adapter locks anyhow. > > I'm using it to lock the rk3x_i2c struct during interrupts. It's needed there > because an operation can timeout, which means the interrupt can occur at any >

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-06-10 Thread Wolfram Sang
+ /* Synchronization notification */ + spinlock_t lock; Why the lock? The core has per-adapter locks anyhow. I'm using it to lock the rk3x_i2c struct during interrupts. It's needed there because an operation can timeout, which means the interrupt can occur at any time and

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-06-07 Thread Max Schwarz
Hi Wolfram, thanks for the review! > > + > > + /* Settings */ > > + unsigned int scl_frequency; > > + > > + /* Synchronization & notification */ > > + spinlock_t lock; > > Why the lock? The core has per-adapter locks anyhow. I'm using it to lock the rk3x_i2c struct during interrupts.

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-06-07 Thread Max Schwarz
Hi Wolfram, thanks for the review! + + /* Settings */ + unsigned int scl_frequency; + + /* Synchronization notification */ + spinlock_t lock; Why the lock? The core has per-adapter locks anyhow. I'm using it to lock the rk3x_i2c struct during interrupts. It's needed

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-06-02 Thread Wolfram Sang
Hi Max, here is the review: > diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig > index c94db1c..f973632 100644 > --- a/drivers/i2c/busses/Kconfig > +++ b/drivers/i2c/busses/Kconfig > @@ -663,6 +663,16 @@ config I2C_PXA_SLAVE > is necessary for systems where the PXA

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-06-02 Thread Wolfram Sang
Hi Max, here is the review: diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig index c94db1c..f973632 100644 --- a/drivers/i2c/busses/Kconfig +++ b/drivers/i2c/busses/Kconfig @@ -663,6 +663,16 @@ config I2C_PXA_SLAVE is necessary for systems where the PXA may be a

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-05-26 Thread Heiko Stübner
Hi, Am Montag, 19. Mai 2014, 11:32:55 schrieb Max Schwarz: > Driver for the native I2C adapter found in Rockchip RK3xxx SoCs. > > Configuration is only possible through devicetree. The driver is > interrupt driven and supports the I2C_M_IGNORE_NAK mangling bit. > > Signed-off-by: Max Schwarz

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-05-26 Thread Heiko Stübner
Hi, Am Montag, 19. Mai 2014, 11:32:55 schrieb Max Schwarz: Driver for the native I2C adapter found in Rockchip RK3xxx SoCs. Configuration is only possible through devicetree. The driver is interrupt driven and supports the I2C_M_IGNORE_NAK mangling bit. Signed-off-by: Max Schwarz

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-05-20 Thread Maxime Ripard
On Mon, May 19, 2014 at 11:32:55AM +0200, Max Schwarz wrote: > Driver for the native I2C adapter found in Rockchip RK3xxx SoCs. > > Configuration is only possible through devicetree. The driver is > interrupt driven and supports the I2C_M_IGNORE_NAK mangling bit. > > Signed-off-by: Max Schwarz

Re: [PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-05-20 Thread Maxime Ripard
On Mon, May 19, 2014 at 11:32:55AM +0200, Max Schwarz wrote: Driver for the native I2C adapter found in Rockchip RK3xxx SoCs. Configuration is only possible through devicetree. The driver is interrupt driven and supports the I2C_M_IGNORE_NAK mangling bit. Signed-off-by: Max Schwarz

[PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-05-19 Thread Max Schwarz
Driver for the native I2C adapter found in Rockchip RK3xxx SoCs. Configuration is only possible through devicetree. The driver is interrupt driven and supports the I2C_M_IGNORE_NAK mangling bit. Signed-off-by: Max Schwarz --- The driver cannot be used without a proper clock driver supporting

[PATCH v4] i2c: add driver for Rockchip RK3xxx SoC I2C adapter

2014-05-19 Thread Max Schwarz
Driver for the native I2C adapter found in Rockchip RK3xxx SoCs. Configuration is only possible through devicetree. The driver is interrupt driven and supports the I2C_M_IGNORE_NAK mangling bit. Signed-off-by: Max Schwarz max.schw...@online.de --- The driver cannot be used without a proper