Re: [PATCHv1 1/7] dt-bindings: use isil prefix for Intersil

2014-12-16 Thread Uwe Kleine-König
Hello Arnaud, On Tue, Dec 16, 2014 at 01:22:27AM +0100, Arnaud Ebalard wrote: This patch fixes DT documentation to reference isil (NASDAQ symbol and the most used prefix inside the kernel) for all Intersil products. It reverts both 7c75c1d5e72b (dt-bindings: Document deprecated device

Re: [PATCH] of: i2c: Add DT bindings for idle states to PCA954x mux driver

2014-12-16 Thread Alexander Sverdlin
Hello Laurent, On 15/12/14 09:40, ext Laurent Pinchart wrote: of: i2c: Add DT bindings for idle states to PCA954x mux driver Introduce two new device tree bindings to specify idle state of PCA954x family of I2C multiplexors: - idle-state: specifies particular child bus to be selected in

Re: [PATCH] of: i2c: Add DT bindings for idle states to PCA954x mux driver

2014-12-16 Thread Laurent Pinchart
Hi Alexander, On Tuesday 16 December 2014 09:09:22 Alexander Sverdlin wrote: Hello Laurent, On 15/12/14 09:40, ext Laurent Pinchart wrote: of: i2c: Add DT bindings for idle states to PCA954x mux driver Introduce two new device tree bindings to specify idle state of PCA954x family

Re: [RFC 1/2] i2c: sh_mobile: refactor DMA setup

2014-12-16 Thread Wolfram Sang
Hm, alignment style changed... I am not too strict on that as long as the result is somewhat readable. I'd be more interested in your thoughts about the general approach taken in this series. signature.asc Description: Digital signature

Re: [RFC 2/2] i2c: sh_mobile: rework deferred probing

2014-12-16 Thread Wolfram Sang
On Wed, Dec 10, 2014 at 02:44:58PM +0100, Geert Uytterhoeven wrote: Hi Wolfram, On Wed, Dec 10, 2014 at 2:21 PM, Wolfram Sang w...@the-dreams.de wrote: DMA is opt-in for this driver. So, we can't use deferred probing in probe, because our driver would get endlessly deferred if DMA support

[PATCH 2/2] i2c: sh_mobile: rework deferred probing

2014-12-16 Thread Wolfram Sang
From: Wolfram Sang wsa+rene...@sang-engineering.com DMA is opt-in for this driver. So, we can't use deferred probing for requesting DMA channels in probe, because our driver would get endlessly deferred if DMA support is compiled in AND the DMA driver is missing. Because we can't know when the

[PATCH 1/2] i2c: sh_mobile: refactor DMA setup

2014-12-16 Thread Wolfram Sang
From: Wolfram Sang wsa+rene...@sang-engineering.com Refactor DMA setup to keep the errno so we can implement better deferred probe support in the next step. Signed-off-by: Wolfram Sang wsa+rene...@sang-engineering.com --- drivers/i2c/busses/i2c-sh_mobile.c | 35

[PATCH 0/2] i2c: sh_mobile: handle deferred probe properly when requesting DMA channels

2014-12-16 Thread Wolfram Sang
Here is the resend of my RFC series. Only patch description updates and whitespace fixes since then. But it seems we agreed on this approach based on the discussions so far: Geert's discovery [1], the RFC [2]. See there for a complete description. [1]

Re: [PATCH] of: i2c: Add DT bindings for idle states to PCA954x mux driver

2014-12-16 Thread Alexander Sverdlin
Hello Laurent, On 16/12/14 12:07, ext Laurent Pinchart wrote: of: i2c: Add DT bindings for idle states to PCA954x mux driver Introduce two new device tree bindings to specify idle state of PCA954x family of I2C multiplexors: - idle-state: specifies particular child bus to be selected in

Re: [PATCH] of: i2c: Add DT bindings for idle states to PCA954x mux driver

2014-12-16 Thread Laurent Pinchart
Hi Alexander, On Tuesday 16 December 2014 13:49:13 Alexander Sverdlin wrote: On 16/12/14 12:07, ext Laurent Pinchart wrote: of: i2c: Add DT bindings for idle states to PCA954x mux driver Introduce two new device tree bindings to specify idle state of PCA954x family of I2C

Re: [PATCHv1 1/7] dt-bindings: use isil prefix for Intersil

2014-12-16 Thread Arnaud Ebalard
Hello Uwe, Uwe Kleine-König u.kleine-koe...@pengutronix.de writes: On Tue, Dec 16, 2014 at 01:22:27AM +0100, Arnaud Ebalard wrote: This patch fixes DT documentation to reference isil (NASDAQ symbol and the most used prefix inside the kernel) for all Intersil products. It reverts both

Re: [PATCH] of: i2c: Add DT bindings for idle states to PCA954x mux driver

2014-12-16 Thread Alexander Sverdlin
Hello Laurent! On 16/12/14 14:38, ext Laurent Pinchart wrote: I understand the use cases of the idle-disconnect property. What do you use idle-state for, when the idle state is different from disconnecting the bus ? We do not have a use case for that. It was done only for the sake of

Re: [RFC 1/2] i2c: sh_mobile: refactor DMA setup

2014-12-16 Thread Sergei Shtylyov
Hello. On 12/16/2014 02:34 PM, Wolfram Sang wrote: Hm, alignment style changed... I am not too strict on that as long as the result is somewhat readable. I'd be more interested in your thoughts about the general approach taken in this series. Sorry, I'm not very familiar with

[PATCHv2 2/8] dt-bindings: use isil prefix for Intersil in I2C trivial-devices.txt

2014-12-16 Thread Arnaud Ebalard
This patch fixes I2C trivial-devices.txt DT documentation file to reference isil (NASDAQ symbol and the most used prefix inside the kernel) for Intersil. It reverts 7c75c1d5e72b (dt-bindings: Document deprecated device vendor name to fix related warning). Signed-off-by: Arnaud Ebalard

[PATCHv2 0/8] from now on, use 'isil' as prefix for Intersil

2014-12-16 Thread Arnaud Ebalard
As suggested by Jason on a v0, I made a specific series for that work, which intend to remove all remaining 'isl' prefix for Intersil to only have 'isil'. More details below. When Intersil ISL12057 driver was introduced by commit 70e123373c05 (rtc: Add support for Intersil ISL12057 I2C RTC

Re: [PATCHv2 2/8] dt-bindings: use isil prefix for Intersil in I2C trivial-devices.txt

2014-12-16 Thread Wolfram Sang
On Tue, Dec 16, 2014 at 10:19:53PM +0100, Arnaud Ebalard wrote: This patch fixes I2C trivial-devices.txt DT documentation file to reference isil (NASDAQ symbol and the most used prefix inside the kernel) for Intersil. It reverts 7c75c1d5e72b (dt-bindings: Document deprecated device vendor

Re: [PATCH] i2c: sh_mobile: I2C_SH_MOBILE should depend on HAS_DMA

2014-12-16 Thread Wolfram Sang
On Mon, Dec 15, 2014 at 02:37:04PM +0100, Geert Uytterhoeven wrote: If NO_DMA=y: drivers/built-in.o: In function `sh_mobile_i2c_dma_unmap': i2c-sh_mobile.c:(.text+0x60de42): undefined reference to `dma_unmap_single' drivers/built-in.o: In function `sh_mobile_i2c_xfer_dma':