Re: [PATCH v3 02/12] ARM: edma: Don't clear EMR of channel in edma_stop

2013-08-08 Thread Sekhar Nori
On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: We certainly don't want error conditions to be cleared any other place but the EDMA error handler, as this will make us 'forget' about missed events we might need to know errors have occurred. This fixes a race condition where the EMR

Re: [PATCH v3 02/12] ARM: edma: Don't clear EMR of channel in edma_stop

2013-08-11 Thread Sekhar Nori
On 8/8/2013 5:19 PM, Sekhar Nori wrote: On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: We certainly don't want error conditions to be cleared any other place but the EDMA error handler, as this will make us 'forget' about missed events we might need to know errors have occurred

Re: [PATCH v3 02/12] ARM: edma: Don't clear EMR of channel in edma_stop

2013-08-12 Thread Sekhar Nori
On Monday 12 August 2013 09:59 AM, Joel Fernandes wrote: On Sun, Aug 11, 2013 at 11:25 PM, Sekhar Nori nsek...@ti.com wrote: On 8/8/2013 5:19 PM, Sekhar Nori wrote: On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: We certainly don't want error conditions to be cleared any other place

Re: [PATCH v3 01/12] dma: edma: Setup parameters to DMA MAX_NR_SG at a time

2013-08-12 Thread Sekhar Nori
On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: Changes are made here for configuring existing parameters to support DMA'ing them out in batches as needed. Also allocate as many as slots as needed by the SG list, but not more than MAX_NR_SG. Then these slots will be reused

Re: [PATCH v3 03/12] dma: edma: remove limits on number of slots

2013-08-12 Thread Sekhar Nori
On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: With this series, this check is no longer required and we shouldn't need to reject drivers DMA'ing more than the MAX number of slots. Signed-off-by: Joel Fernandes jo...@ti.com --- drivers/dma/edma.c |6 -- 1 file changed,

Re: [PATCH v3 07/12] dma: edma: Add function to dump a PaRAM set from PaRAM

2013-08-12 Thread Sekhar Nori
On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: Previously, such a dump function was used but it wasn't reading from the PaRAM, rather just from a edmacc_param structure, we add a helpful function for debugging that directly reads from the PaRAM and gives the current state correctly

Re: [PATCH v3 08/12] dma: edma: Add one more required slot to MAX slots

2013-08-12 Thread Sekhar Nori
On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: We'd now need a separate slot just for the channel and separate ones for the 2 linked sets, so we make adjustments to allocate an extra channel accordingly. Signed-off-by: Joel Fernandes jo...@ti.com No need for a separate patch for

Re: [PATCH v3 09/12] dma: edma: Implement multiple linked sets for continuity

2013-08-12 Thread Sekhar Nori
continues till the end of the SG-lists without any interruption. Suggested-by: Sekhar Nori nsek...@ti.com Signed-off-by: Joel Fernandes jo...@ti.com --- drivers/dma/edma.c | 157 +++- 1 file changed, 118 insertions(+), 39 deletions

Re: [PATCH v2 3/3] ata: add new-style AHCI platform driver for DaVinci DA850 AHCI controller

2014-03-24 Thread Sekhar Nori
platform library code. Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@deeprootsystems.com Cc: Hans de Goede hdego...@redhat.com Signed-off-by: Bartlomiej Zolnierkiewicz b.zolnier...@samsung.com Looks much better now and re-tested it on DA850 EVM. Few issues pointed out below

Re: [PATCH V3 00/20] bus: omap_l3_noc: driver cleanups and support for DRA7/AM4372

2014-04-29 Thread Sekhar Nori
and tested by - minor changes verified Tested the series on AM437x ePOS EVM and works great. Tested-by: Sekhar Nori nsek...@ti.com Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http

Re: [PATCH v2 3/8] davinci: da850: Use cpufreq_for_each_entry macro for iteration

2014-04-21 Thread Sekhar Nori
On Wednesday 16 April 2014 03:56 AM, Stratos Karafotis wrote: The cpufreq core now supports the cpufreq_for_each_entry macro helper for iteration over the cpufreq_frequency_table, so use it. It should have no functional changes. Signed-off-by: Stratos Karafotis strat...@semaphore.gr I

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

2014-04-11 Thread Sekhar Nori
On Tuesday 01 April 2014 06:36 PM, Peter Ujfalusi wrote: 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 Acked-by: Sekhar Nori nsek...@ti.com --- arch/arm/common/edma.c | 3 ++- 1

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

2014-04-11 Thread Sekhar Nori
On Tuesday 01 April 2014 06:36 PM, Peter Ujfalusi wrote: 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

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

2014-04-11 Thread Sekhar Nori
On Friday 11 April 2014 02:20 PM, Peter Ujfalusi wrote: On 04/11/2014 11:17 AM, Sekhar Nori wrote: On Tuesday 01 April 2014 06:36 PM, Peter Ujfalusi wrote: 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

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

2014-04-11 Thread Sekhar Nori
On Friday 11 April 2014 03:12 PM, Vinod Koul wrote: On Fri, Apr 11, 2014 at 12:38:00PM +0300, Peter Ujfalusi wrote: On 04/11/2014 11:56 AM, Sekhar Nori wrote: On Friday 11 April 2014 02:20 PM, Peter Ujfalusi wrote: On 04/11/2014 11:17 AM, Sekhar Nori wrote: On Tuesday 01 April 2014 06:36 PM

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

