RE: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-13 Thread Chris Brandt
Hi Geert, Thank you for your review. On Friday, January 13, 2017, Geert Uytterhoeven wrote: > > +The OSTM comes with 2 independent channels. > > +We will use the first channel (OSTM0) as a free running clocksource > > +and the second channel (OSTM1) as a interrupt driven clock event. > > + > >

[PATCH] net: constify mdiobb_ops structures

2017-01-13 Thread Bhumika Goyal
Declare mdiobb_ops structures as const as they are only stored in the ops field of mdiobb_ctrl structures. This field is of type const, so mdiobb_ops structures having this property can be declared const too. Done using Coccinelle: @r disable optional_qualifier@ identifier x; position p; @@

RE: [PATCH] ARM: dts: r7s72100: fix sdhi clock define

2017-01-13 Thread Chris Brandt
Hi Geert, On Friday, January 13, 2017, Geert Uytterhoeven wrote: > > In sh_mobile_sdhi.c, can we change sh_mobile_sdhi_probe() so that if > > there are 2 clocks specified (in DT or platform data), it > > automatically enables the 2nd clock (forever) and just uses the 1st > clock as the on/off

Re: [PATCH] ARM: dts: r7s72100: fix sdhi clock define

2017-01-13 Thread Geert Uytterhoeven
Hi Chris, On Fri, Jan 13, 2017 at 6:16 PM, Chris Brandt wrote: >> > But...that would make me think on boot it would be set to '01' (setting >> > prohibited). >> >> Yeah, running with enabled SDHI core and disabled card detect sounds silly. > > I just did some testing

RE: [PATCH] ARM: dts: r7s72100: fix sdhi clock define

2017-01-13 Thread Chris Brandt
Hi Geert, > > But...that would make me think on boot it would be set to '01' (setting > > prohibited). > > Yeah, running with enabled SDHI core and disabled card detect sounds silly. I just did some testing and with only 1 clock enabled ('01'), the core works but card detect doesn't work. If

Re: [git pull] pinctrl: sh-pfc: Updates for v4.11

2017-01-13 Thread Linus Walleij
On Fri, Jan 13, 2017 at 3:00 PM, Geert Uytterhoeven wrote: > Hi Linus, > > The following changes since commit 7ce7d89f48834cefece7804d38fc5d85382edf77: > > Linux 4.10-rc1 (2016-12-25 16:13:08 -0800) > > are available in the git repository at: > >

Re: [PATCH net] ravb: do not use zero-length alighment DMA request

2017-01-13 Thread Sergei Shtylyov
Hello! On 01/13/2017 04:09 PM, Simon Horman wrote: From: Masaru Nagai Due to alignment requirements of the hardware transmissions are split into two DMA descriptors, a small padding descriptor of 0 - 3 bytes in length OK but you forgot to use "descriptor"

Re: [PATCH] ARM: dts: r7s72100: fix sdhi clock define

2017-01-13 Thread Geert Uytterhoeven
Hi Chris, On Fri, Jan 13, 2017 at 3:44 PM, Chris Brandt wrote: > On Thursday, January 12, 2017, Geert Uytterhoeven wrote: >> This is strange. There are two SDHI channels, but the STBCR12 >> documentation (all versions up to rev. 3.00) says the register has MSTP >> bits

RE: [PATCH 3/3] ARM: dts: rskrza1: add ostm DT support

2017-01-13 Thread Chris Brandt
Hi Geert and Sergei, On Friday, January 13, 2017, Geert Uytterhoeven wrote: > On Thu, Jan 12, 2017 at 8:01 PM, Chris Brandt > wrote: > > --- a/arch/arm/boot/dts/r7s72100.dtsi > > +++ b/arch/arm/boot/dts/r7s72100.dtsi > > @@ -513,7 +513,8 @@ > >

RE: [PATCH] ARM: dts: r7s72100: fix sdhi clock define

2017-01-13 Thread Chris Brandt
Hi Geert, On Thursday, January 12, 2017, Geert Uytterhoeven wrote: > This is strange. There are two SDHI channels, but the STBCR12 > documentation (all versions up to rev. 3.00) says the register has MSTP > bits for four SD host interfaces? > > Can you please enlighten me? Thanks! If you look

[git pull] clk: renesas: Updates for v4.11

