Re: [PATCH v12 05/11] edma: config: Enable config options for EDMA

2013-06-21 Thread Sekhar Nori
On 6/21/2013 2:36 AM, Joel A Fernandes wrote: > From: Joel A Fernandes > > Build TI_EDMA by default for ARCH_DAVINCI and ARCH_OMAP2PLUS > > Signed-off-by: Joel A Fernandes You should sign-off with author e-mail address. > --- > arch/arm/Kconfig|1 + >

Re: [PATCH v12 00/11] DMA Engine support for AM33XX

2013-06-21 Thread Sekhar Nori
Joel, On 6/21/2013 2:36 AM, Joel A Fernandes wrote: > From: Joel A Fernandes > > This series is remaining of Matt Porter's EDMA patches for AM33XX EDMA support > with changes for few pending review comments on v11 series. I have posted a branch with fixes for patches accepted by me in this

Re: [PATCH v12 09/11] spi: omap2-mcspi: convert to dma_request_slave_channel_compat()

2013-06-21 Thread Sekhar Nori
Fixing Mark's e-mail. Mark, On 6/21/2013 3:56 PM, Sourav Poddar wrote: > Hi Mark, > On Friday 21 June 2013 02:36 AM, Joel A Fernandes wrote: >> From: Matt Porter >> >> Convert dmaengine channel requests to use >> dma_request_slave_channel_compat(). This supports the DT case of >> platforms

Re: [PATCH v13] ARM: edma: Add DT and runtime PM support to the private EDMA API

2013-06-24 Thread Sekhar Nori
On 6/21/2013 3:23 PM, Sekhar Nori wrote: > From: Matt Porter > > Adds support for parsing the TI EDMA DT data into the required EDMA > private API platform data. Enables runtime PM support to initialize > the EDMA hwmod. Enables build on OMAP. > > Changes by Joel: > *

Re: [PATCH v12 04/11] dmaengine: edma: enable build for AM33XX

2013-06-24 Thread Sekhar Nori
On 6/24/2013 3:47 PM, Tony Lindgren wrote: > * Sekhar Nori [130621 03:21]: >> On 6/21/2013 2:36 AM, Joel A Fernandes wrote: >>> From: Matt Porter >>> >>> Enable TI EDMA option on OMAP and TI_PRIV_EDMA >>> >>> Signed-off-by: Matt Porter >

Re: [PATCH v12 05/11] edma: config: Enable config options for EDMA

2013-06-24 Thread Sekhar Nori
On 6/22/2013 3:23 AM, Joel A Fernandes wrote: > Hi Arnd, > > On Fri, Jun 21, 2013 at 1:44 PM, Arnd Bergmann wrote: >> On Friday 21 June 2013, Joel A Fernandes wrote: >>> I think we are talking about different things, I agree the 'select >>> DMADEVICES' can be dropped but lets please keep the

Re: [PATCH v12 05/11] edma: config: Enable config options for EDMA

2013-06-24 Thread Sekhar Nori
On 6/24/2013 4:53 PM, Sekhar Nori wrote: > On 6/22/2013 3:23 AM, Joel A Fernandes wrote: >> Hi Arnd, >> >> On Fri, Jun 21, 2013 at 1:44 PM, Arnd Bergmann wrote: >>> On Friday 21 June 2013, Joel A Fernandes wrote: >>>> I think we are talking a

Re: [PATCH v12 00/11] DMA Engine support for AM33XX

2013-06-24 Thread Sekhar Nori
Sourav, On 6/24/2013 3:49 PM, Tony Lindgren wrote: > Hi, > > For merging this series, I suggest the following sets: > > * Joel A Fernandes [130620 14:13]: >> spi: omap2-mcspi: add generic DMA request support to the DT binding >> spi: omap2-mcspi: convert to

Re: [PATCH v12 05/11] edma: config: Enable config options for EDMA

2013-06-24 Thread Sekhar Nori
On 6/22/2013 8:23 AM, Joel A Fernandes wrote: config TI_EDMA tristate "TI EDMA support" default m if 'ARCH_DAVINCI || ARCH_OMAP1 || ARCH_OMAP2 select DMA_ENGINE select DMA_VIRTUAL_CHANNELS >>> >>> >>> MMC depends on EDMA specially on AM33xx

Re: [PATCH 3/3] ARM: davinci: da850: adopt to pinctrl-single driver to configure multiple pins

2013-06-24 Thread Sekhar Nori
On 6/24/2013 5:18 PM, Manjunathappa, Prakash wrote: > Hi Sekhar, > > On Tue, May 21, 2013 at 19:38:02, Manjunathappa, Prakash wrote: >> function-mask property is a mask for a pin at each pin configure offset >> in a pincontrol register. >> > > Got 1/3 and 2/3 accepted, I do not know if this gets

Re: [PATCH v12 00/11] DMA Engine support for AM33XX

2013-06-25 Thread Sekhar Nori
On 6/25/2013 12:24 PM, Tony Lindgren wrote: > * Benoit Cousson [130624 07:42]: >> Hi Tony, >> >> On 06/24/2013 12:19 PM, Tony Lindgren wrote: >>> Hi, >>> >>> For merging this series, I suggest the following sets: >>> >>> * Joel A Fernandes [130620 14:13]: Joel A Fernandes (3):

Re: [PATCH v4 1/6] gpio: davinci: Fixed a check for unbanked gpio

2013-11-06 Thread Sekhar Nori
On Wednesday 06 November 2013 03:45 PM, Prabhakar Lad wrote: > Hi Linus, > > On Wed, Nov 6, 2013 at 3:26 PM, Linus Walleij > wrote: >> On Wed, Nov 6, 2013 at 10:33 AM, Prabhakar Lad >> wrote: >>> On Tue, Nov 5, 2013 at 6:09 PM, Linus Walleij >>> wrote: On Sat, Nov 2, 2013 at 4:39 PM,