2014-04-14 Thread Sekhar Nori
On Monday 14 April 2014 05:26 PM, Peter Ujfalusi wrote: Hi Vinod, On 04/11/2014 03:46 PM, Vinod Koul wrote: I think the number shouldn't be viewed in absolute terms. If we decide that (lets say) 0-7, then any controller should map 0 to lowest and 7 to highest. For your case you can do

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

2014-04-14 Thread Sekhar Nori
On Monday 14 April 2014 06:11 PM, Peter Ujfalusi wrote: On 04/14/2014 03:12 PM, Sekhar Nori wrote: On Monday 14 April 2014 05:26 PM, Peter Ujfalusi wrote: Hi Vinod, On 04/11/2014 03:46 PM, Vinod Koul wrote: I think the number shouldn't be viewed in absolute terms. If we decide that (lets

Re: [PATCH 2/4] davinci: da8xx/omap-l1: Remove hard coding of rtc device wakeup

2013-07-01 Thread Sekhar Nori
: Sekhar Nori nsek...@ti.com davinci: da8xx/omap-l1: mark RTC as a wakeup source Signed-off-by: Hebbar Gururaja gururaja.heb...@ti.com Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@linaro.org Cc: Russell King li...@arm.linux.org.uk Subject line should be prefixed with ARM

Re: [PATCH 3/4] rtc: omap: add rtc wakeup support to alarm events

2013-07-02 Thread Sekhar Nori
device dt compatibility node. Signed-off-by: Hebbar Gururaja gururaja.heb...@ti.com Cc: Grant Likely grant.lik...@linaro.org Cc: Rob Herring rob.herr...@calxeda.com Cc: Rob Landley r...@landley.net Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@linaro.org Cc: Alessandro Zummo a.zu

Re: [PATCH 3/4] rtc: omap: add rtc wakeup support to alarm events

2013-07-02 Thread Sekhar Nori
...@landley.net Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@linaro.org Cc: Alessandro Zummo a.zu...@towertech.it Cc: rtc-li...@googlegroups.com Cc: devicetree-disc...@lists.ozlabs.org Cc: linux-...@vger.kernel.org --- [...] -#defineOMAP_RTC_DATA_DA830_IDX 1 +#define

Re: [PATCH 4/4] ARM: dts: AM33XX: update rtc node compatibility

2013-07-02 Thread Sekhar Nori
driver. Signed-off-by: Hebbar Gururaja gururaja.heb...@ti.com Cc: Tony Lindgren t...@atomide.com Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@linaro.org Cc: b-cous...@ti.com --- :100644 100644 77aa1b0... dde180a... March/arm/boot/dts/am33xx.dtsi arch/arm/boot/dts

Re: [PATCH 3/4] rtc: omap: add rtc wakeup support to alarm events

2013-07-02 Thread Sekhar Nori
: Grant Likely grant.lik...@linaro.org Cc: Rob Herring rob.herr...@calxeda.com Cc: Rob Landley r...@landley.net Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@linaro.org Cc: Alessandro Zummo a.zu...@towertech.it Cc: rtc-li...@googlegroups.com Cc: devicetree-disc...@lists.ozlabs.org Cc

Re: [PATCHv2] drivers: spi: Add qspi flash controller

2013-07-02 Thread Sekhar Nori
On 7/2/2013 2:26 PM, Sourav Poddar wrote: The patch add basic support for the quad spi controller. QSPI is a kind of spi module that allows single, dual and quad read access to external spi devices. The module has a memory mapped interface which provide direct interface for accessing data

Re: [PATCH 0/2] ARM: DRA7: hwmod: fixes for 3.16

2014-06-23 Thread Sekhar Nori
On Monday 23 June 2014 10:27 AM, Paul Walmsley wrote: Hi Roger On Wed, 18 Jun 2014, Roger Quadros wrote: These patches add the missing OCP2SCP3 module and fixup the SATA hwmod for DRA7 SoC. Both patches are tested on 3.16-rc1. Could you please post a quick serial capture of a boot log

Re: [PATCH 0/2] ARM: DRA7: hwmod: fixes for 3.16

2014-06-23 Thread Sekhar Nori
On Monday 23 June 2014 10:04 PM, Paul Walmsley wrote: On Mon, 23 Jun 2014, Sekhar Nori wrote: On Monday 23 June 2014 10:27 AM, Paul Walmsley wrote: Hi Roger On Wed, 18 Jun 2014, Roger Quadros wrote: These patches add the missing OCP2SCP3 module and fixup the SATA hwmod for DRA7 SoC. Both

Re: [PATCH 1/2] ARM: DRA7: hwmod: Add OCP2SCP3 module

2014-06-20 Thread Sekhar Nori
On Thursday 19 June 2014 11:18 AM, Paul Walmsley wrote: On Wed, 18 Jun 2014, Roger Quadros wrote: This module is needed for the SATA and PCIe PHYs. Signed-off-by: Roger Quadros rog...@ti.com Tested-by: Roger Quadros rog...@ti.com Is this one a fix? It looks to me like a new IP block

Re: [PATCH 00/15] atmel_mxt_ts - device tree, bootloader, etc

