[RFC PATCH v2 2/7] ALSA: ac97: add an ac97 bus

2016-05-27 Thread Robert Jarzmik
/sound/ac97/codec.h new file mode 100644 index ..a1c9746c987e --- /dev/null +++ b/include/sound/ac97/codec.h @@ -0,0 +1,111 @@ +/* + * Copyright (C) 2016 Robert Jarzmik <robert.jarz...@free.fr> + * + * This program is free software; you can redistribute it and/or modify + * it und

[RFC PATCH v2 0/7] AC97 device/driver model revamp

2016-05-27 Thread Robert Jarzmik
st more deeply. I'll also test by removing the wm9713 change to see how robust the implementation is. Happy review. -- Robert Robert Jarzmik (7): ALSA: ac97: split out the generic ac97 registers ALSA: ac97: add an ac97 bus ASoC: add new ac97 bus support ASoC: wm9713: add ac97 new bus support ASo

Re: [PATCH 4/5] usb: pxa27x_udc: remove unused function argument

2016-06-16 Thread Robert Jarzmik
ect than "Fixes" : Fixes: 70189a63d408 ("usb: gadget: pxa27x_udc: convert to udc_start/udc_stop") Apart form that: Acked-by: Robert Jarzmik <robert.jarz...@free.fr> Cheers. -- Robert

Re: [PATCH v4 4/7] arm: use the new LED disk activity trigger

2016-06-16 Thread Robert Jarzmik
dm644x-evm.c | 2 +- > arch/arm/mach-omap1/board-osk.c | 2 +- > arch/arm/mach-pxa/spitz.c| 2 +- > 8 files changed, 8 insertions(+), 8 deletions(-) For mach-pxa: Acked-by: Robert Jarzmik <robert.jarz...@free.fr> Cheers. -- Robert

[PATCH v2] mmc: pxamci: fix the device-tree probe deferral path

2016-02-06 Thread Robert Jarzmik
. More broadly, this patch uses devm_*() primitives whenever possible in the probe function. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Since v1: broaden the devm_*() enveloppe --- drivers/mmc/host/pxamci.c | 24 +++- 1 file changed, 7 insertions(+), 17 del

[PATCH v2] net: ethernet: davicom: fix devicetree irq resource

2016-02-06 Thread Robert Jarzmik
(), which handles the deferral. Moreover, since commit Fixes: 7085a7401ba5 ("drivers: platform: parse IRQ flags from resources"), the interrupt trigger flags are honored in platform_get_irq(), so remove the needless code in dm9000. Signed-off-by: Robert Jarzmik <robert.jarz...@f

Re: [PATCH v2 18/51] mtd: docg3: use mtd_set_ecclayout() where appropriate

2016-02-06 Thread Robert Jarzmik
Boris Brezillon <boris.brezil...@free-electrons.com> writes: > Use the mtd_set_ecclayout() helper instead of directly assigning the > mtd->ecclayout field. > > Signed-off-by: Boris Brezillon <boris.brezil...@free-electrons.com> Acked-by: Robert Jarzmik <robert.jarz...@free.fr> Cheers. -- Robert

Re: [PATCH v2 20/51] mtd: docg3: switch to mtd_ooblayout_ops

2016-02-06 Thread Robert Jarzmik
Boris Brezillon <boris.brezil...@free-electrons.com> writes: > Replace the nand_ecclayout definition by the equivalent mtd_ooblayout_ops > definition. > > Signed-off-by: Boris Brezillon <boris.brezil...@free-electrons.com> Acked-by: Robert Jarzmik <robert.jarz...@free.fr> Cheers. -- Robert

[PATCH v2] net: smc91x: propagate irq return code

2016-02-06 Thread Robert Jarzmik
The smc91x driver doesn't honor the probe deferral mechanism when the interrupt source is not yet available, such as one provided by a gpio controller not probed. Fix this by propagating the platform_get_irq() error code as the probe return value. Signed-off-by: Robert Jarzmik <robert.j

[PATCH] ARM: pxa: pxa3xx device-tree support cleanup

2016-02-06 Thread Robert Jarzmik
Clocks, timer and several other drivers have well defined and working device-tree bindings. Clean-up the code to leave only the strict minimum. The final goal will be to remove the lookup array. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- arch/arm/mach-pxa/pxa-dt.

Re: [PATCH] ARM: pxa: fix irq initialization for pxa3xx devicetree

2016-02-06 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > The initialization of interrupt controller for devicetree platform was > done for pxa2xx platforms, but not for pxa3xx ones. > > Fix this my adding the missing initialization call. > > Fixes: 089d03629b04 ("ARM: pxa

Re: [PATCH] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-09 Thread Robert Jarzmik
Vinod Koul <vinod.k...@intel.com> writes: > On Tue, Feb 09, 2016 at 08:20:05AM +0100, Robert Jarzmik wrote: >> Vinod Koul <vinod.k...@intel.com> writes: >> >> > On Mon, Feb 08, 2016 at 03:18:51PM +0100, Robert Jarzmik wrote: >> >> The cu

Re: [PATCH] net: smc911x: convert pxa dma to dmaengine

2016-02-06 Thread Robert Jarzmik
David Miller <da...@davemloft.net> writes: > From: Robert Jarzmik <robert.jarz...@free.fr> > Date: Fri, 05 Feb 2016 22:44:56 +0100 > >> Apart from Alberto who answered he cannot test it by lack of hardware, the >> others didn't answer. >> >> So how ca

Re: [PATCH v2] net: ethernet: davicom: fix devicetree irq resource

2016-02-07 Thread Robert Jarzmik
Sergei Shtylyov writes: >> Moreover, since commit Fixes: 7085a7401ba5 ("drivers: platform: parse > >Fixes: tag here? Heuh no. It doesn't fix this commit, it uses a feature brought by this commit. >> +if (irq_get_trigger_type(dev->irq) ==

Re: [PATCH v2] net: ethernet: davicom: fix devicetree irq resource

2016-02-07 Thread Robert Jarzmik
Francois Romieu <rom...@fr.zoreil.com> writes: > Robert Jarzmik <robert.jarz...@free.fr> : > [...] >> diff --git a/drivers/net/ethernet/davicom/dm9000.c >> b/drivers/net/ethernet/davicom/dm9000.c >> index cf94b72dbacd..2bae5c8c1f85 100644 >> --- a/driv

Re: [PATCH] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-08 Thread Robert Jarzmik
Vinod Koul <vinod.k...@intel.com> writes: > On Mon, Feb 08, 2016 at 03:18:51PM +0100, Robert Jarzmik wrote: >> The current number of requestor lines is limited to 31. This was an >> error of a previous commit, as this number is platform dependent, and is >> actu

Re: [PATCH v2] mmc: pxamci: fix the device-tree probe deferral path

2016-02-08 Thread Robert Jarzmik
Ulf Hansson <ulf.hans...@linaro.org> writes: > On 6 February 2016 at 22:14, Robert Jarzmik <robert.jarz...@free.fr> wrote: >> When the gpio driver is probed after the mmc one, the read/write gpio >> and card detection one return -EPROBE_DEFER. Unfortunately, the memory

[PATCH v3] mmc: pxamci: fix the device-tree probe deferral path

2016-02-08 Thread Robert Jarzmik
. More broadly, this patch uses devm_*() primitives whenever possible in the probe function. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Since v1: broaden the devm_*() enveloppe Since v2: further broaden devm_*(): regulator + clock --- drivers/mmc/host/pxamci.

[PATCH] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-08 Thread Robert Jarzmik
, and in these only the camera has a driver using dma. Fixes: e87ffbdf0697 ("dmaengine: pxa_dma: fix the no-requestor case") Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- drivers/dma/pxa_dma.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --g

Re: [PATCH v5 1/4] media: pxa_camera: fix the buffer free path

2016-02-08 Thread Robert Jarzmik
Guennadi Liakhovetski <g.liakhovet...@gmx.de> writes: > Hi Robert, > > On Tue, 27 Oct 2015, Robert Jarzmik wrote: > >> Guennadi Liakhovetski <g.liakhovet...@gmx.de> writes: >> >> > Hi Robert, >> > >> > Didn't you tell me, that y

Re: [PATCH] net: smc911x: convert pxa dma to dmaengine

2016-02-05 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > David Miller <da...@davemloft.net> writes: > >> From: Robert Jarzmik <robert.jarz...@free.fr> >> Date: Mon, 30 Nov 2015 22:40:28 +0100 >> >>> Convert the dma transfers to be dmaengine based, n

[PATCH 3/4] ARM: pxa: add the number of DMA requestor lines

2016-02-11 Thread Robert Jarzmik
Declare the number of DMA requestor lines per platform : - for pxa25x: 40 requestor lines - for pxa27x: 75 requestor lines - for pxa3xx: 100 requestor lines This information will be used to activate the DMA flow control or not. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> ---

[PATCH 0/4] Fix pxa dma requestor lines

2016-02-11 Thread Robert Jarzmik
/Ian ack on patch 1/4 - Haojian's ack on patch 2/4 - Vinod's ack on patch 4/4 Thanks in advance. -- Robert Robert Jarzmik (4): dma: mmp_pdma: Add the #requestors DT property documentation dmaengine: mmp-pdma: add number of requestors ARM: pxa: add the number of DMA requestor lines

[PATCH 4/4] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-11 Thread Robert Jarzmik
the faulty constant as on pxa[23]xx platforms, only camera, MSL and USB are above requestor 32, and in these only the camera has a driver using dma. Fixes: e87ffbdf0697 ("dmaengine: pxa_dma: fix the no-requestor case") Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- drivers/dm

[PATCH 2/4] dmaengine: mmp-pdma: add number of requestors

2016-02-11 Thread Robert Jarzmik
The DMA chip has a fixed number of requestor lines used for flow control. This number is platform dependent. The pxa_dma dma driver will use this value to activate or not the flow control. There won't be any impact on mmp_pdma driver. Signed-off-by: Robert Jarzmik <robert.jarz...@free

[PATCH 1/4] dma: mmp_pdma: Add the #requestors DT property documentation

2016-02-11 Thread Robert Jarzmik
For pxa based platforms, the number of requestor lines should be specified, so that the driver can check if the flow control should be activated (when a requestor line is asked for) or not. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Documentation/devicetree/bindings/d

Re: [PATCH] mmc: pxamci: fix the device-tree probe deferral path

2016-02-04 Thread Robert Jarzmik
Ulf Hansson <ulf.hans...@linaro.org> writes: > On 3 February 2016 at 23:19, Robert Jarzmik <robert.jarz...@free.fr> wrote: >> When the gpio driver is probed after the mmc one, the read/write gpio >> and card detection one return -EPROBE_DEFER. Unfortunately, the memory

Re: [PATCH] net: ethernet: davicom: fix devicetree irq resource

2016-02-04 Thread Robert Jarzmik
Sergei Shtylyov writes: > Hello. > >Your patch summary prefixes are too verbose, it was enough to say only > "dm9000: ". Well, I don't agree here. The subsystem should be fully specified, at least this is something I require in pxa, something that is also

[PATCH] usb: host: ohci-pxa27x: propagate the irq error code

2016-02-13 Thread Robert Jarzmik
. Even if in this specific case the driver was not broken, because the interrupt controller is always probed before drivers, propagate the proper return code. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- drivers/usb/host/ohci-pxa27x.c | 2 +- 1 file changed, 1 insertion(+), 1 de

Re: [PATCH 1/7] usb: gadget: pxa25x_udc: move register definitions from arch

2016-01-29 Thread Robert Jarzmik
Arnd Bergmann writes: > ixp4xx and pxa25x both use this driver and provide a slightly > different set of register definitions for it. Aside from that, > the definition in the ixp4xx-regs.h header conflicts with the > on in the pxa27x device driver when compile-testing that: > > In

Re: [PATCH 2/7] usb: gadget: pxa25x_udc cleanup

2016-01-29 Thread Robert Jarzmik
h should > not be changing the behavior of the driver, other than using > ioremap() on the platform resource to replace the hardcoded > virtual address pointers. > > Signed-off-by: Arnd Bergmann <a...@arndb.de> Acked-by: Robert Jarzmik <robert.jarz...@free.fr> It's too bad cocc

Re: [PATCH 3/7] usb: gadget: pxa25x_udc: use readl/writel for mmio

2016-01-29 Thread Robert Jarzmik
and after this patch. > > Signed-off-by: Arnd Bergmann <a...@arndb.de> For pxa25x_udc: Acked-by: Robert Jarzmik <robert.jarz...@free.fr> Cheers. -- Robert

Re: [PATCH 1/7] usb: gadget: pxa25x_udc: move register definitions from arch

2016-01-29 Thread Robert Jarzmik
Arnd Bergmann <a...@arndb.de> writes: > On Friday 29 January 2016 10:32:07 Robert Jarzmik wrote: >> > This addresses both issues by moving all the definitions into the >> > pxa25x_udc driver itself. It turns out the only difference between >> > them was

[PATCH] mmc: pxamci: fix again read-only gpio detection polarity

2016-01-28 Thread Robert Jarzmik
but still necessary. Fixes: Fixes: 26d49fe71953 ("mmc: pxamci: fix read-only gpio detection polarity") Reported-by: Andrea Adami <andrea.ad...@gmail.com> Tested-by: Andrea Adami <andrea.ad...@gmail.com> Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> Cc: sta...@

Re: [PATCH] net: smc91x: propagate irq return code

2016-02-01 Thread Robert Jarzmik
Sergei Shtylyov <sergei.shtyl...@cogentembedded.com> writes: > On 02/01/2016 11:41 PM, Robert Jarzmik wrote: > >>>> The smc91x driver doesn't honor the probe deferral mechanism when the >>>> interrupt source is not yet available, such as one provided

Re: [PATCH] net: smc91x: propagate irq return code

2016-02-01 Thread Robert Jarzmik
Sergei Shtylyov <sergei.shtyl...@cogentembedded.com> writes: > Hello. > > On 2/1/2016 1:46 AM, Robert Jarzmik wrote: > >> The smc91x driver doesn't honor the probe deferral mechanism when the >> interrupt source is not yet available, such as one provided by a

Re: [PATCH v2] mtd: nand: pxa3xx_nand: add register access debug

2016-02-01 Thread Robert Jarzmik
Brian Norris writes: > Seems reasonable. I made that change myself, and applied it to > l2-mtd.git. > >> But if you feel it's just bikeshedding then: > > Maybe :) It's all right, that's perfectly fine by me, thanks for the inline change. Cheers. -- Robert

[PATCH] net: smc91x: propagate irq return code

2016-01-31 Thread Robert Jarzmik
The smc91x driver doesn't honor the probe deferral mechanism when the interrupt source is not yet available, such as one provided by a gpio controller not probed. Fix this by propagating the platform_get_irq() error code as the probe return value. Signed-off-by: Robert Jarzmik <robert.j

[PATCH] ARM: pxa: fix irq initialization for pxa3xx devicetree

2016-01-31 Thread Robert Jarzmik
The initialization of interrupt controller for devicetree platform was done for pxa2xx platforms, but not for pxa3xx ones. Fix this my adding the missing initialization call. Fixes: 089d03629b04 ("ARM: pxa: add devicetree code for irq handling") Signed-off-by: Robert Jarzmik &l

[PATCH] net: ethernet: davicom: fix devicetree irq resource

2016-02-03 Thread Robert Jarzmik
(), which handles the deferral. Moreover, since commit Fixes: 7085a7401ba5 ("drivers: platform: parse IRQ flags from resources"), the interrupt trigger flags are honored in platform_get_irq(), so remove the needless code in dm9000. Signed-off-by: Robert Jarzmik <robert.jarz...@f

[PATCH] mmc: pxamci: fix the device-tree probe deferral path

2016-02-03 Thread Robert Jarzmik
. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- drivers/mmc/host/pxamci.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/mmc/host/pxamci.c b/drivers/mmc/host/pxamci.c index 28a057fae0a1..405ec339a1b8 100644 --- a/drivers/mmc/host/pxamci.c +++ b/drive

Re: [REGRESSION] gpio: pxa: change initcall level second attempt

2016-01-30 Thread Robert Jarzmik
Marcel, would you try the patch here after ? I have tested it on my cm-x300 with a devicetree build, let's see if that is a solution to your issue. Cheers. -- Robert ---8<--- >From 5901e6d55061c0cd627cfbf090ef6362c712b3c8 Mon Sep 17 00:00:00 2001 From: Robert Jarzmik <robert.jarz..

Re: [PATCH 0/7] USB changes for rare warnings

2016-02-03 Thread Robert Jarzmik
Felipe Balbi writes: > Hi Arnd, > >> arch/arm/mach-ixp4xx/include/mach/ixp4xx-regs.h | 198 - >> arch/arm/mach-pxa/include/mach/pxa25x-udc.h | 163 > > for arch/arm I need Acked-by from relevant folks. You already have mine for

Re: [PATCH] pinctrl: pxa: export pxa2xx_pinctrl_init()

2016-01-27 Thread Robert Jarzmik
Linus Walleij writes: > On Sat, Jan 23, 2016 at 8:25 PM, Masahiro Yamada > wrote: > >> Building pinctrl-pxa27x.c as a module causes a link error: >> >> ERROR: "pxa2xx_pinctrl_init" [drivers/pinctrl/pxa/pinctrl-pxa27x.ko] >> undefined! >>

Re: [RFC PATCH 1/2] memremap: add arch specific hook for MEMREMAP_WB mappings

2016-02-23 Thread Robert Jarzmik
Dan Williams writes: > On Tue, Feb 23, 2016 at 4:26 AM, Ard Biesheuvel > wrote: >> On 23 February 2016 at 13:03, Ard Biesheuvel >> wrote: >>> On 23 February 2016 at 12:58, Russell King - ARM Linux >>>

Re: [PATCH v5 1/4] media: pxa_camera: fix the buffer free path

2016-02-22 Thread Robert Jarzmik
Guennadi Liakhovetski <g.liakhovet...@gmx.de> writes: > On Sun, 21 Feb 2016, Robert Jarzmik wrote: > Please, have a look at > http://git.linuxtv.org/gliakhovetski/v4l-dvb.git/log/?h=for-4.6-2 > If all is good there, no need for a v6 Thanks for fixing the *_dma_irq

Re: [PATCH v2 2/4] dmaengine: mmp-pdma: add number of requestors

2016-02-23 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > The DMA chip has a fixed number of requestor lines used for flow > control. This number is platform dependent. The pxa_dma dma driver will > use this value to activate or not the flow control. > > There won't be any impa

[PATCH v2] ASoC: wm9713: fix regmap free path

2016-02-22 Thread Robert Jarzmik
initialization code with the non devm_() variant. Fixes: 700dadfefc3d ASoC: wm9713: convert to regmap Cc: Lars-Peter Clausen <l...@metafoo.de> Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Since v1: changed the fix into removing the devm_() variant --- sound/soc/codecs/

Re: [PATCH v1] arm: pxa: fix DT node name for PXA27X usb

2016-02-23 Thread Robert Jarzmik
Sergei Ianovich <ynv...@gmail.com> writes: > Hi Robert,  > > On Mon, 2016-02-22 at 16:35 +0100, Robert Jarzmik wrote: >> You're right, I haven't seen the pxa2xx.dtsi statement before. >> >> As such, could you amend a bit your patch please to : >>  - add:

[PATCH RFC 0/2] pxa3xx devicetree support

2016-02-27 Thread Robert Jarzmik
AC97 rework, that's the reason I'm holding it for now. It's here mainly to show application of 1/2. Any rework ideas would be appreciated. Cheers. -- Robert Robert Jarzmik (2): ARM: dts: pxa3xx: add pincontrol helpers ARM: dts: pxa: add zylonite devicetree board arch/arm/boot/dts/pxa3xx.dtsi

[PATCH RFC 1/2] ARM: dts: pxa3xx: add pincontrol helpers

2016-02-27 Thread Robert Jarzmik
The various pxa3xx variants have a really weird pin scheme assignement, when you want the pin number relative to a known gpio pin. This change adds the various tools to ease up writing the pinmux and pinconf devicetree parts. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- ar

[PATCH RFC 2/2] ARM: dts: pxa: add zylonite devicetree board

2016-02-27 Thread Robert Jarzmik
Add the description of the Zylonite board. This should be the equivalent of arch/arm/mach-pxa/zylonite.c and part of arch/arm/mach-pxa/zylonite_pxa300.c Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- arch/arm/boot/dts/zylonite310.dts | 682 +++

Re: [PATCH] ARM: dts: pxa: fix dma engine node to pxa3xx-nand

2016-02-26 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > Since the switch from mmp_pdma to pxa_dma driver for pxa architectures, > the pxa_dma requires 2 arguments, namely the requestor line and the > requested priority. > > Fix the only left device node which was still pass

Re: [PATCH v2 4/4] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-26 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > Vinod Koul <vinod.k...@intel.com> writes: > >> On Mon, Feb 15, 2016 at 09:57:48PM +0100, Robert Jarzmik wrote: >>> The current number of requestor lines is limited to 31. This was an >>> error of a p

Re: [PATCH 3/4] ASoC: pxa: add binding for pxa2xx-ac97 audio complex

2016-02-26 Thread Robert Jarzmik
Rob Herring writes: >> +Example: >> +ac97: sound@4050 { >> +compatible = "marvell,pxa2xx-ac97"; >> +reg = < 0x4050 0x1000 >; >> +interrupts = <14>; >> +reset-gpio = < 113 GPIO_ACTIVE_HIGH>; >> +

Re: [PATCH v2] dmaengine: pxa_dma: fix cyclic transfers

2016-02-26 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > While testing audio with pxa2xx-ac97, underrun were happening while the > user application was correctly feeding the music. Debug proved that the > cyclic transfer is not cyclic, ie. the last descriptor did not loop on > the

Re: [PATCH 1/4] ASoC: wm9713: add binding for WM9713 codec

2016-02-26 Thread Robert Jarzmik
Mark Brown <broo...@kernel.org> writes: > On Fri, Feb 26, 2016 at 02:33:49AM +0100, Robert Jarzmik wrote: >> Mark Brown <broo...@kernel.org> writes: > >> > It will not be called, the generic AC'97 code will be used. > >> Ok, if it's not called no code

Re: [PATCH 22/50] pinctrl: pxa: Use devm_pinctrl_register() for pinctrl registration

2016-02-26 Thread Robert Jarzmik
;haojian.zhu...@gmail.com> > Cc: Robert Jarzmik <robert.jarz...@free.fr> > Cc: linux-arm-ker...@lists.infradead.org Acked-by: Robert Jarzmik <robert.jarz...@free.fr> Cheers. -- Robert

Re: [PATCH] mtd: nand: pxa3xx_nand: fix dmaengine initialization

2016-02-27 Thread Robert Jarzmik
Ezequiel Garcia <ezequ...@vanguardiasur.com.ar> writes: > On 12 February 2016 at 19:29, Robert Jarzmik <robert.jarz...@free.fr> wrote: >> When the driver is initialized in a pure device-tree platform, the >> driver's probe fails allocating the dma channel : >> [

Re: [PATCH v1] arm: pxa: fix DT node name for PXA27X usb

2016-02-22 Thread Robert Jarzmik
dtsi > and pxa27x.dtsi. > > Signed-off-by: Sergei Ianovich <ynv...@gmail.com> > CC: Robert Jarzmik <robert.jarz...@free.fr> Hi Sergei, You're right, I haven't seen the pxa2xx.dtsi statement before. As such, could you amend a bit your patch please to : - add: Fixes: 0ec1939668e

Re: [PATCH 1/4] ASoC: wm9713: add binding for WM9713 codec

2016-02-25 Thread Robert Jarzmik
Mark Brown writes: >> > It will eumerate the AC'97 bus by itself and does not need the CODEC to >> > be described. > >> I think I still don't get it. > >> So let's rephrase it another way : how will the function wm9713_probe() be >> called, ie. what is the possible function

Re: [PATCH 9/9] ARM: mmp: mark usb_dma_mask as __maybe_unused

2016-02-22 Thread Robert Jarzmik
Arnd Bergmann writes: > This variable may be used by some devices that each have their > on Kconfig symbol, or by none of them, and that causes a build > warning: > > arch/arm/mach-mmp/devices.c:241:12: error: 'usb_dma_mask' defined but not > used [-Werror=unused-variable] > >

Re: [PATCH v2 4/4] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-22 Thread Robert Jarzmik
Vinod Koul <vinod.k...@intel.com> writes: > On Mon, Feb 15, 2016 at 09:57:48PM +0100, Robert Jarzmik wrote: >> The current number of requestor lines is limited to 31. This was an >> error of a previous commit, as this number is platform dependent, and is >> actu

Re: [REGRESSION] gpio: pxa: change initcall level second attempt

2016-01-25 Thread Robert Jarzmik
Marcel Ziswiler writes: > Hi Robert > > I tried latest next-20160122 on Colibri PXA270 with a previously > working device tree and got the following DM9000 Ethernet driver issue: > > [1.062495] dm9000 800.ethernet: insufficient resources > [1.068439] dm9000

[PATCH v2] mtd: nand: pxa3xx_nand: add register access debug

2016-01-25 Thread Robert Jarzmik
Add verbose debug for register accesses. This enables easier debugging by following where and how hardware is stimulated, and how it answers. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- drivers/mtd/nand/pxa3xx_nand.c | 22 +- 1 file changed, 17 insertions

Re: [PATCH 0/4] rotary-encoder: use generic device properties

2016-01-23 Thread Robert Jarzmik
Dmitry Torokhov writes: > Hi, > > This series tries to move the driver away from legacy platform data towards > using platform-independent way of retrieving both configuration parameters and > GPIOs (device_property_read_* and gpiod APIs). > > This is based on top of 2

Re: [PATCH] ARM: pxa: let PXA27x select PINCTRL

2016-01-23 Thread Robert Jarzmik
-single should be used), and I had not the time to create the map for pxa25x. Cheers. -- Robert ---8<--- >From 8e0da8ff579bd38a7923f15a905133814153ebdb Mon Sep 17 00:00:00 2001 From: Robert Jarzmik <robert.jarz...@free.fr> Date: Sun, 1 Nov 2015 20:07:31 +0100 Subject: [PATCH] ARM: pxa: use pin

Re: [PATCH] pinctrl: pxa: export pxa2xx_pinctrl_init()

2016-01-23 Thread Robert Jarzmik
cionext.com> Yeah, I see no other way out to spare namespace polution. Acked-by: Robert Jarzmik <robert.jarz...@free.fr> Cheers. -- Robert

Re: [PATCH] pinctrl: pxa: guard sub-directory with CONFIG_PINCTRL_PXA

2016-01-23 Thread Robert Jarzmik
Masahiro Yamada writes: > 2016-01-24 2:56 GMT+09:00 kbuild test robot : >> Hi Masahiro, >> >> [auto build test ERROR on next-20160122] >> [cannot apply to v4.4-rc8 v4.4-rc7 v4.4-rc6 v4.4] >> [if your patch is applied to the wrong git tree, please

Re: [PATCH] ARM: pxa: let PXA27x select PINCTRL

2016-01-26 Thread Robert Jarzmik
Masahiro Yamada writes: > Hi Robert, > > It's OK if you will this later. > Just disregard mine. Ok, great. I'll submit in the next 4 weeks I think. Cheers. -- Robert

Re: [REGRESSION] gpio: pxa: change initcall level second attempt

2016-01-26 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: >> Have you seen this as well or do you know how exactly that should be >> worked around? > Hi Marcel, > > I haven't seen that before on my devicetree boards, I will try this evening on > top of next-20160125. > &g

Re: [PATCH] ARM: pxa: let PXA27x select PINCTRL

2016-01-24 Thread Robert Jarzmik
Arnd Bergmann <a...@arndb.de> writes: > On Saturday 23 January 2016 23:07:44 Robert Jarzmik wrote: >> I have this patch in my local tree for 2 monthes (see [1]). This is the patch >> I'd rather have. I had not posted it as I didn't think the pinctrl work was >> o

Re: [PATCH 1/4] dma: mmp_pdma: Add the #requestors DT property documentation

2016-02-12 Thread Robert Jarzmik
Arnd Bergmann <a...@arndb.de> writes: > On Thursday 11 February 2016 22:23:15 Robert Jarzmik wrote: >> @@ -12,6 +12,8 @@ Required properties: >> Optional properties: >> - #dma-channels: Number of DMA channels supported by the controller >> (defaul

[PATCH] mtd: nand: pxa3xx_nand: fix dmaengine initialization

2016-02-12 Thread Robert Jarzmik
resource is not acquired through platform resources but by OF bindings. Fix this by ensuring that DMA IO resources are only queried in the non device-tree case. Fixes: 8f5ba31aa565 ("mtd: nand: pxa3xx-nand: switch to dmaengine") Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr>

[PATCH] ARM: dts: pxa: fix dma engine node to pxa3xx-nand

2016-02-12 Thread Robert Jarzmik
configuration, ie. failing all data transfers. Fixes: c943646d1f49 ("ARM: dts: pxa: add dma engine node to pxa3xx-nand") Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- arch/arm/boot/dts/pxa3xx.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/a

[PATCH] dmaengine: pxa_dma: fix cyclic transfers

2016-02-16 Thread Robert Jarzmik
ngine: pxa: add pxa dmaengine driver") Reported-by: Vasily Khoruzhick <anars...@gmail.com> Tested-by: Vasily Khoruzhick <anars...@gmail.com> Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- drivers/dma/pxa_dma.c | 6 ++ 1 file changed, 6 insertions(+) diff --g

[PATCH v2] dmaengine: pxa_dma: fix cyclic transfers

2016-02-16 Thread Robert Jarzmik
, because of an trivial operator issue. This was tested on a pxa27x platform. Fixes: a57e16cf0333 ("dmaengine: pxa: add pxa dmaengine driver") Reported-by: Vasily Khoruzhick <anars...@gmail.com> Tested-by: Vasily Khoruzhick <anars...@gmail.com> Signed-off-by: Robert Jarzmik &

[PATCH] ASoC: wm9713: fix regmap free path

2016-02-19 Thread Robert Jarzmik
this by removing the doubled regmap free. Fixes: 700dadfefc3d ASoC: wm9713: convert to regmap Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- sound/soc/codecs/wm9713.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/sound/soc/codecs/wm9713.c b/sound/soc/

[PATCH 2/4] ASoC: wm9713: add device tree support

2016-02-20 Thread Robert Jarzmik
Add the code to be able to use this codec in a devicetree platform. This is tested with the zylonite pxa310 board. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- sound/soc/codecs/wm9713.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/sound/soc/codecs/wm97

[PATCH 1/4] ASoC: wm9713: add binding for WM9713 codec

2016-02-20 Thread Robert Jarzmik
This adds a binding for the Wolfson WM9713 audio codec. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Documentation/devicetree/bindings/sound/wm9713.txt | 14 ++ 1 file changed, 14 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/wm9713.txt

[PATCH 3/4] ASoC: pxa: add binding for pxa2xx-ac97 audio complex

2016-02-20 Thread Robert Jarzmik
This adds a binding for the Marvell PXA audio complex, available in pxa2xx and pxa3xx variants. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- .../bindings/sound/marvell,pxa2xx-ac97.txt | 25 ++ 1 file changed, 25 insertions(+) create mode

[PATCH 4/4] ASoC: pxa: add pxa2xx-ac97 devicetree support

2016-02-20 Thread Robert Jarzmik
Add the devicetree support, so that the driver can be used in a devicetree platform. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- sound/arm/pxa2xx-ac97-lib.c | 11 +++ sound/soc/pxa/pxa2xx-ac97.c | 10 ++ 2 files changed, 21 insertions(+) diff --git a/sou

Re: [PATCH] net: smc911x: convert pxa dma to dmaengine

2016-02-20 Thread Robert Jarzmik
Hitoshi Mitake <mitake.hito...@gmail.com> writes: > Hi Robert, > > On Sat, Feb 6, 2016 at 6:05 PM, Robert Jarzmik <robert.jarz...@free.fr> wrote: > > Please just keep pinging people to properly test this. > > Okay, let's have another try. >

Re: [PATCH 1/4] ASoC: wm9713: add binding for WM9713 codec

2016-02-20 Thread Robert Jarzmik
Mark Brown <broo...@kernel.org> writes: > On Sat, Feb 20, 2016 at 03:37:56PM +0100, Robert Jarzmik wrote: > >> +WM9713 audio CODEC >> + >> +This devices supports I2C. > > No, it clearly doesn't... Right, it supports AC97. > The problem with doing this is t

[PATCH v2] ASoC: wm9713: fix regmap free path

2016-02-20 Thread Robert Jarzmik
this by removing the doubled regmap free. Fixes: 700dadfefc3d ASoC: wm9713: convert to regmap Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Since v1: fix the diff second hunk, god knows where it came from. --- sound/soc/codecs/wm9713.c | 1 - 1 file changed, 1 deletion(-) diff --git a

Re: [PATCH 1/4] ASoC: wm9713: add binding for WM9713 codec

2016-02-20 Thread Robert Jarzmik
Mark Brown <broo...@kernel.org> writes: > On Sat, Feb 20, 2016 at 07:22:04PM +0100, Robert Jarzmik wrote: >> Mark Brown <broo...@kernel.org> writes: Removed DT people from this conversation. >> > Instead we should be probing at runtime (as the non-ASoC AC'97 code

[PATCH v3] net: ethernet: davicom: fix devicetree irq resource

2016-02-20 Thread Robert Jarzmik
(), which handles the deferral. Moreover, since commit Fixes: 7085a7401ba5 ("drivers: platform: parse IRQ flags from resources"), the interrupt trigger flags are honored in platform_get_irq(), so remove the needless code in dm9000. Signed-off-by: Robert Jarzmik <robert.jarz...@f

Re: [PATCH 3/7] usb: gadget: pxa25x_udc: use readl/writel for mmio

2016-02-20 Thread Robert Jarzmik
Arnd Bergmann writes: > Coming back to the specific pxa25x_udc case: using __raw_* accessors > in the driver would possibly end up breaking the PXA25x machines in > the (very unlikely) case that someone wants to make it work with > big-endian kernels, assuming it does not have the

Re: [PATCH] ARM: pxa: pxa3xx device-tree support cleanup

2016-02-20 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > Clocks, timer and several other drivers have well defined and working > device-tree bindings. Clean-up the code to leave only the strict > minimum. The final goal will be to remove the lookup array. > > Signed-off-by: Robert

Re: [PATCH v2 4/4] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-20 Thread Robert Jarzmik
Robert Jarzmik <robert.jarz...@free.fr> writes: > The current number of requestor lines is limited to 31. This was an > error of a previous commit, as this number is platform dependent, and is > actually : > - for pxa25x: 40 requestor lines > - for pxa27x: 75 requestor

Re: [PATCH 1/4] ASoC: wm9713: add binding for WM9713 codec

2016-02-20 Thread Robert Jarzmik
Mark Brown <broo...@kernel.org> writes: > On Sat, Feb 20, 2016 at 09:32:58PM +0100, Robert Jarzmik wrote: >> Mark Brown <broo...@kernel.org> writes: >> > On Sat, Feb 20, 2016 at 07:22:04PM +0100, Robert Jarzmik wrote: >> I will. By now I fail to see how

Re: [PATCH v5 1/4] media: pxa_camera: fix the buffer free path

2016-02-21 Thread Robert Jarzmik
Guennadi Liakhovetski writes: >> Okay Guennadi, I retested this version on top of v4.5-rc2, still good to >> go. There is a minor conflict in the includes since this submission, and I >> can >> repost a v6 which solves it. > > How did you test it with that patchg #3?? I

Re: [PATCH 4/4] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-15 Thread Robert Jarzmik
Vinod Koul <vinod.k...@intel.com> writes: > On Thu, Feb 11, 2016 at 10:23:18PM +0100, Robert Jarzmik wrote: >> @@ -1399,13 +1405,17 @@ static int pxad_probe(struct platform_device *op) >> return PTR_ERR(pdev->base); >> >> of_id

Re: [PATCH 4/4] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-15 Thread Robert Jarzmik
Vinod Koul <vinod.k...@intel.com> writes: > On Mon, Feb 15, 2016 at 06:24:57PM +0100, Robert Jarzmik wrote: >> Vinod Koul <vinod.k...@intel.com> writes: >> >> > On Thu, Feb 11, 2016 at 10:23:18PM +0100, Robert Jarzmik wrote: >> >> @@ -

[PATCH v2 1/4] dma: mmp_pdma: Add the #dma-requests DT property documentation

2016-02-15 Thread Robert Jarzmik
For pxa based platforms, the number of requestor lines should be specified, so that the driver can check if the flow control should be activated (when a requestor line is asked for) or not. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Since v1: rename of property to #dma-re

[PATCH v2 2/4] dmaengine: mmp-pdma: add number of requestors

2016-02-15 Thread Robert Jarzmik
The DMA chip has a fixed number of requestor lines used for flow control. This number is platform dependent. The pxa_dma dma driver will use this value to activate or not the flow control. There won't be any impact on mmp_pdma driver. Signed-off-by: Robert Jarzmik <robert.jarz...@free

[PATCH v2 1/4] dma: mmp_pdma: Add the #dma-requests DT property documentation

2016-02-15 Thread Robert Jarzmik
For pxa based platforms, the number of requestor lines should be specified, so that the driver can check if the flow control should be activated (when a requestor line is asked for) or not. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Since v1: rename of property to #dma-re

[PATCH v2 3/4] ARM: pxa: add the number of DMA requestor lines

2016-02-15 Thread Robert Jarzmik
Declare the number of DMA requestor lines per platform : - for pxa25x: 40 requestor lines - for pxa27x: 75 requestor lines - for pxa3xx: 100 requestor lines This information will be used to activate the DMA flow control or not. Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> ---

[PATCH v2 4/4] dmaengine: pxa_dma: fix the maximum requestor line

2016-02-15 Thread Robert Jarzmik
the faulty constant as on pxa[23]xx platforms, only camera, MSL and USB are above requestor 32, and in these only the camera has a driver using dma. Fixes: e87ffbdf0697 ("dmaengine: pxa_dma: fix the no-requestor case") Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> --- Since v1

Re: [PATCH v6] serial: rewrite pxa2xx-uart to use 8250_core

2016-02-21 Thread Robert Jarzmik
ime which should make the transition to the new driver faster. > >> If so, great, get the platform maintainer to sign off on this please and >> then resend it. If you're talking about the PXA maintainer, then : Acked-by: Robert Jarzmik <robert.jarz...@free.fr> I'm happy with the patch, all pxa board maintainers have been warned, and there exists a fallback solution. So let's go forward, and if people complain, let them complain and forward them to me. Cheers. -- Robert

<    2   3   4   5   6   7   8   9   10   11   >