Re: [PATCH 00/55]: ARM: OMAP2+: PRCM move to drivers

2014-04-01 Thread Tero Kristo
On 04/01/2014 01:09 AM, Tony Lindgren wrote: * Tero Kristo t-kri...@ti.com [140331 08:20]: Hi, This set is continuation for the work started earlier to cleanup the CM/PRM and attempt to make it a separate driver. This set depends on these two sets: CM/PRM cleanup set:

[PATCH 1/1] ARM: dts: omap5: Add clocks to USB3 PHY node

2014-04-01 Thread Roger Quadros
The USB3 PHY driver (ti-pipe3) was updated so that the relevant clock phandles are expected in the DT node. Provide the necessary clocks. Reported-by: Kishon Vijay Abraham I kis...@ti.com Signed-off-by: Roger Quadros rog...@ti.com --- arch/arm/boot/dts/omap5.dtsi | 6 ++ 1 file changed, 6

[PATCH 0/1] OMAP5: USB3 fix for 3.15

2014-04-01 Thread Roger Quadros
Hi Tony, The following patch fixes USB3 on OMAP5 on linux-next. There was a patch [1] that added named clocks to the USB3 phy driver but somehow the dts part got missed out. Due to that USB3 on OMAP5 is currently broken on linux-next and this patch should fix it. cheers, -roger [1] -

[PATCH v2 12/14] dma: edma: Prefix debug prints where the text were identical in prep callbacks

2014-04-01 Thread Peter Ujfalusi
prep_slave_sg and prep_dma_cyclic callbacks have mostly same failure cases with the same texts printed in case we hit them. It helps when debugging if we know exactly which callback generated the errors. At the same time change the debug level for descriptor allocation failure from dbg to err

[PATCH v2 13/14] dma: edma: Add channel number to debug prints

2014-04-01 Thread Peter Ujfalusi
It helps to identify issues if we have some information regarding to the channel which the event is associated. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- drivers/dma/edma.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/dma/edma.c

[PATCH v2 14/14] dma: edma: Print the direction value as well when it is not supported

2014-04-01 Thread Peter Ujfalusi
In case of not supported direction it is better to print the direction also. It is unlikely, but in such an event it helps with the debugging. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- drivers/dma/edma.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 11/14] dma: edma: Reduce debug print verbosity for non verbose debugging

2014-04-01 Thread Peter Ujfalusi
Do not print the paRAM information when verbose debugging is not asked and also reduce the number of lines printed in edma_prep_dma_cyclic() Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- drivers/dma/edma.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git

[PATCH v2 09/14] dma: edma: Implement device_slave_caps callback

2014-04-01 Thread Peter Ujfalusi
With the callback implemented omap-dma can provide information to client drivers regarding to supported address widths, directions, residue granularity, etc. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- drivers/dma/edma.c | 18 ++ 1 file changed, 18 insertions(+) diff

[PATCH v2 05/14] arm: common: edma: Select event queue 1 as default when booted with DT

2014-04-01 Thread Peter Ujfalusi
Use the EVENTQ_1 for default and leave the EVENTQ_0 to be used by high priority channels, like audio. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- arch/arm/common/edma.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/arm/common/edma.c

[PATCH v2 06/14] arm: common: edma: Save the number of event queues/TCs

2014-04-01 Thread Peter Ujfalusi
For later use save the number of queues available for the CC. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- arch/arm/common/edma.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c index 19520e2519d9..be267b2080be 100644 ---

[PATCH v2 10/14] dma: edma: Simplify direction configuration in edma_config_pset()

2014-04-01 Thread Peter Ujfalusi
We only support DEV_TO_MEM or MEM_TO_DEV directions with edma driver and the check for the direction has been already done in the function calling edma_config_pset(). The error reporting is redundant and also the else if () can be removed. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com ---

[PATCH v2 07/14] arm: common: edma: API to request non default queue for a channel

2014-04-01 Thread Peter Ujfalusi
When using eDMA3 via dmaengine all dma channels will use the default queue. Since during request time we do not have means to change this it need to be done later, before the DMA has been started. With the added function it is possible to move the channel to a non default queue if it is possible,

[PATCH v2 08/14] DMA: edma: Use different eventq for cyclic channels

2014-04-01 Thread Peter Ujfalusi
To improve latency with cyclic DMA operation it is preferred to use different eventq/tc than the default which is used by all other drivers (mmc, spi, i2c, etc). When preparing the cyclic dma ask for non default queue for the channel which is going to be used with cyclic mode. Signed-off-by:

[PATCH v2 03/14] dma: edma: Add support for DMA_PAUSE/RESUME operation

2014-04-01 Thread Peter Ujfalusi
Pause/Resume can be used by the audio stack when the stream is paused/resumed The edma platform code has support for this and the legacy audio stack used this. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- drivers/dma/edma.c | 28 1 file changed, 28