2014-07-07 Thread Sekhar Nori
Hi Nick, On Thursday 03 July 2014 08:31 PM, nick.d...@itdev.co.uk wrote: Hi Dimitry- Here is another set of atmel_mxt_ts patches for upstream. There are some really useful new features, but I hope nothing too controversial. I was unable to get the touchscreen working on my board after

Re: [PATCH 00/15] atmel_mxt_ts - device tree, bootloader, etc

2014-07-08 Thread Sekhar Nori
On Monday 07 July 2014 05:08 PM, Nick Dyer wrote: On 07/07/14 12:21, Sekhar Nori wrote: I was unable to get the touchscreen working on my board after applying just these patches. It does work correctly with your for-next branch so I guess I need to wait for you to post the rest of your

Re: [PATCH v2 2/2] gpio: davinci: reuse for Keystone SoC

2013-12-22 Thread Sekhar Nori
. Documentation: http://www.ti.com/lit/ug/sprugv1/sprugv1.pdf Cc: Linus Walleij linus.wall...@linaro.org Cc: Alexandre Courbot gnu...@gmail.com Cc: Sekhar Nori nsek...@ti.com Cc: devicet...@vger.kernel.org Acked-by: Santosh Shilimkar santosh.shilim...@ti.com Signed-off-by: Grygorii

Re: [PATCH v2 2/2] gpio: davinci: reuse for Keystone SoC

2013-12-22 Thread Sekhar Nori
. Documentation: http://www.ti.com/lit/ug/sprugv1/sprugv1.pdf Cc: Linus Walleij linus.wall...@linaro.org Cc: Alexandre Courbot gnu...@gmail.com Cc: Sekhar Nori nsek...@ti.com Cc: devicet...@vger.kernel.org Acked-by: Santosh Shilimkar santosh.shilim...@ti.com Signed-off-by: Grygorii

Re: [PATCH v2 1/2] gpio: davinci: don't create irq_domain in case of unbanked irqs

2013-12-22 Thread Sekhar Nori
are enabled and correct gpio_to_irq_banked() to handle this properly. Cc: Linus Walleij linus.wall...@linaro.org Cc: Alexandre Courbot gnu...@gmail.com Cc: Sekhar Nori nsek...@ti.com Acked-by: Santosh Shilimkar santosh.shilim...@ti.com Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com

Re: [PATCH 0/3] ARM/dma: edma: Serve cyclic clients via high priority queue

2014-07-28 Thread Sekhar Nori
channel to queue0 (highest priority) and move it back to default queue when the channel is terminated. ping? For 1/3 and 2/3: Acked-by: Sekhar Nori nsek...@ti.com Vinod, can you take the series together with your tree (if you are happy with the series, that is)? I don't have anything else

Re: [PATCH 6/6] tty: serial: Add 8250-core based omap driver

2014-07-16 Thread Sekhar Nori
On Wednesday 16 July 2014 06:02 PM, Tony Lindgren wrote: * Sebastian Andrzej Siewior bige...@linutronix.de [140716 05:14]: On 07/11/2014 08:41 AM, Tony Lindgren wrote: If the RX pin does not wake it up, you need to configure the pinctrl-single entry for it, and configure that pin as a wake-up

Re: [PATCH] ARM: edma: Fix configuration parsing for SoCs with multiple eDMA3 CC

2014-08-26 Thread Sekhar Nori
On Monday 04 August 2014 05:56 PM, Peter Ujfalusi wrote: The edma_setup_from_hw() should know about the CC number when parsing the CCCFG register - when it reads the register to be precise. The base addresses for CCs stored in an array and we need to provide the correct id to edma_read() in

Re: [PATCH 2/2] ARM: davinci: board-da850-evm: Add needed regulators for tlv320aic3106 codec

2014-08-26 Thread Sekhar Nori
On Monday 28 July 2014 04:54 PM, Peter Ujfalusi wrote: IOVDD: tps65070's dcdc2 AVDD and DRVDD: fixed regulator derived from 5V via TPS73701DCQ DVDD: fixed regulator derived from 5V via TPS73701DCQ This patch needed to be able to probe the audio codec. Signed-off-by: Peter Ujfalusi

Re: [PATCH v2 0/6] ARM: DT/davinci: Audio for da850-evm in DT boot

2014-08-26 Thread Sekhar Nori
On Friday 01 August 2014 11:43 AM, Peter Ujfalusi wrote: Hi, Changes since v1: - fixed the address missmatch for tlv320aic3106 codec (@1b - 18) - The edma patches has been taken by Vinod, they should be in linux-next soon. I assume all of these 6 patches are meant to go through ARM SoC. I

Re: [PATCH v2 0/6] ARM: DT/davinci: Audio for da850-evm in DT boot

2014-08-26 Thread Sekhar Nori
On Tuesday 26 August 2014 03:54 PM, Sekhar Nori wrote: On Friday 01 August 2014 11:43 AM, Peter Ujfalusi wrote: Hi, Changes since v1: - fixed the address missmatch for tlv320aic3106 codec (@1b - 18) - The edma patches has been taken by Vinod, they should be in linux-next soon. I assume

Re: [PATCH 2/6] ARM: DTS: da850: Add node for edma0

