/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
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
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
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
.
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
(), 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
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
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
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
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.
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
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
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
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) ==
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
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
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
.
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.
, 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
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
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
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>
---
/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
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
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
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
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
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
.
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
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
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
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
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
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...@
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
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
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
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
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
(), 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
.
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
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..
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
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!
>>
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
>>>
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
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
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/
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:
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
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
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 +++
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
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
Rob Herring writes:
>> +Example:
>> +ac97: sound@4050 {
>> +compatible = "marvell,pxa2xx-ac97";
>> +reg = < 0x4050 0x1000 >;
>> +interrupts = <14>;
>> +reset-gpio = < 113 GPIO_ACTIVE_HIGH>;
>> +
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
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
;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
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 :
>> [
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
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
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]
>
>
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
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
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
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
-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
cionext.com>
Yeah, I see no other way out to spare namespace polution.
Acked-by: Robert Jarzmik <robert.jarz...@free.fr>
Cheers.
--
Robert
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
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
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
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
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
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>
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
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
,
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 &
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/
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
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
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
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
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.
>
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
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
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
(), 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
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
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
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
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
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
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
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:
>> >> @@ -
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
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
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
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>
---
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
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
601 - 700 of 2627 matches
Mail list logo