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

Hi Wolfram, On Wed, Dec 17, 2014 at 7:58 AM, Wolfram Sang w...@the-dreams.de wrote: 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

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

On Wed, Dec 17, 2014 at 09:04:52AM +0100, Geert Uytterhoeven wrote: Hi Wolfram, On Wed, Dec 17, 2014 at 7:58 AM, Wolfram Sang w...@the-dreams.de wrote: On Mon, Dec 15, 2014 at 02:37:04PM +0100, Geert Uytterhoeven wrote: If NO_DMA=y: drivers/built-in.o: In function

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

Hi Wolfram, On Wed, Dec 17, 2014 at 9:39 AM, Wolfram Sang w...@the-dreams.de wrote: On Wed, Dec 17, 2014 at 09:04:52AM +0100, Geert Uytterhoeven wrote: On Wed, Dec 17, 2014 at 7:58 AM, Wolfram Sang w...@the-dreams.de wrote: On Mon, Dec 15, 2014 at 02:37:04PM +0100, Geert Uytterhoeven wrote:

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

If ARM and SUPERH imply HAS_DMA, that formula is almost identical to the one I proposed in my patch? Ehrm, true :) depends on HAS_DMA depends on SUPERH || ARCH_SHMOBILE || COMPILE_TEST I just prefer to split dependencies in two lines where it makes sense (hard dependencies

Re: [PATCH 1/2] i2c: i2c-mv64xxx: use BIT() macro for register value definitions

On Thu, Dec 11, 2014 at 05:33:45PM +0100, Thomas Petazzoni wrote: Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com Applied to for-current, thanks! signature.asc Description: Digital signature

Re: [PATCH 2/2] i2c: i2c-mv64xxx: rework offload support to fix several problems

On Thu, Dec 11, 2014 at 05:33:46PM +0100, Thomas Petazzoni wrote: Originally, the I2C controller supported by the i2c-mv64xxx driver requires a lot of software support: an interrupt is generated at each step of an I2C transaction (after the start bit, after sending the address, etc.) and the

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

On Tue, Dec 16, 2014 at 01:31:25PM +0100, Wolfram Sang wrote: 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 Applied

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

On Tue, Dec 16, 2014 at 01:31:26PM +0100, Wolfram Sang wrote: 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

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

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':

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

Hi Wolfram, Wolfram Sang w...@the-dreams.de writes: 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

[PATCH] input: adxl34x: Add OF match support

The I2C subsystem can match devices without explicit OF support based on the part of their compatible property after the comma. However, this mechanism uses the first compatible value only. For adxl34x OF device nodes the compatible property should list the more specific adi,adxl345 or adi,adxl346