2014-08-01 Thread Sekhar Nori
On Friday 01 August 2014 10:39 AM, Peter Ujfalusi wrote: On 07/31/2014 05:26 PM, Sergei Shtylyov wrote: On 07/31/2014 02:18 PM, Peter Ujfalusi wrote: Add DT node for edma0. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- arch/arm/boot/dts/da850.dtsi | 6 ++ 1 file changed,

Re: [PATCH] MAINTAINERS: drop list entry for davinci

2014-10-29 Thread Sekhar Nori
On Wednesday 29 October 2014 01:50 PM, Lad, Prabhakar wrote: As davinci-linux-open-sou...@linux.davincidsp.com is now shut and no more maintained by TI any more, drop the entry. Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil

Re: [PATCH v2] MAINTAINERS: drop list entry for davinci

2014-10-29 Thread Sekhar Nori
: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@deeprootsystems.com Cc: a...@kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org --- Acked-by: Sekhar Nori nsek...@ti.com Arnd, Kevin, Olof, Can you please pick this up directly through ARM-SoC tree

Re: [PATCH] davinci: Use standard logging styles

2014-11-18 Thread Sekhar Nori
On Saturday 01 November 2014 06:21 AM, Joe Perches wrote: Convert printks to pr_level and pr_warning to pr_warn. Other miscellanea: o Coalesce formats o Realign arguments o Use %s, __func__ instead of embedded function names o Add pr_fmt to mityomapl138 and mux Signed-off-by: Joe

Re: [PATCH] ARM: davinci: Remove redundant casts

2014-11-18 Thread Sekhar Nori
On Tuesday 21 October 2014 09:40 PM, Sergei Shtylyov wrote: Hello. On 10/21/2014 06:53 PM, Rasmus Villemoes wrote: These casts to char* are unnecessary and slightly confusing, since both operands actually have type const char*. Both operands of what? Typecast only has 1 operand. :-)

Re: [PATCH v4 3/6] mfd: ti_am335x_tscadc: Remove unwanted reg_se_cache save

2014-11-20 Thread Sekhar Nori
. Again, no breakage but no improvement as well. All testing was done using ts_test FWIW, you can add: Tested-by: Sekhar Nori nsek...@ti.com Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo

Re: [PATCH v4 3/6] mfd: ti_am335x_tscadc: Remove unwanted reg_se_cache save

2014-11-21 Thread Sekhar Nori
On Thursday 20 November 2014 08:10 PM, Richard Cochran wrote: On Thu, Nov 20, 2014 at 07:26:00PM +0530, Sekhar Nori wrote: I tested this using lcd7 cape connected to beaglebone black. The latest kernel I could find on this board was a TI BSP based v3.14 kernel. So I had to port these patches

Re: [PATCH v4 3/6] mfd: ti_am335x_tscadc: Remove unwanted reg_se_cache save

2014-11-21 Thread Sekhar Nori
On Friday 21 November 2014 08:41 PM, Johannes Pointner wrote: I tested version 4 of the patch series on a custom design and I saw also the jumps as described by Richard. I played a little with the sample delay and it got better but I couldn't completely remove the jumps. And before the

Re: [PATCH v3 06/10] [media] platform: Make use of media_bus_format enum

2014-11-10 Thread Sekhar Nori
On Saturday 08 November 2014 02:48 PM, Prabhakar Lad wrote: Hi, Thanks for the patch, On Fri, Nov 7, 2014 at 2:07 PM, Boris Brezillon boris.brezil...@free-electrons.com wrote: In order to have subsytem agnostic media bus format definitions we've moved media bus definition to

Re: [PATCH v3 06/10] [media] platform: Make use of media_bus_format enum

2014-11-10 Thread Sekhar Nori
On Tuesday 11 November 2014 10:12 AM, Sekhar Nori wrote: On Saturday 08 November 2014 02:48 PM, Prabhakar Lad wrote: Hi, Thanks for the patch, On Fri, Nov 7, 2014 at 2:07 PM, Boris Brezillon boris.brezil...@free-electrons.com wrote: In order to have subsytem agnostic media bus format

Re: [PATCH] ARM / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM

2014-12-08 Thread Sekhar Nori
PM_RUNTIME if PM_SLEEP is selected) which is only in linux-next at the moment (via the linux-pm tree). Please let me know if it is OK to take this one into linux-pm. --- arch/arm/kernel/perf_event.c |2 +- arch/arm/mach-davinci/pm_domain.c |2 +- Acked-by: Sekhar Nori nsek

Re: [PATCH] ARM: mach-davinci: Remove driver CDCE949

2015-01-18 Thread Sekhar Nori
Hi, On Monday 19 January 2015 05:07 AM, Rickard Strandqvist wrote: Remove driver CDCE949 because it is not used anywhere. Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c

Re: [PATCH] ARM: mach-davinci: Remove driver CDCE949

2015-01-20 Thread Sekhar Nori
On Monday 19 January 2015 10:29 AM, Sekhar Nori wrote: Hi, On Monday 19 January 2015 05:07 AM, Rickard Strandqvist wrote: Remove driver CDCE949 because it is not used anywhere. Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se diff --git a/arch/arm/mach-davinci

Re: [PATCH] arch: arm: mach-davinci: cdce949.c: Remove unused function