2017-01-13 Thread Geert Uytterhoeven
Hi Mike, Stephen, The following changes since commit 7ce7d89f48834cefece7804d38fc5d85382edf77: Linux 4.10-rc1 (2016-12-25 16:13:08 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git tags/clk-renesas-for-v4.11-tag1

Re: [PATCH net] ravb: do not use zero-length alighment DMA request

2017-01-13 Thread Simon Horman
On Fri, Jan 13, 2017 at 02:09:00PM +0100, Simon Horman wrote: > From: Masaru Nagai > > Due to alignment requirements of the hardware transmissions are split into > two DMA descriptors, a small padding descriptor of 0 - 3 bytes in length > followed by a descriptor for

[PATCH net] ravb: do not use zero-length alighment DMA request

2017-01-13 Thread Simon Horman
From: Masaru Nagai Due to alignment requirements of the hardware transmissions are split into two DMA descriptors, a small padding descriptor of 0 - 3 bytes in length followed by a descriptor for rest of the packet. In the case of IP packets the first descriptor

Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS

2017-01-13 Thread Geert Uytterhoeven
Hi Robin, On Fri, Jan 13, 2017 at 1:17 PM, Robin Murphy wrote: > On 13/01/17 11:59, Geert Uytterhoeven wrote: >> On Fri, Jan 13, 2017 at 12:32 PM, Robin Murphy wrote: >>> On 13/01/17 11:07, Geert Uytterhoeven wrote: Add support for

Re: [PATCH] dmaengine: rcar-dmac: unmap slave resource when channel is freed

2017-01-13 Thread Vinod Koul
On Wed, Jan 11, 2017 at 03:39:31PM +0100, Niklas Söderlund wrote: > The slave mapping should be removed together with other channel > resources when the channel is freed. If it's not unmapped it will hang > around forever after the channel is freed. Applied, thanks -- ~Vinod

Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS

2017-01-13 Thread Robin Murphy
On 13/01/17 11:59, Geert Uytterhoeven wrote: > Hi Robin, > > On Fri, Jan 13, 2017 at 12:32 PM, Robin Murphy wrote: >> On 13/01/17 11:07, Geert Uytterhoeven wrote: >>> Add support for DMA_ATTR_FORCE_CONTIGUOUS to the generic IOMMU DMA code. >>> This allows to allocate

Re: [PATCH/RFC v2 net-next] ravb: unmap descriptors when freeing rings

2017-01-13 Thread Simon Horman
On Thu, Jan 12, 2017 at 07:33:51PM +0300, Sergei Shtylyov wrote: > On 01/12/2017 04:18 PM, Simon Horman wrote: > > >... > > > Here, it stop once an untransmitted buffer is encountered... > >>> > >>>Yes, I see that now. > >>> > >>>I wonder if we should: > >>> > >>>a) paramatise

Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS

2017-01-13 Thread Geert Uytterhoeven
Hi Robin, On Fri, Jan 13, 2017 at 12:32 PM, Robin Murphy wrote: > On 13/01/17 11:07, Geert Uytterhoeven wrote: >> Add support for DMA_ATTR_FORCE_CONTIGUOUS to the generic IOMMU DMA code. >> This allows to allocate physically contiguous DMA buffers on arm64 >> systems with

Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS

2017-01-13 Thread Robin Murphy
On 13/01/17 11:07, Geert Uytterhoeven wrote: > Add support for DMA_ATTR_FORCE_CONTIGUOUS to the generic IOMMU DMA code. > This allows to allocate physically contiguous DMA buffers on arm64 > systems with an IOMMU. Can anyone explain what this attribute is actually used for? I've never quite

Re: [PATCH 1/2] dma-mapping: let arch know origin of dma range passed to arch_setup_dma_ops()

2017-01-13 Thread kbuild test robot
-Yushchenko/dma-mapping-let-arch-know-origin-of-dma-range-passed-to-arch_setup_dma_ops/20170113-152733 config: x86_64-randconfig-u0-01131618 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH

Re: [PATCH 2/2] arm64: avoid increasing DMA masks above what hardware supports

2017-01-13 Thread kbuild test robot
-Yushchenko/dma-mapping-let-arch-know-origin-of-dma-range-passed-to-arch_setup_dma_ops/20170113-152733 config: arm64-defconfig (attached as .config) compiler: aarch64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git

[PATCH v4 2/2] dt-bindings: iio: document MAX11100 ADC

2017-01-13 Thread Jacopo Mondi
Add device tree bindings documentation for Maxim MAX11100 single-channel ADC Signed-off-by: Jacopo Mondi --- .../devicetree/bindings/iio/adc/max11100.txt | 19 +++ 1 file changed, 19 insertions(+) create mode 100644

[PATCH v4 1/2] iio: adc: Add Maxim MAX11100 driver