Re: [PATCH v4 1/6] gpio: davinci: Fixed a check for unbanked gpio

2013-11-06 Thread Sekhar Nori
Hi Grygorii, On Wednesday 06 November 2013 05:03 PM, Grygorii Strashko wrote: > Hi Sekhar, Linus, > > On 11/06/2013 12:49 PM, Sekhar Nori wrote: >> On Wednesday 06 November 2013 03:45 PM, Prabhakar Lad wrote: >>> Hi Linus, >>> >>> On Wed, Nov 6,

Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used

2013-11-24 Thread Sekhar Nori
ovember 2013 07:44 PM, Santosh Shilimkar wrote: >>>>> On Wednesday 13 November 2013 12:02 AM, Sekhar Nori wrote: >>>>>> On Monday 11 November 2013 10:40 PM, Khoronzhuk, Ivan wrote: >>>>>>> If Davinci AEMIF is used we don't need to set timin

Re: [PATCH v6 1/6] gpio: davinci: use readl/writel instead of __raw_*

2013-11-25 Thread Sekhar Nori
Prabhakar, On Monday 25 November 2013 09:42 AM, Prabhakar Lad wrote: > Hi Taras, > > On Fri, Nov 22, 2013 at 3:38 PM, Taras Kondratiuk > wrote: >> On 21 November 2013 20:15, Prabhakar Lad wrote: >>> From: "Lad, Prabhakar" >>> >>> This patch replaces the __raw_readl/writel with >>> readl and

Re: [PATCH v6 4/6] gpio: davinci: add OF support

2013-11-25 Thread Sekhar Nori
On Thursday 21 November 2013 11:45 PM, Prabhakar Lad wrote: > From: KV Sujith > > This patch adds OF parser support for davinci gpio > driver and also appropriate documentation in gpio-davinci.txt > located at Documentation/devicetree/bindings/gpio/. > > Acked-by: Linus Walleij > Acked-by: Rob

Re: [PATCH 1/6] watchdog: davinci: change driver to use WDT core

2013-11-25 Thread Sekhar Nori
On Monday 18 November 2013 10:48 PM, Ivan Khoronzhuk wrote: > @@ -211,29 +129,34 @@ static int davinci_wdt_probe(struct platform_device > *pdev) > > clk_prepare_enable(wdt_clk); > > - if (heartbeat < 1 || heartbeat > MAX_HEARTBEAT) > - heartbeat = DEFAULT_HEARTBEAT; > +

Re: [PATCH v2 0/6] Update Davinci watchdog driver

2013-11-25 Thread Sekhar Nori
On Monday 18 November 2013 10:48 PM, Ivan Khoronzhuk wrote: > These patches are intended to update Davinci watchdog to use WDT core > and reuse driver for keystone arch, because Keystone uses the similar > IP like Davinci. This series causes a regression on all DaVinci platforms because after the

Re: [PATCH 06/12] mtd: nand: davinci: adjust DT properties to MTD generic

2013-11-25 Thread Sekhar Nori
On Monday 11 November 2013 10:31 PM, Khoronzhuk, Ivan wrote: > The properties davinci-ecc-mode, davinci-nand-use-bbt, davinci-nand-buswidth > are MTD generic. Correct names for them are: nand-ecc-mode, nand-on-flash-bbt, > nand-bus-width accordingly. So rename them in dts and documentation. > >

Re: [PATCH 07/12] memory: davinci-aemif: introduce AEMIF driver

2013-11-25 Thread Sekhar Nori
On Monday 11 November 2013 10:36 PM, Khoronzhuk, Ivan wrote: > Add new AEMIF driver for EMIF16 davinci controller. The EMIF16 module > is intended to provide a glue-less interface to a variety of > asynchronous memory devices like ASRA M, NOR and NAND memory. A total > of 256M bytes of any of

Re: [PATCH v6 4/6] gpio: davinci: add OF support

2013-11-26 Thread Sekhar Nori
On Tuesday 26 November 2013 06:03 PM, Grygorii Strashko wrote: > On 11/25/2013 01:00 PM, Sekhar Nori wrote: >> On Thursday 21 November 2013 11:45 PM, Prabhakar Lad wrote: >>> From: KV Sujith >>> >>> This patch adds OF parser support for davinci gpio >>&

Re: [PATCH 07/12] memory: davinci-aemif: introduce AEMIF driver

2013-11-26 Thread Sekhar Nori
On 11/26/2013 8:35 PM, Santosh Shilimkar wrote: > On Tuesday 26 November 2013 02:20 AM, Sekhar Nori wrote: >> On Monday 11 November 2013 10:36 PM, Khoronzhuk, Ivan wrote: >>> Add new AEMIF driver for EMIF16 davinci controller. The EMIF16 module >>> is intended to pr

Re: [PATCH v6 4/6] gpio: davinci: add OF support

2013-11-26 Thread Sekhar Nori
On Wednesday 27 November 2013 01:11 AM, Grygorii Strashko wrote: > On 11/26/2013 07:12 PM, Sekhar Nori wrote: >> On Tuesday 26 November 2013 06:03 PM, Grygorii Strashko wrote: >>> On 11/25/2013 01:00 PM, Sekhar Nori wrote: >>>> On Thursday 21 November 2013 11:45 PM, P

Re: [PATCH 07/12] memory: davinci-aemif: introduce AEMIF driver