2015-01-16 Thread Sekhar Nori
On Thursday 15 January 2015 04:26 AM, Rickard Strandqvist wrote: 2015-01-14 11:22 GMT+01:00 Sekhar Nori nsek...@ti.com: Hi Rickard, On Sunday 07 December 2014 04:32 AM, Rickard Strandqvist wrote: Remove the function cdce_set_rate() that is not used anywhere. This was partially found

Re: [PATCH 05/11] ARM: davinci: multi-soc kernels require AUTO_ZRELADDR

2015-02-16 Thread Sekhar Nori
CONFIG_AUTO_ZRELADDR to be set This patch changes Kconfig to make the two families mutually exclusive when this is unset. Signed-off-by: Arnd Bergmann a...@arndb.de Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@deeprootsystems.com Acked-by: Sekhar Nori nsek...@ti.com Thanks

Re: [PATCH 04/11] ARM: davinci: davinci_cfg_reg cannot be init

2015-02-16 Thread Sekhar Nori
dm365evm_mmc_configure lacks a __init annotation or the annotation of davinci_cfg_reg is wrong. This removes the extraneous __init_or_module annotation. Signed-off-by: Arnd Bergmann a...@arndb.de Cc: Sekhar Nori nsek...@ti.com Cc: Kevin Hilman khil...@deeprootsystems.com Acked-by: Sekhar Nori

Re: [PATCH] arch: arm: mach-davinci: serial.c: Cleaning up function that are not used anywhere

2015-01-14 Thread Sekhar Nori
On Wednesday 03 December 2014 03:38 AM, Rickard Strandqvist wrote: Remove function serial_read_reg() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se

Re: [PATCH] arch: arm: common: edma.c: Remove some unused functions

2015-01-14 Thread Sekhar Nori
On Monday 29 December 2014 01:23 PM, Peter Ujfalusi wrote: On 12/07/2014 12:53 AM, Rickard Strandqvist wrote: Removes some functions that are not used anywhere: edma_shadow0_read() edma_or_array() This was partially found by using a static code analysis program called cppcheck. Acked-by:

Re: [PATCH 0/3] ARM: edma: Correct header file usage

2015-01-14 Thread Sekhar Nori
On Thursday 27 November 2014 04:11 PM, Peter Ujfalusi wrote: Hi, The linux/platform_data/edma.h file was used for API definition as well, which is not correct since the header should only contain platform data related structures, defines, etc. I would like to queue this series through

Re: [PATCH] arch: arm: mach-davinci: cdce949.c: Remove unused function

2015-01-14 Thread Sekhar Nori
Hi Rickard, On Sunday 07 December 2014 04:32 AM, Rickard Strandqvist wrote: Remove the function cdce_set_rate() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist

Re: [RESEND 0/7] ARM: davinci: McASP related cleanups and fixes

2015-03-18 Thread Sekhar Nori
On Thursday 12 March 2015 02:59 PM, Sekhar Nori wrote: Hi Peter, On Thursday 12 March 2015 01:36 PM, Peter Ujfalusi wrote: Hi, V1 was sent back in 10th of February [1], so resending it rebased on current linux-next. Clean up series of McASP device creation for da8xx and dm646x DMSoCs

Re: [RESEND 6/7] ARM: davinci: irqs: Correct McASP1 TX interrupt definition for DM646x

2015-03-18 Thread Sekhar Nori
dm646x_default_priorities[]. AEMIF interrupt on DM6467 is 60 not 30 and the entry for the correct interrupt number is already present in the same table. Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com [nsek...@ti.com: remove bogus entry from dm646x_default_priorities[]] Signed-off-by: Sekhar

Re: [RESEND 0/7] ARM: davinci: McASP related cleanups and fixes

2015-03-12 Thread Sekhar Nori
Hi Peter, On Thursday 12 March 2015 01:36 PM, Peter Ujfalusi wrote: Hi, V1 was sent back in 10th of February [1], so resending it rebased on current linux-next. Clean up series of McASP device creation for da8xx and dm646x DMSoCs. All applied. Will send out the pull request soon after a

Re: [Resend PATCH] gpio/davinci: add interrupt support for GPIOs 16-31

2015-07-02 Thread Sekhar Nori
On Thursday 02 July 2015 03:43 PM, Grygorii Strashko wrote: On 07/01/2015 08:41 PM, Vitaly Andrianov wrote: Interrupts for GPIOs 16 through 31 are enabled by bit 1 in the binten register (offset 8). Previous versions of GPIO only used bit 0, which enables GPIO 0-15 interrupts. Cc: Sekhar

Re: [RFC PATCH 4/5] ARM: dts: DRA7: Add memory map region entries for qspi

2015-07-31 Thread Sekhar Nori
On Tuesday 28 July 2015 02:11 PM, Vignesh R wrote: Add qspi memory mapped region entries for DRA7xx based SoCs. Signed-off-by: Vignesh R vigne...@ti.com --- arch/arm/boot/dts/am4372.dtsi | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/am4372.dtsi

Re: [RESEND][PATCH 3/4] ARM: davinci: Set proper SPI prescale limit value