2017-01-13 Thread Jacopo Mondi
From: Jacopo Mondi Add iio driver for Maxim MAX11100 single-channel ADC. Signed-off-by: Jacopo Mondi Tested-by: Marek Vasut --- drivers/iio/adc/Kconfig| 9 +++ drivers/iio/adc/Makefile | 1 +

[PATCH v4 0/2] iio: adc: Add Maxim MAX11100 driver

2017-01-13 Thread Jacopo Mondi
Hello, sending out v4 splitting device tree bindings documentation and actual ADC driver. No changes in driver code since v3. Same question for iio maintainers here: I would like to have clarified the measure unit returned by read_raw(). Currently (value_raw * value_scale) return the ADC input

Re: [PATCH 3/3] ARM: dts: rskrza1: add ostm DT support

2017-01-13 Thread Sergei Shtylyov
Hello! On 1/12/2017 10:01 PM, Chris Brandt wrote: Signed-off-by: Chris Brandt [...] diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi index 247bbe0..974c084 100644 --- a/arch/arm/boot/dts/r7s72100.dtsi +++

Re: [PATCH v3] iio: adc: Add Maxim MAX11100 driver

2017-01-13 Thread jacopo mondi
Hi Geert, On 13/01/2017 09:27, Geert Uytterhoeven wrote: Hi Jacopo, On Thu, Jan 12, 2017 at 11:51 PM, Jacopo Mondi wrote: From: Jacopo Mondi Add IIO driver for Maxim MAX11100 single-channel ADC. Add DT bindings documentation. Signed-off-by:

Re: [PATCH 3/3] ARM: dts: rskrza1: add ostm DT support

2017-01-13 Thread Geert Uytterhoeven
Hi Chris, On Thu, Jan 12, 2017 at 8:01 PM, Chris Brandt wrote: > --- a/arch/arm/boot/dts/r7s72100.dtsi > +++ b/arch/arm/boot/dts/r7s72100.dtsi > @@ -513,7 +513,8 @@ > interrupts =GIC_SPI 103 IRQ_TYPE_EDGE_RISING>; > > -

Re: [PATCH RESEND 0/2] arm64: gen3: enable eMMC HS200 in DT

2017-01-13 Thread Wolfram Sang
> Sorry for missing these. > I have queued them up for v4.11. Thank you! signature.asc Description: PGP signature

Re: [PATCH 1/3] ARM: dts: r7s72100: add ostm clock to device tree

2017-01-13 Thread Geert Uytterhoeven
On Thu, Jan 12, 2017 at 8:01 PM, Chris Brandt wrote: > Signed-off-by: Chris Brandt Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux

Re: [PATCH RESEND 0/2] arm64: gen3: enable eMMC HS200 in DT

2017-01-13 Thread Simon Horman
On Thu, Jan 12, 2017 at 11:43:53PM +0100, Wolfram Sang wrote: > On Wed, Jan 04, 2017 at 06:39:52PM +0100, Wolfram Sang wrote: > > Simon, > > > > as Ulf picked up the driver patches for HS200 support, it is now safe to > > pick > > up the DTS patches, I'd say? I rebased them to the dt-for-4.11

Re: [PATCH] ARM: dts: r7s72100: fix sdhi clock define

2017-01-13 Thread Simon Horman
On Thu, Jan 12, 2017 at 08:34:26PM +, Chris Brandt wrote: > Hi Geert, > > On Thursday, January 12, 2017, Geert Uytterhoeven wrote: > > This is strange. There are two SDHI channels, but the STBCR12 > > documentation (all versions up to rev. 3.00) says the register has MSTP > > bits for four SD

Re: [PATCH 1/2] dt-bindings: document renesas-ostm timer

2017-01-13 Thread Geert Uytterhoeven
Hi Chris, On Thu, Jan 12, 2017 at 7:59 PM, Chris Brandt wrote: > Signed-off-by: Chris Brandt > --- > .../devicetree/bindings/timer/renesas,ostm.txt | 36 > ++ > 1 file changed, 36 insertions(+) > create mode 100644

Re: [PATCH v3] iio: adc: Add Maxim MAX11100 driver

2017-01-13 Thread Geert Uytterhoeven
Hi Jacopo, On Thu, Jan 12, 2017 at 11:51 PM, Jacopo Mondi wrote: > From: Jacopo Mondi > > Add IIO driver for Maxim MAX11100 single-channel ADC. > Add DT bindings documentation. > > Signed-off-by: Jacopo Mondi > Tested-by: Marek