[PATCH v2 04/14] dma: edma: Set DMA_CYCLIC capability flag

2014-04-01 Thread Peter Ujfalusi
Indicate that the edma dmaengine driver has support for cyclic mode. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- arch/arm/common/edma.c | 1 + drivers/dma/edma.c | 1 + 2 files changed, 2 insertions(+) diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c index

[PATCH v2 00/14] dma: edma: Fixes for cyclic (audio) operation

2014-04-01 Thread Peter Ujfalusi
Hi, This is basically a resend of the previous series: https://lkml.org/lkml/2014/3/13/119 with removed ASoC patches (most of them are applied already). Changes since v1: - ASoC patches removed - Comments from Andriy Shevchenko addressed - patch added to fix cases when src/dst_maxburst is set to

[PATCH v2 02/14] dma: edma: Correct the handling of src/dst_maxburst == 0

2014-04-01 Thread Peter Ujfalusi
When clients asks for maxburst = 0 it is basically the same case as if they were asking for maxburst = 1 since in both case ASYNC need to be used and the eDMA is expected to write/read one word per DMA request. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- drivers/dma/edma.c | 4

[PATCH v2 01/14] platform_data: edma: Be precise with the paRAM struct

2014-04-01 Thread Peter Ujfalusi
The edmacc_param struct should follow the layout of the paRAM area in the HW. Be explicit on the size of the fields (u32) and also mark the struct as packed to avoid any padding on non 32bit architectures. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- include/linux/platform_data/edma.h

[PATCH v4 0/5] drivers: bus: omap_l3: Conversion to devm_*

2014-04-01 Thread Peter Ujfalusi
Hi, Changes since v3: - Tony's ack added Changes since v2: Comments from Alexander Shiyan addressed: - Do not check the return of platform_get_resource() - no need to do that - Use devm_ioremap_resource() instead devm_request_and_ioremap() Changes since v1: - Fixed Santosh's email address in

[PATCH v4 2/5] drivers: bus: omap_l3: Convert to use devm_ioremap_resource()

2014-04-01 Thread Peter Ujfalusi
We can then remove the iounmap() calls from probe and remove. Since the driver requests the resources via index we can do the mem resource request within a for loop. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com Reviewed-by: Santosh Shilimkar santosh.shilim...@ti.com Acked-by: Tony Lindgren

[PATCH v4 5/5] drivers: bus: omap_l3: Change pr_crit() to dev_err() when IRQ request fails

2014-04-01 Thread Peter Ujfalusi
Use dev_err() which will going to print the driver's name as well and the KERN_ERR level is sufficient in this case (we also print via dev_err when there is an error with the mem resources) Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com Reviewed-by: Santosh Shilimkar santosh.shilim...@ti.com

[PATCH v4 3/5] drivers: bus: omap_l3: Convert to use devm_request_irq()

2014-04-01 Thread Peter Ujfalusi
With this we can remove the free_irq() calls from probe and remove. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com Reviewed-by: Santosh Shilimkar santosh.shilim...@ti.com Acked-by: Tony Lindgren t...@atomide.com --- drivers/bus/omap_l3_noc.c | 23 +-- 1 file changed, 5

[PATCH v4 4/5] drivers: bus: omap_l3: Remove the platform driver's remove function

2014-04-01 Thread Peter Ujfalusi
It is NOP after the devm_* conversion. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com Reviewed-by: Santosh Shilimkar santosh.shilim...@ti.com Acked-by: Tony Lindgren t...@atomide.com --- drivers/bus/omap_l3_noc.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH v4 1/5] drivers: bus: omap_l3: Convert to use devm_kzalloc

2014-04-01 Thread Peter Ujfalusi
We can remove the kfree() calls from probe and remove. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com Reviewed-by: Santosh Shilimkar santosh.shilim...@ti.com Acked-by: Tony Lindgren t...@atomide.com --- drivers/bus/omap_l3_noc.c | 11 +++ 1 file changed, 3 insertions(+), 8

AM3517: Address Hole seen by USB_OTG at address dc0f9c0c

2014-04-01 Thread Nishanth Menon
] [0.392169] WARNING: CPU: 0 PID: 1 at drivers/bus/omap_l3_smx.c:162 omap3_l3_app_irq+0x100/0x128() [0.392186] Modules linked in: [0.392221] CPU: 0 PID: 1 Comm: swapper/0 Not tainted 3.14.0-next-20140401-6-g6e78bd6 #1 [0.392279] [c0014cd8] (unwind_backtrace) from [c0011968

[RESEND 3/3] mfd: twl6040: Clear the interrupt ID register before requesting IRQ

2014-04-01 Thread Peter Ujfalusi
If for some reason the boot loader enabled the audpwron GPIO we will have pending IRQs to be handled. This seams to break twl6040 for some reason leading to non working i2c communication (i2c timeouts). Clearing the INTID register after we requested the audpwron GPIO (and set it to low) will