2015-08-11 Thread Sekhar Nori
On Tuesday 11 August 2015 05:17 AM, Franklin S Cooper Jr. wrote: On 07/24/2015 06:50 AM, Sekhar Nori wrote: On Wednesday 22 July 2015 06:02 PM, Franklin S Cooper Jr wrote: SPI Davinci driver has been updated to allow SOCs to specify their minimum prescale value. Update the various SOCs

Re: [RESEND][PATCH 3/4] ARM: davinci: Set proper SPI prescale limit value

2015-08-11 Thread Sekhar Nori
On Tuesday 11 August 2015 02:33 PM, Mark Brown wrote: On Tue, Aug 11, 2015 at 12:28:21PM +0530, Sekhar Nori wrote: On Tuesday 11 August 2015 05:17 AM, Franklin S Cooper Jr. wrote: ping. Patches 1 and 2 have already been pulled into Mark's spi tree and are currently in in linux-next. Mark

Re: [RESEND][PATCH 2/4] spi: davinci: Choose correct pre-scaler limit based on SOC

2015-07-24 Thread Sekhar Nori
for Keystone devices to support their unique pre-scaler limit value. Signed-off-by: Franklin S Cooper Jr fcoo...@ti.com Reviewed-by: Sekhar Nori nsek...@ti.com Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org

Re: [RESEND][PATCH 3/4] ARM: davinci: Set proper SPI prescale limit value

2015-07-24 Thread Sekhar Nori
...@ti.com Acked-by: Sekhar Nori nsek...@ti.com in case Mark is okay to take it through his tree. It does not clash with anything else I have for mach-davinci. Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org

Re: [PATCH] ARM: edma: Clear IRQ if we get interrupted by a unknown event

2015-07-13 Thread Sekhar Nori
Hi Roger, On Monday 13 July 2015 06:12 PM, Roger Quadros wrote: It looks like we can get an interrupt even when none of the events that we're expecting occur. This can typically happen due to incorrect usage of the dma engine by the clients. If we don't Ack this interrupt then we get a flood

Re: [PATCH] gpio/davinci: add interrupt support for GPIOs 16-31

2015-07-16 Thread Sekhar Nori
On Tuesday 14 July 2015 07:31 PM, Linus Walleij wrote: On Thu, Jun 18, 2015 at 7:10 PM, Vitaly Andrianov vita...@ti.com wrote: Interrupts for GPIOs 16 through 31 are enabled by bit 1 in the binten register (offset 8). Previous versions of GPIO only used bit 0, which enables GPIO 0-15

Re: [PATCH v2] gpio/davinci: add interrupt support for GPIOs 16-31

2015-07-16 Thread Sekhar Nori
-by: Grygorii Strashko grygorii.stras...@ti.com Reviewed-by: Sekhar Nori nsek...@ti.com Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please

Re: [PATCH] gpio/davinci: add interrupt support for GPIOs 16-31

2015-07-16 Thread Sekhar Nori
On Thursday 16 July 2015 11:11 PM, Vitaly Andrianov wrote: On 07/16/2015 05:04 AM, Sekhar Nori wrote: On Tuesday 14 July 2015 07:31 PM, Linus Walleij wrote: On Thu, Jun 18, 2015 at 7:10 PM, Vitaly Andrianov vita...@ti.com wrote: Interrupts for GPIOs 16 through 31 are enabled by bit 1

[PATCH] irqchip: omap-intc: fix spurious irq handling

2015-10-19 Thread Sekhar Nori
conditions (see the section of AM335x TRM for details) and is recoverable, we do not need a warning splat for users to report. It can however result in reduced performance so we add a ratelimited debug print to aid developers. Signed-off-by: Sekhar Nori <nsek...@ti.com> --- drivers/irqchip/irq-

Re: [PATCH] irqchip: omap-intc: fix spurious irq handling

2015-10-20 Thread Sekhar Nori
On Monday 19 October 2015 08:20 PM, Tony Lindgren wrote: > Hi, > > * Sekhar Nori <nsek...@ti.com> [151019 02:51]: >> Under some conditions, irq sorting procedure used by INTC can go wrong >> resulting in a spurious irq getting reported. >> >> This condition

Re: [PATCH v4 00/25] dmaengine/ARM: Merge the edma drivers into one

2015-10-07 Thread Sekhar Nori
. It worked well. For the arch/arm/mach-davinci parts, please add Acked-by: Sekhar Nori <nsek...@ti.com> Vinod, if you prefer me replying with acks on individual patches, I can do that too. Regards, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-kernel&

Re: [RFC 3/3] serial: 8250_omap: try to avoid IER_RDI with DMA

2015-08-26 Thread Sekhar Nori
On Friday 14 August 2015 09:31 PM, Sebastian Andrzej Siewior wrote: It has been observed on DRA7-evm that the UART triggers the interrupt and reading IIR says IIR_NO_INT. It seems that we receive data via RX-DMA but the interrupt is triggered anyway. I have hardly observed it on AM335x and not

Re: [PATCH 1/3] serial: 8250: move rx_running out of the bitfield

2015-08-26 Thread Sekhar Nori
to avoid more such errors. Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Signed-off-by: John Ogness john.ogn...@linutronix.de Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de Tested with wilink BT module on TI's DRA7 EVM. Tested-by: Sekhar Nori nsek...@ti.com Thanks

Re: [PATCH 1/3] serial: 8250: move rx_running out of the bitfield