2013-11-26 Thread Sekhar Nori
On Tuesday 26 November 2013 11:14 PM, ivan.khoronzhuk wrote: >>> +static int davinci_aemif_probe(struct platform_device *pdev) >>> +{ >>> + int ret = -ENODEV, i; >>> + struct resource *res; >>> + struct device *dev = >dev; >>> + struct device_node *np = dev->of_node; >>>

Re: [PATCH v3 1/6] watchdog: davinci: change driver to use WDT core

2013-11-26 Thread Sekhar Nori
On Monday 25 November 2013 07:34 PM, Ivan Khoronzhuk wrote: > To reduce code duplicate and increase code readability use WDT core > code to handle WDT interface. > > Remove io_lock as the WDT core uses mutex to lock each wdt device. > Remove wdt_state as the WDT core tracks state with its own

Re: [PATCH] ARM: davinci: aemif: get rid of davinci-nand driver dependency on aemif

2013-11-27 Thread Sekhar Nori
+ MTD maintainers On Tuesday 26 November 2013 01:30 AM, Ivan Khoronzhuk wrote: > The problem that the set timings code contains the call of Davinci > platform function davinci_aemif_setup_timing() which is not > accessible if kernel is built for another platform like Keystone. > > The Keysone

Re: [PATCH] ARM: davinci: aemif: get rid of davinci-nand driver dependency on aemif

2013-11-27 Thread Sekhar Nori
On Wednesday 27 November 2013 04:31 PM, ivan.khoronzhuk wrote: >>> @@ -192,9 +193,15 @@ static __init void davinci_ntosd2_init(void) >>> davinci_cfg_reg(DM644X_ATAEN_DISABLE); >>> >>> /* only one device will be jumpered and detected */ >>> -if (HAS_NAND) >>> +if

Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt

2013-11-27 Thread Sekhar Nori
On Wednesday 27 November 2013 07:01 PM, Ivan Khoronzhuk wrote: > As we switch to use the watchdog core which permits more than one > active watchdog in the system, rename platform driver to > "davinci-wdt" to be identifiable. > > Signed-off-by: Ivan Khoronzhuk Looks good to me. Since bulk of

Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt

2013-11-27 Thread Sekhar Nori
On Wednesday 27 November 2013 09:27 PM, Guenter Roeck wrote: > On 11/27/2013 06:00 AM, Sekhar Nori wrote: >> On Wednesday 27 November 2013 07:01 PM, Ivan Khoronzhuk wrote: >>> As we switch to use the watchdog core which permits more than one >>> active watchdog in

Re: [PATCH v2] ARM: davinci: aemif: get rid of davinci-nand driver dependency on aemif

2013-11-28 Thread Sekhar Nori
ode (aemif.c) has to be removed once Davinci > will be converted to DT and use ti-aemif.c driver. > > The long device name "davinci_ntosd2_nandflash_device" was renamed > to "ntosd2_nandflash" as requested by Sekhar Nori, because after > adding changes the line is so

Re: [PATCH v6 4/6] gpio: davinci: add OF support

2013-11-29 Thread Sekhar Nori
On Friday 29 November 2013 01:16 PM, Linus Walleij wrote: > On Tue, Nov 26, 2013 at 6:12 PM, Sekhar Nori wrote: >> On Tuesday 26 November 2013 06:03 PM, Grygorii Strashko wrote: > >>> Actually, the same was proposed by Linus, but we've tried avoid such huge >>> re

Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt

2013-11-29 Thread Sekhar Nori
Wim, On Thursday 28 November 2013 06:29 PM, Guenter Roeck wrote: > On 11/27/2013 10:04 PM, Sekhar Nori wrote: >> On Wednesday 27 November 2013 09:27 PM, Guenter Roeck wrote: >>> On 11/27/2013 06:00 AM, Sekhar Nori wrote: >>>> On Wednesday 27 November 2013 0

Re: [PATCH] ARM: davinci: Fix McASP mem resource names

2013-11-29 Thread Sekhar Nori
Peter, On Wednesday 13 November 2013 08:18 PM, Peter Ujfalusi wrote: > The ASoC McASP driver looks for the mem resources by name and: > "mpu" and "dat" regions. > Change/add the needed name for the mem resources so the driver can pick the > correct resource. > > Signed-off-by: Peter Ujfalusi

Re: [PATCH 0/2] DaVinci: GPIO: fixes

2013-11-11 Thread Sekhar Nori
On Monday 11 November 2013 09:13 PM, Grygorii Strashko wrote: > Hi Sekhar, Prabhakar Lad > > On 11/08/2013 08:45 AM, Prabhakar Lad wrote:> From: "Lad, Prabhakar" > >> >> This patch series fixes gpio driver regestration >> and offset check for unbanked gpio. >> >> Lad, Prabhakar (2): >>gpio:

Re: [PATCH 1/2] gpio: davinci: Fix a check for unbanked gpio

2013-11-11 Thread Sekhar Nori
On Friday 08 November 2013 12:15 PM, Prabhakar Lad wrote: > From: "Lad, Prabhakar" > > This patch fixes a check for offset in gpio_to_irq_unbanked() > and also assigns gpio_irq, gpio_unbanked of chips[0] to > appropriate values which is used in gpio_to_irq_unbanked() > function. > >

Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used

2013-11-12 Thread Sekhar Nori
On Monday 11 November 2013 10:40 PM, Khoronzhuk, Ivan wrote: > If Davinci AEMIF is used we don't need to set timings and bus width. > It is done by AEMIF driver (drivers/memory/davinci-aemfi.c). > > Signed-off-by: Ivan Khoronzhuk > --- > drivers/mtd/nand/davinci_nand.c | 22

Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used

2013-11-14 Thread Sekhar Nori
On Wednesday 13 November 2013 07:44 PM, Santosh Shilimkar wrote: > On Wednesday 13 November 2013 12:02 AM, Sekhar Nori wrote: >> On Monday 11 November 2013 10:40 PM, Khoronzhuk, Ivan wrote: >>> If Davinci AEMIF is used we don't need to set timings and bus width. >>>

Re: [PATCH 1/2] gpio: davinci: Fix a check for unbanked gpio

2013-11-20 Thread Sekhar Nori
On Monday 18 November 2013 04:18 PM, Linus Walleij wrote: > On Tue, Nov 12, 2013 at 7:18 AM, Sekhar Nori wrote: >> On Friday 08 November 2013 12:15 PM, Prabhakar Lad wrote: >>> From: "Lad, Prabhakar" >>> >>> This patch fixes a check for offset

Re: [PATCH v4 1/5] watchdog: davinci: change driver to use WDT core

2013-12-04 Thread Sekhar Nori
On 11/27/2013 6:18 PM, Ivan Khoronzhuk wrote: > To reduce code duplicate and increase code readability use WDT core > code to handle WDT interface. > > Remove io_lock as the WDT core uses mutex to lock each wdt device. > Remove wdt_state as the WDT core tracks state with its own variable. > >

Re: [PATCH v4 0/5] Update Davinci watchdog driver

2013-12-04 Thread Sekhar Nori
On 11/29/2013 6:48 PM, Grygorii Strashko wrote: > Hi Sekhar, > > On 11/27/2013 02:48 PM, Ivan Khoronzhuk wrote: >> These patches are intended to update Davinci watchdog to use WDT core >> and reuse driver for keystone arch, because Keystone uses the similar >> IP like Davinci. >> >> See

Re: [PATCH v7] gpio: davinci: use {readl|writel}_relaxed() instead of __raw_*

2013-12-15 Thread Sekhar Nori
On Friday 13 December 2013 09:34 AM, Prabhakar Lad wrote: > Hi Linus, > > On Fri, Dec 13, 2013 at 2:01 AM, Linus Walleij > wrote: >> On Wed, Dec 11, 2013 at 6:52 PM, Prabhakar Lad >> wrote: >> >>> From: "Lad, Prabhakar" >>> >>> This patch replaces the __raw_readl/writel with >>>

Re: [PATCH 1/2] gpio: davinci: get rid of DAVINCI_N_GPIO

2013-12-15 Thread Sekhar Nori
On Friday 29 November 2013 02:03 PM, Linus Walleij wrote: > On Thu, Nov 21, 2013 at 4:34 PM, Grygorii Strashko > wrote: > >> Since Davinci GPIO driver is moved to support gpiolib it has to use >> ARCH_NR_GPIOS (can be configured using CONFIG_ARCH_NR_GPIO Kconfig >> option) configuration instead

Re: [PATCH 2/2] gpio: introduce GPIO_DAVINCI kconfig option

2013-12-15 Thread Sekhar Nori
On Friday 29 November 2013 02:05 PM, Linus Walleij wrote: > On Thu, Nov 21, 2013 at 4:34 PM, Grygorii Strashko > wrote: > >> The compatible to Davinci GPIO HW block is used by other TI SoCs, like >> Keystone, where GPIO support is declared as optional. >> >> Hence, introduce GPIO_DAVINCI Kconfig

Re: [PATCH v6 2/6] This patch converts the davinci gpio driver to use irqdomain support.

2013-12-15 Thread Sekhar Nori
On Thursday 21 November 2013 11:45 PM, Prabhakar Lad wrote: > From: "Lad, Prabhakar" > > Signed-off-by: Lad, Prabhakar > [grygorii.stras...@ti.com: > - switch to use one irq-domain per all GPIO banks > - keep irq_create_mapping() call in gpio_to_irq_banked() as it >simply transformed to

Re: [PATCH v6 3/6] gpio: davinci: remove unused variable intc_irq_num

2013-12-15 Thread Sekhar Nori
On Thursday 21 November 2013 11:45 PM, Prabhakar Lad wrote: > From: "Lad, Prabhakar" > > As the davinci-gpio driver is migrated to use irqdomain > there is no need to pass the irq base for the gpio driver. > This patch removes this variable from davinci_gpio_platform_data > and also the

Re: [PATCH v6 5/6] ARM: davinci: da850: add GPIO DT node

2013-12-15 Thread Sekhar Nori
On Thursday 21 November 2013 11:45 PM, Prabhakar Lad wrote: > From: KV Sujith > > Add DT node for Davinci GPIO driver. > > Signed-off-by: KV Sujith > Signed-off-by: Philip Avinash > Signed-off-by: Lad, Prabhakar Added to v3.14/dt branch of my k.org tree. Thanks, Sekhar -- To unsubscribe

Re: [PATCH v6 6/6] ARM: davinci: da850 evm: add GPIO pinumux entries DT node

2013-12-15 Thread Sekhar Nori
umux entries DT node Add GPIO DT node and pinmux entries for DA850 EVM. GPIO is configurable differently on different boards. So add GPIO pinmuxing in dts file. Signed-off-by: KV Sujith Signed-off-by: Philip Avinash Signed-off-by: Lad, Prabhakar Signed-off-by: Sekhar Nori --- arch/arm/boot/

Re: [RFC v1 3/9] gpio: davinci: use chained_irq_enter/chained_irq_exit API

2013-12-15 Thread Sekhar Nori
On Wednesday 27 November 2013 01:10 AM, Grygorii Strashko wrote: > It's unsafe to call IRQ chip callbacks (.irq_mask/irq_unmask/irq_ack) > from chained IRQ handler directly. Because, Davinci GPIO block is used > by different SoCs, which, in turn, have different Main IRQ controllers > (Davinci -

Re: [PATCH 0/2] gpio: davinci: reuse for keystone arch

2013-12-15 Thread Sekhar Nori
On Sunday 15 December 2013 12:41 AM, Santosh Shilimkar wrote: > Linus, Sekhar, > > On Thursday 12 December 2013 01:12 PM, Grygorii Strashko wrote: >> This series is intended to update Davinci GPIO driver and reuse >> it for Keystone SoCs, because Keystone uses the similar GPIO IP like Davinci. >>

Re: [PATCH 0/2] gpio: davinci: reuse for keystone arch

2013-12-15 Thread Sekhar Nori
On Sunday 15 December 2013 07:20 PM, Sekhar Nori wrote: > On Sunday 15 December 2013 12:41 AM, Santosh Shilimkar wrote: >> Linus, Sekhar, >> >> On Thursday 12 December 2013 01:12 PM, Grygorii Strashko wrote: >>> This series is intended to update Davinci GPIO driver

Re: [RESEND 0/2] davinci: Add device tree data for tps6507x.

2013-03-15 Thread Sekhar Nori
Hi Mark, On 3/7/2013 11:56 AM, Vishwanathrao Badarkhe, Manish wrote: > Add device tree data for regulator via tps6507x mfd device > in da850-evm. > Applies on top of v3.9-rc1 of linus tree. I would like to take this series via the davinci tree to manage dependencies with rest of the DT patches.

Re: [PATCH] rtc: omap: update to devm_* API

2013-03-15 Thread Sekhar Nori
On 3/13/2013 12:21 PM, Vishwanathrao Badarkhe, Manish wrote: > From: Vishwanathrao Badarkhe, Manish > > Update the code to use devm_* API so that driver > core will manage resources. > > Signed-off-by: Vishwanathrao Badarkhe, Manish Tested on DA850 EVM. Tested-by: Se

Re: [PATCH v6 1/2] ARM: davinci: dm365: add support for v4l2 video display

2013-03-18 Thread Sekhar Nori
On 3/15/2013 8:41 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > Create platform devices for various video modules like venc,osd, > vpbe and v4l2 driver for dm365. > > Signed-off-by: Lad, Prabhakar > --- > arch/arm/mach-davinci/board-dm365-evm.c |4 +- >

Re: [PATCH v5 1/2] ARM: davinci: dm355: add support for v4l2 video display

2013-03-18 Thread Sekhar Nori
On 3/16/2013 3:47 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > Create platform devices for various video modules like venc,osd, > vpbe and v4l2 driver for dm355. > > Signed-off-by: Lad, Prabhakar Comments on the DM365 series apply to this one too. In addition: > +static int

Re: [PATCH v6 1/2] ARM: davinci: dm365: add support for v4l2 video display

2013-03-18 Thread Sekhar Nori
On 3/18/2013 2:46 PM, Prabhakar Lad wrote: > On Mon, Mar 18, 2013 at 1:48 PM, Sekhar Nori wrote: >> On 3/15/2013 8:41 PM, Prabhakar lad wrote: >>> +int __init dm365_init_video(struct vpfe_config *vpfe_cfg, >>> + s

Re: [PATCH v3 02/11] clk: davinci - add PSC clock driver

2013-03-19 Thread Sekhar Nori
On 10/29/2012 12:54 AM, Linus Walleij wrote: > On Thu, Oct 25, 2012 at 6:11 PM, Murali Karicheri wrote: > >> This is the driver for the Power Sleep Controller (PSC) hardware >> found on DM SoCs as well Keystone SoCs (c6x). This driver borrowed >> code from arch/arm/mach-davinci/psc.c and

Re: [PATCH v2 1/3] ARM: davinci: clk framework support for enable/disable functionality

2013-03-20 Thread Sekhar Nori
On 3/20/2013 12:11 PM, Philip Avinash wrote: > DAVINCI clock framework currently not supporting clock enable/disable > functionality on clock nodes. In DAVINCI platform EHRPWM module requires Wrong. clock enable/disable is supported in the DaVinci clock implementation, but just for PSC clocks. >

Re: [PATCH v2 2/3] arm: davinci: clock node support for ECAP & EHRPWM

2013-03-20 Thread Sekhar Nori
On 3/20/2013 12:11 PM, Philip Avinash wrote: > Add clock node support for ECAP and EHRPWM modules. > Also adds TBCLK for EHRWPM TBCLK to comply with pwm-tiehrpwm > driver. > > Signed-off-by: Philip Avinash > --- > Changes Since v1: > - TBCLK make it as actual clock with enable/disable

Re: [PATCH v2 3/3] ARM: davinci: da850: add EHRPWM & ECAP DT node

2013-03-20 Thread Sekhar Nori
On 3/20/2013 12:11 PM, Philip Avinash wrote: > Add da850 EHRPWM & ECAP DT node. > Also adds OF_DEV_AUXDATA for EHRPWM & ECAP driver to use EHRPWM & ECAP > clock. > > Signed-off-by: Philip Avinash > --- > Changes since v1: > - Reusing ti,am33xx as compatible field as both IP's are >

Re: [PATCH v2 4/6] ARM: davinci: da850: add DT node for eth0.

2013-02-04 Thread Sekhar Nori
On 2/4/2013 10:37 AM, Prabhakar Lad wrote: > Sekhar , > > On Sun, Feb 3, 2013 at 5:33 PM, Sekhar Nori wrote: >> On 1/28/2013 7:17 PM, Prabhakar Lad wrote: >>> From: Lad, Prabhakar >>> >>> Add eth0 device tree node information and pinmux for mii t

Re: [PATCH] i2c: davinci: update to devm_* API

2013-02-05 Thread Sekhar Nori
On 2/5/2013 1:54 PM, Vishwanathrao Badarkhe, Manish wrote: > Update the code to use devm_* API so that driver > core will manage resources. > > Signed-off-by: Vishwanathrao Badarkhe, Manish You can use devm_clk_get() as well. We should also move to runtime PM but that can be a different

Re: [PATCH] ARM: davinci: da850: add DT node for I2C0

2013-02-07 Thread Sekhar Nori
On 2/6/2013 3:06 PM, Vishwanathrao Badarkhe, Manish wrote: > Add I2C0 device tree and pin muxing information to da850-evm. > Also, add OF_DEV_AUXDATA for I2C0 controller driver in da850 > board dt file to use I2C0 clock. > Verified i2c0 node gets created in sys class interface as >

Re: [PATCH] watchdog: davinci_wdt: update to devm_* API

2013-02-07 Thread Sekhar Nori
On 2/7/2013 9:02 AM, Kumar, Anil wrote: > Update the code to use devm_* API so that driver > core will manage resources. > > Signed-off-by: Kumar, Anil > --- > This patch applies on top of v3.8-rc6. > > Tested on da850 EVM. > > :100644 100644 e8e8724... 6ad76a3... M

Re: [PATCH V2 1/2] ARM: davinci: da850: add wdt DT node

2013-02-07 Thread Sekhar Nori
On 2/6/2013 9:30 AM, Kumar, Anil wrote: > Add da850 wdt DT node. > Add OF_DEV_AUXDATA for wdt driver to use wdt clock. > > Signed-off-by: Kumar, Anil > --- a/arch/arm/mach-davinci/da8xx-dt.c > +++ b/arch/arm/mach-davinci/da8xx-dt.c > @@ -37,11 +37,17 @@ static void __init da8xx_init_irq(void) >

Re: [PATCH V2 2/2] ARM: davinci: restart: fix wdt to machine restart with DT

2013-02-07 Thread Sekhar Nori
On 2/6/2013 9:30 AM, Kumar, Anil wrote: > In non DT case da8xx_register_watchdog() is called to register platform device > "da8xx_wdt_device" by board file. But in DT case it is not called and wdt > device get registered via wdt DT node. > > Currently code is passing platform device

Re: [PATCH] watchdog: davinci_wdt: update to devm_* API

2013-02-07 Thread Sekhar Nori
On 2/8/2013 8:05 AM, Kumar, Anil wrote: > On Thu, Feb 07, 2013 at 23:20:48, Nori, Sekhar wrote: >> On 2/7/2013 9:02 AM, Kumar, Anil wrote: >>> Update the code to use devm_* API so that driver >>> core will manage resources. >>> Signed-off-by: Kumar, Anil >>> diff --git

Re: [PATCH RFC] davinci: poll for sleep completion in resume routine.

2013-01-31 Thread Sekhar Nori
On 1/31/2013 2:56 PM, Vishwanathrao Badarkhe, Manish wrote: > As per OMAP-L138 TRM, Software must poll for > SLEEPCOMPLETE bit until it is set to 1 before clearing > SLEEPENABLE bit in DEEPSLEEP register in resume routine. > Modifications are as per datasheet: >

Re: [PATCH V2 1/6] pinctrl: pinctrl-single: use arch_initcall and module_exit

2013-01-31 Thread Sekhar Nori
On 1/29/2013 4:29 PM, Linus Walleij wrote: > On Tue, Jan 29, 2013 at 8:38 AM, Vishwanathrao Badarkhe, Manish > wrote: > >> Currently, I2C driver gets probed before pinctrl driver. >> To achieve I2C pin muxing via pinctrl driver before I2C >> probe get called, register pinctrl driver in

Re: [PATCH v2 1/6] ARM: davinci: da850: fix clock lookup for mdio device

2013-02-03 Thread Sekhar Nori
> Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > Cc: davinci-linux-open-sou...@linux.davincidsp.com > Cc: net...@vger.kernel.org > Cc: devicetree-disc...@lists.ozlabs.org > Cc: Sekhar Nori > Cc: Heiko Schocher > --- > arch/arm/mach-davinci/da850.c |

Re: [PATCH v2 2/6] ARM: davinci: da850: add DT node for mdio device

2013-02-03 Thread Sekhar Nori
nel@vger.kernel.org > Cc: davinci-linux-open-sou...@linux.davincidsp.com > Cc: net...@vger.kernel.org > Cc: devicetree-disc...@lists.ozlabs.org > Cc: Sekhar Nori > Cc: Heiko Schocher > --- > arch/arm/boot/dts/da850-evm.dts |3 +++ > arch/arm/boot/dts/da850.dtsi|7

Re: [PATCH v2 4/6] ARM: davinci: da850: add DT node for eth0.

2013-02-03 Thread Sekhar Nori
adead.org > Cc: linux-kernel@vger.kernel.org > Cc: davinci-linux-open-sou...@linux.davincidsp.com > Cc: net...@vger.kernel.org > Cc: devicetree-disc...@lists.ozlabs.org > Cc: Sekhar Nori > Cc: Heiko Schocher > --- > arch/arm/boot/dts/da850-evm.dts |5 ++

Re: [PATCH v2 6/6] ARM: davinci: da850: configure system configuration chip(CFGCHIP3) for emac

2013-02-03 Thread Sekhar Nori
ff-by: Lad, Prabhakar > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > Cc: davinci-linux-open-sou...@linux.davincidsp.com > Cc: net...@vger.kernel.org > Cc: devicetree-disc...@lists.ozlabs.org > Cc: Sekhar Nori > Cc: Heiko Schocher > -

Re: [PATCH V2 3/6] ARM: davinci: da850: add DT node for I2C0

2013-02-03 Thread Sekhar Nori
On 1/29/2013 1:08 PM, Vishwanathrao Badarkhe, Manish wrote: > Add I2C0 device tree node information to da850-evm. > Also, add I2C0 pin muxing information in da850-evm. > > Signed-off-by: Vishwanathrao Badarkhe, Manish > --- > Changes since V1: > - Updated i2c0 node names in dts and dtsi file.

Re: [PATCH RFC v2] media: tvp514x: add OF support

2013-02-03 Thread Sekhar Nori
On 2/3/2013 3:43 PM, Prabhakar Lad wrote: > Hi Sylwester, > > Thanks for the review. > > On Sat, Feb 2, 2013 at 4:07 AM, Sylwester Nawrocki > wrote: >> Hi Prabhakar, >> >> On 01/29/2013 02:07 PM, Prabhakar Lad wrote: >> [...] >> >>> diff --git

Re: [PATCH v2 6/6] ARM: davinci: da850: configure system configuration chip(CFGCHIP3) for emac

2013-02-03 Thread Sekhar Nori
On 2/3/2013 6:15 PM, Sekhar Nori wrote: > On 1/28/2013 7:17 PM, Prabhakar Lad wrote: >> From: Lad, Prabhakar > There are couple of issues with this implementation. > > 1) da8xx_config_emac() is specific to DA850 EVM, but masquerades as >generic for da8xx. Looks like y

Re: [PATCH 2/2] ARM: davinci: da850: add wdt OF_DEV_AUXDATA entry

2013-02-04 Thread Sekhar Nori
ate: Thu, 24 Jan 2013 14:08:14 +0530 Subject: [PATCH 1/1] ARM: davinci: da850: add wdt DT node Add da850 wdt DT node. Signed-off-by: Kumar, Anil Signed-off-by: Sekhar Nori --- arch/arm/boot/dts/da850-evm.dts |3 +++ arch/arm/boot/dts/da850.dtsi |5 + arch/arm/mach-davinci/da8

Re: [PATCH v3 0/6] Generic PHY Framework

2013-04-19 Thread Sekhar Nori
Hi Kishon, On 3/20/2013 2:41 PM, Kishon Vijay Abraham I wrote: > Added a generic PHY framework that provides a set of APIs for the PHY drivers > to create/destroy a PHY and APIs for the PHY users to obtain a reference to > the PHY with or without using phandle. To obtain a reference to the PHY >

Re: [PATCH 0/6] Davinci fbdev driver and enable it for DMx platform

2013-05-03 Thread Sekhar Nori
On 4/26/2013 11:16 AM, Prabhakar Lad wrote: > Hi Laurent, > > On Thu, Apr 25, 2013 at 2:32 AM, Laurent Pinchart > wrote: >> Hi Prabhakar, >> >> Thank you for the patch. >> >> On Wednesday 24 April 2013 17:30:02 Prabhakar Lad wrote: >>> From: Lad, Prabhakar >>> >>> This patch series adds an

Re: [PATCH v4 5/5] ARM: davinci: da850: override mmc DT node device name

2013-02-28 Thread Sekhar Nori
ed-off-by: Manjunathappa, Prakash >> Cc: linux-...@vger.kernel.org >> Cc: linux-arm-ker...@lists.infradead.org >> Cc: linux-kernel@vger.kernel.org >> Cc: davinci-linux-open-sou...@linux.davincidsp.com >> Cc: devicetree-disc...@lists.ozlabs.org >> Cc: c...@laptop.org &

Re: [PATCH v8 1/2] ARM: davinci: dm355: add support for v4l2 video display

2013-04-12 Thread Sekhar Nori
On 4/8/2013 2:56 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > Create platform devices for various video modules like venc,osd, > vpbe and v4l2 driver for dm355. > > Signed-off-by: Lad, Prabhakar For patches 1/2 and 2/2: Acked-by: Sekhar Nori Thanks, Sekhar -- T

Re: [PATCH] ARM: daVinci: dm644x/dm355/dm365: replace V4L2_STD_525_60/625_50 with V4L2_STD_NTSC/PAL

2013-04-12 Thread Sekhar Nori
On 4/12/2013 4:35 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > This patch replaces V4L2_STD_525_60/625_50 with V4L2_STD_NTSC/PAL > respectively as this are the proper video standards. > > Signed-off-by: Lad, Prabhakar > Reported-by: Hans Verkuil > Cc: Sekhar

Re: [PATCH v5] ARM: davinci: da850: add EHRPWM & ECAP DT node

2013-04-15 Thread Sekhar Nori
On 4/10/2013 5:42 PM, Philip Avinash wrote: > Add da850 EHRPWM & ECAP DT node along with pin-mux details. > Also adds OF_DEV_AUXDATA for EHRPWM & ECAP driver to use EHRPWM & ECAP > clock. > > Signed-off-by: Philip Avinash Looks good to me. Will try and see if this can make it to v3.10. Thanks,

Re: [PATCH v2 3/3] ARM: davinci: da850: add EHRPWM & ECAP DT node

2013-03-21 Thread Sekhar Nori
On 3/21/2013 1:31 PM, Philip, Avinash wrote: > On Wed, Mar 20, 2013 at 18:17:59, Peter Korsgaard wrote: >>>>>>> "Sekhar" == Sekhar Nori writes: >> >> Sekhar> On 3/20/2013 12:11 PM, Philip Avinash wrote: >> >> Add da850 EHRPWM &

Re: [PATCH v3 02/11] clk: davinci - add PSC clock driver

2013-03-22 Thread Sekhar Nori
Mike, On 11/28/2012 6:52 PM, Sekhar Nori wrote: > On 11/27/2012 10:59 PM, Mike Turquette wrote: >> Also have you looked into regmap? Since you are defining your own clock >> type that might be something nice for you. > > No, haven't looked at regmap yet. Will look

Re: [PATCH 1/2] media: davinci: vpss: enable vpss clocks

2013-03-24 Thread Sekhar Nori
On 3/22/2013 1:23 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > By default the VPSS clocks are only enabled in capture driver > for davinci family which creates duplicates. This > patch adds support to enable the VPSS clocks in VPSS driver. > This avoids duplication of code and also adding

Re: [PATCH 2/2] media: davinci: vpbe: venc: move the enabling of vpss clocks to driver

2013-03-24 Thread Sekhar Nori
On 3/22/2013 1:23 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > The vpss clocks were enabled by calling a exported function from a driver > in a machine code. calling driver code from platform code is incorrect way. > > This patch fixes this issue and calls the function from driver code

Re: [PATCH v7 1/2] ARM: davinci: dm365: add support for v4l2 video display

2013-03-25 Thread Sekhar Nori
Prabhakar, On 3/22/2013 1:33 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > Create platform devices for various video modules like venc,osd, > vpbe and v4l2 driver for dm365. > > Signed-off-by: Lad, Prabhakar > --- > arch/arm/mach-davinci/board-dm365-evm.c |4 +- >

Re: [PATCH v3 02/11] clk: davinci - add PSC clock driver

2013-03-25 Thread Sekhar Nori
Hi Mike, On 3/23/2013 2:07 AM, Mike Turquette wrote: > Quoting Sekhar Nori (2013-03-22 04:20:28) >> Mike, >> >> On 11/28/2012 6:52 PM, Sekhar Nori wrote: >>> On 11/27/2012 10:59 PM, Mike Turquette wrote: >> >>>> Also have you looked into regmap?

Re: [PATCH v3 0/3] Platform support for EHRPWM & ECAP devices in DAVINCI.

2013-03-25 Thread Sekhar Nori
On 3/25/2013 1:19 PM, Philip Avinash wrote: > Add platform support for EHRPWM and ECAP by providing clock nodes and > device tree nodes. > This series depends on [1] and [2] and is available for testing at [3]. > Tested for back light support in da850 EVM with EHRPWM PWM device. > > [1] >

Re: [PATCH 2/2] cpufreq: drivers: Remove unnecessary assignments of policy-> members

2013-03-25 Thread Sekhar Nori
On 3/24/2013 8:59 PM, Viresh Kumar wrote: > Some assignments of policy-> min/max/cur/cpuinfo.min_freq/cpuinfo.max_freq > aren't required as part of it is done by cpufreq driver or cpufreq core. > > Remove them. > > At some places we merge multiple lines together too. >

Re: [PATCH 2/2] cpufreq: drivers: Remove unnecessary assignments of policy-> members

2013-03-25 Thread Sekhar Nori
On 3/25/2013 2:15 PM, Viresh Kumar wrote: > On 25 March 2013 14:06, Sekhar Nori wrote: >> There is a line in the code a little above the ones you deleted that >> also sets these same variables. I guess you were relying on that line to >> set policy->cur, but that als

Re: [PATCH 1/2] media: davinci: vpss: enable vpss clocks

2013-03-25 Thread Sekhar Nori
On 3/25/2013 3:44 PM, Prabhakar Lad wrote: > Hi Sekhar, > > Thanks for the review! > > On Mon, Mar 25, 2013 at 11:02 AM, Sekhar Nori wrote: >> On 3/22/2013 1:23 PM, Prabhakar lad wrote: >>> From: Lad, Prabhakar >>> >>> By default th

Re: [PATCH v7 1/2] ARM: davinci: dm365: add support for v4l2 video display

2013-03-25 Thread Sekhar Nori
On 3/25/2013 3:52 PM, Prabhakar Lad wrote: > Hi Sekhar, > > Thanks for the review! > > On Mon, Mar 25, 2013 at 11:42 AM, Sekhar Nori wrote: >> Prabhakar, >> >> On 3/22/2013 1:33 PM, Prabhakar lad wrote: >>> From: Lad, Prabhakar >>> >>

Re: [PATCH 2/2] media: davinci: vpbe: venc: move the enabling of vpss clocks to driver

2013-03-25 Thread Sekhar Nori
On 3/25/2013 3:45 PM, Prabhakar Lad wrote: > Hi Sekhar, > > On Mon, Mar 25, 2013 at 11:09 AM, Sekhar Nori wrote: >> On 3/22/2013 1:23 PM, Prabhakar lad wrote: >>> From: Lad, Prabhakar >>> >>> The vpss clocks were enabled by calling a exported fu

Re: [PATCH 2/2] cpufreq: drivers: Remove unnecessary assignments of policy-> members

2013-03-26 Thread Sekhar Nori
On 3/25/2013 3:54 PM, Viresh Kumar wrote: > On 25 March 2013 15:11, Sekhar Nori wrote: >> So down in the cpufreq driver probe below, we bail out if freq_table is >> not provided. So all this checking for freq_table in the code you pasted >> above is superfluous. If you c

Re: [PATCH 0/5] ARM: davinci: tnetv107x removal

2014-02-27 Thread Sekhar Nori
of order is fine since we just want > to remove the code. However, I'm looking for an Ack from > Cyril Chemparathy and Sekhar Nori first, to be sure we > won't need this code in the future. Kevin Hilman has > already mentioned that he sees no reason to keep this > code. Acked-by: Sekhar

Re: [PATCH v3 0/2] gpio: davinci: reuse for keystone arch

2014-01-07 Thread Sekhar Nori
/kernel/git/nsekhar/linux-davinci.git/log/?h=v3.14/gpio >> >> Changes in v3: >> - fixed code, changed by mistake; fixed sparse warning >> Changes in v2: >> - minor comments applied, no functional changes >> >> v2: https://lkml.org/lkml/2013/12/18/135 >

Re: [PATCH v3 0/2] gpio: davinci: reuse for keystone arch

2014-01-09 Thread Sekhar Nori
On Wednesday 08 January 2014 07:38 PM, Santosh Shilimkar wrote: > On Tuesday 07 January 2014 11:06 PM, Sekhar Nori wrote: >> On Tuesday 07 January 2014 11:22 PM, Santosh Shilimkar wrote: >>> Sekhar, >>> >>> On Tuesday 24 December 2013 06:41 AM, Grygorii Strashko

Re: [PATCH v3] ARM: davinci: aemif: get rid of davinci-nand driver dependency on aemif

2014-01-09 Thread Sekhar Nori
On Thursday 26 December 2013 05:36 PM, ivan.khoronzhuk wrote: > On 12/18/2013 02:13 PM, Sekhar Nori wrote: >> On Wednesday 18 December 2013 05:21 PM, ivan.khoronzhuk wrote: >>> Hi, Sekhar >>> >>> This patch is based on "Reuse davinci-nand driver for Keys

Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt

2014-01-09 Thread Sekhar Nori
On Friday 29 November 2013 01:44 PM, Sekhar Nori wrote: > Wim, > > On Thursday 28 November 2013 06:29 PM, Guenter Roeck wrote: >> On 11/27/2013 10:04 PM, Sekhar Nori wrote: >>> On Wednesday 27 November 2013 09:27 PM, Guenter Roeck wrote: >>>> On 11

<    8   9   10   11   12   13   14   15   16   17   >