[RESEND 2/3] mfd: twl6040: Move register patching earlier in probe

2014-04-01 Thread Peter Ujfalusi
Make sure that we patch the ACCCTL register as the first thing when the driver loads, thus configuring I2C fast mode and i2c access for dual access registers. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- drivers/mfd/twl6040.c | 7 +++ 1 file changed, 3 insertions(+), 4

[RESEND 1/3] mfd: twl6040: Select i2c fast mode as default with regmap patch

2014-04-01 Thread Peter Ujfalusi
All boards using twl6040 configures the i2c bus to 400KHz. While twl6040's defaults to normal mode (100KHz). So far twl6040 has no problem with i2c communication in this configuration it is safer to select fast i2c mode. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com ---

[RESEND 0/3] mfd: twl6040: Updates for i2s speed and fix for chip deadlock

2014-04-01 Thread Peter Ujfalusi
Hi, While looking into a report by Florian Vaussard [1] I have noticed couple of most likely unrelated issues: - all boards using twl6040 configures the i2c bus to 400KHz while twl6040 is set to 100KHz as default. - if I set the audpwron GPIO high [2] in the bootloader the i2c communication

am3517-craneboard: regression?

2014-04-01 Thread Nishanth Menon
next-20140401-omap2plus_defconfig craneboard: Boot FAIL: http://slexy.org/raw/s2QxOJkvil multi_v7_defconfig seems to boot fine as well (http://slexy.org/raw/s20Tg7ldyt). next-20140331-omap2plus_defconfig craneboard: Boot PASS: http://slexy.org/raw/s21s8CTKxd diffstat between next-20140331

Re: am3517-craneboard: regression?

2014-04-01 Thread Nishanth Menon
On Tue, Apr 1, 2014 at 8:57 AM, Nishanth Menon n...@ti.com wrote: next-20140401-omap2plus_defconfig craneboard: Boot FAIL: http://slexy.org/raw/s2QxOJkvil Looks like a false alarm.. Apologies.. http://slexy.org/raw/s21jnBobtP - i did rerun multiple times and i see it working consistently

Re: [PATCH 1/1] ARM: dts: omap5: Add clocks to USB3 PHY node

2014-04-01 Thread Felipe Balbi
On Tue, Apr 01, 2014 at 01:37:27PM +0300, Roger Quadros wrote: The USB3 PHY driver (ti-pipe3) was updated so that the relevant clock phandles are expected in the DT node. Provide the necessary clocks. Reported-by: Kishon Vijay Abraham I kis...@ti.com Signed-off-by: Roger Quadros

[PATCH] ARM: AM335X: EVM: fix pinmux documentation in devicetree

2014-04-01 Thread Wolfram Sang
Wrong documentation in pinmux description can be especially confusing. Keep it proper. Signed-off-by: Wolfram Sang w...@the-dreams.de --- arch/arm/boot/dts/am335x-evm.dts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/am335x-evm.dts

Re: OMAP baseline test results for v3.14-rc4 (SDP2430 fail to boot - mmc fs)

2014-04-01 Thread Nishanth Menon
On 02/24/2014 12:35 PM, Paul Walmsley wrote: Boot to userspace: FAIL ( 1/12): 2430sdp http://www.pwsan.com/omap/testlogs/test_v3.14-rc4/20140224102552/boot/2430sdp/ Figured out why 2430sdp does not boot with MMC filesystem. DEVCONTROL0,1 needs to be modelled in DT-only boot as well.

Re: [PATCH 00/55]: ARM: OMAP2+: PRCM move to drivers

2014-04-01 Thread Tony Lindgren
* Tero Kristo t-kri...@ti.com [140401 01:38]: On 04/01/2014 01:09 AM, Tony Lindgren wrote: We don't want to make access to these registers available without proper frameworks as that will lead into misuse by various drivers. And cleaning up that mess later in is a huge pain. Currently,

[PATCH] irq: crossbar: improve allocate_free_irq() complexity

2014-04-01 Thread Felipe Balbi
current algorithm in allocate_free_irq() is O(n), by just keeping track of last allocated IRQ with a simple unsigned integer, we can find a free IRQ in O(1). Signed-off-by: Felipe Balbi ba...@ti.com --- compile-tested only as J6 DTS is currently missing crossbar altogether :-( There's a

Re: [PATCH] irq: crossbar: improve allocate_free_irq() complexity

2014-04-01 Thread Felipe Balbi
On Tue, Apr 01, 2014 at 05:44:19PM -0500, Felipe Balbi wrote: current algorithm in allocate_free_irq() is O(n), by just keeping track of last allocated IRQ with a simple unsigned integer, we can find a free IRQ in O(1). Signed-off-by: Felipe Balbi ba...@ti.com --- compile-tested only as