2015-08-26 Thread Sekhar Nori
On Wednesday 26 August 2015 06:13 PM, Peter Hurley wrote: On 08/26/2015 05:37 AM, Sekhar Nori wrote: On Friday 14 August 2015 09:31 PM, Sebastian Andrzej Siewior wrote: From: John Ogness john.ogn...@linutronix.de That bitfield is modified by read + or + write operation. If someone sets any

Re: [PATCH 2/3] serial: 8250_omap: check how many bytes were injected

2015-08-26 Thread Sekhar Nori
On Wednesday 26 August 2015 06:14 PM, Peter Hurley wrote: On 08/14/2015 12:01 PM, Sebastian Andrzej Siewior wrote: The function tty_insert_flip_string() returns an int and as such it might fail. So the result is that I kindly asked to insert 48 bytes and the function only insterted 32. I have

Re: [PATCH 2/3] serial: 8250_omap: check how many bytes were injected

2015-08-26 Thread Sekhar Nori
-by: Sekhar Nori nsek...@ti.com Thanks, Sekhar -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH v2 00/23] dmaengine/ARM: Merge the edma drivers into one

2015-09-12 Thread Sekhar Nori
On Friday 11 September 2015 05:56 PM, Peter Ujfalusi wrote: > Hi, > > Changes since v1: > - Convert edma platform device registration to use > platform_device_register_full > - Moved the PM callback also to the dmaengine driver - missed in v1 > - Commit message added to: > ARM/dmaengine: edma:

Re: [PATCH] ARM: davinci: clock: Correct return values for API functions

2015-09-22 Thread Sekhar Nori
On Monday 14 September 2015 02:59 PM, Peter Ujfalusi wrote: > Fix the values returned by the publicly used functions. > These function should return 0 when they are called with clk == NULL in > similar manner as the clock API does to avoid different behavior in drivers > used by daVinci and other

[PATCH v2] irqchip: omap-intc: add support for spurious irq handling

2015-12-08 Thread Sekhar Nori
in omap-intc driver to prevent this. Signed-off-by: Sekhar Nori <nsek...@ti.com> --- v2: increment error irq counter, use pr_err_once, add a comment on tips to debug spurious irq condition. This patch results in a checkpatch warning about extern definition of irq_err_count, but looks like

Re: [PATCH] irqchip: omap-intc: fix spurious irq handling

2015-12-03 Thread Sekhar Nori
Hi Tony, On Tuesday 20 October 2015 08:22 PM, Tony Lindgren wrote: > * John Ogness <john.ogn...@linutronix.de> [151020 00:33]: >> On 2015-10-20, Sekhar Nori <nsek...@ti.com> wrote: >>>> Do you know what really is causing the spurious interrupts in yo

Re: [PATCH 0/6] ARM: DTS: da850: eDMA new bindings and MMC/SPI DMA support

2015-12-08 Thread Sekhar Nori
On Wednesday 09 December 2015 12:49 PM, Peter Ujfalusi wrote: > Sekhar, > > On 12/04/2015 03:53 PM, Peter Ujfalusi wrote: >> Hi, >> >> Switch to use the new eDMA bindings and enable DMA for MMC0, SPI1. >> Add node for MMC1. > > Can you hold on this one for a bit since the eDMA bindings might

Re: [PATCH] irqchip: omap-intc: fix spurious irq handling

2015-12-03 Thread Sekhar Nori
On Thursday 03 December 2015 08:32 PM, Tony Lindgren wrote: > * Sekhar Nori <nsek...@ti.com> [151203 03:29]: >> On Tuesday 20 October 2015 08:22 PM, Tony Lindgren wrote: >>> >>> OK thanks for testing. My guess from the above list would be EDMA >>> or CPSW

Re: [PATCH v3] ARM: davinci: fix a problematic usage of WARN()

2015-12-15 Thread Sekhar Nori
On Thursday 26 November 2015 06:05 AM, Geliang Tang wrote: > WARN() takes a condition and a format string. The condition was > omitted. So I added it. > > Signed-off-by: Geliang Tang Applied. Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 2/6] ARM: DTS: da850: Use the new DT bindings for the eDMA3

2015-12-15 Thread Sekhar Nori
Hi Peter, On Friday 04 December 2015 07:23 PM, Peter Ujfalusi wrote: > Switch to use the ti,edma3-tpcc and ti,edma3-tptc binding for the eDMA3. > With the new bindings boards can customize and tweak the DMA channel > priority to match their needs. With the new binding the memcpy is safe > to be

Re: [PATCH 2/6] ARM: DTS: da850: Use the new DT bindings for the eDMA3

2015-12-15 Thread Sekhar Nori
On Tuesday 15 December 2015 05:14 PM, Peter Ujfalusi wrote: > On 12/15/2015 11:38 AM, Sekhar Nori wrote: >> Hi Peter, >> >> On Friday 04 December 2015 07:23 PM, Peter Ujfalusi wrote: >>> Switch to use the ti,edma3-tpcc and ti,edma3-tptc binding for the eDMA3. >&g

Re: [PATCH] [media] staging/davinci_vfpe: allow modular build

2015-12-10 Thread Sekhar Nori
On Thursday 10 December 2015 08:10 PM, Mauro Carvalho Chehab wrote: > Em Thu, 10 Dec 2015 15:29:38 +0100 > Arnd Bergmann escreveu: > >> It has never been possible to actually build this driver as >> a loadable module, only built-in because the Makefile attempts >> to build each

Re: [PATCH v2] irqchip: omap-intc: add support for spurious irq handling

2015-12-10 Thread Sekhar Nori
Hi Felipe, On Tuesday 08 December 2015 07:15 PM, Felipe Balbi wrote: > Sekhar Nori <nsek...@ti.com> writes: >> +/* >> + * A spurious IRQ can result if interrupt that triggered the >> + * sorting is no longer active during the sorting (10 INTC >> +

[PATCH v2 2/2] spi: davinci: use dev_err() for error reporting

2015-12-10 Thread Sekhar Nori
Use dev_err() for reporting errors rather than dev_dbg(). Signed-off-by: Sekhar Nori <nsek...@ti.com> --- drivers/spi/spi-davinci.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/spi/spi-davinci.c b/drivers/spi/spi-davinci.c index 57d696

[PATCH v2 1/2] spi: davinci: fix spurious i/o error

2015-12-10 Thread Sekhar Nori
wait_for_completion_timeout(). Signed-off-by: Sekhar Nori <nsek...@ti.com> --- drivers/spi/spi-davinci.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi-davinci.c b/drivers/spi/spi-davinci.c index 7d3af3eacf57..57d6960a6252 100644 --- a/drivers/s

[PATCH v2 0/2] spi: davinci: fix spurious i/o error

2015-12-10 Thread Sekhar Nori
This patch series fixes a spurious I/O error with DaVinci SPI. v2: - Update to handle DMA case - add a patch to use dev_err() for error reporting Sekhar Nori (2): spi: davinci: fix spurious i/o error spi: davinci: use dev_err() for error reporting drivers/spi/spi-davinci.c | 17

Re: [PATCH v2] irqchip: omap-intc: add support for spurious irq handling

2015-12-15 Thread Sekhar Nori
On Thursday 10 December 2015 08:46 PM, Sekhar Nori wrote: > Hi Felipe, > > On Tuesday 08 December 2015 07:15 PM, Felipe Balbi wrote: >> Sekhar Nori <nsek...@ti.com> writes: > >>> + /* >>> +* A spurious IRQ can result if interrupt that triggered

[PATCH v3] irqchip: omap-intc: add support for spurious irq handling

2015-12-15 Thread Sekhar Nori
in omap-intc driver to prevent this. Measurements using kernel function profiler on AM335x EVM running at 720MHz show that after this patch omap_intc_handle_irq() takes about 37.4us against 34us before this patch. Signed-off-by: Sekhar Nori <nsek...@ti.com> --- v3: add unlikely(), add pro

Re: [PATCH 2/6] ARM: DTS: da850: Use the new DT bindings for the eDMA3

2015-12-15 Thread Sekhar Nori
On Tuesday 15 December 2015 07:35 PM, Peter Ujfalusi wrote: > On 12/15/2015 03:48 PM, Sekhar Nori wrote: >>>> In future, if/when we gain QDMA support, the QDMA channels could be used >>>> for memcopy. >>> >>> Well, in short there is no way to get t

Re: [PATCH v3] irqchip: omap-intc: add support for spurious irq handling

2016-01-04 Thread Sekhar Nori
Hi Thomas, On Tuesday 15 December 2015 08:58 PM, Tony Lindgren wrote: > * Sekhar Nori <nsek...@ti.com> [151215 06:26]: >> Under some conditions, irq sorting procedure used >> by INTC can go wrong resulting in a spurious irq >> getting reported. >> >> If th

[PATCH] PCI: keystone: fix crash with deferred probe

2015-11-20 Thread Sekhar Nori
, it should be added back with a comment explaining why. Signed-off-by: Sekhar Nori <nsek...@ti.com> --- drivers/pci/host/pci-keystone-dw.c | 2 +- drivers/pci/host/pci-keystone.c| 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/pci/host/pci-keystone-d

Re: [PATCH] PCI: keystone: fix crash with deferred probe

2015-11-23 Thread Sekhar Nori
On Monday 23 November 2015 04:57 PM, Jisheng Zhang wrote: > On Mon, 23 Nov 2015 19:22:09 +0800 > Jisheng Zhang <jszh...@marvell.com> wrote: > >> Dear Sekhar, >> >> On Fri, 20 Nov 2015 16:47:02 +0530 >> Sekhar Nori wrote: >> >>> Drop __

[PATCH] spi: davinci: fix spurious i/o error

2015-11-24 Thread Sekhar Nori
wait_for_completion_timeout(). Signed-off-by: Sekhar Nori <nsek...@ti.com> --- drivers/spi/spi-davinci.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi-davinci.c b/drivers/spi/spi-davinci.c index 7d3af3eacf57..38026aa1afd7 100644 --- a/drivers/s

Re: [PATCH] spi: davinci: fix spurious i/o error

2015-11-24 Thread Sekhar Nori
On Tuesday 24 November 2015 04:04 PM, Sekhar Nori wrote: > davinci_spi_bufs() uses wait_for_completion_interruptible() > without bothering to handle -ERESTARTSYS. Due to this, > sometime, it returns prematurely when a signal is received. > Since the return value is never checke

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