[PATCH v5 6/7] ARM: davinci: da850: add GPIO DT node

2013-11-08 Thread Prabhakar Lad
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 --- arch/arm/boot/dts/da850.dtsi | 14 ++ 1 file changed, 14 insertions(+) diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da

[PATCH v5 3/7] gpio: davinci: use irqdomain

2013-11-08 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch converts the davinci gpio driver to use irqdomain support. Signed-off-by: Lad, Prabhakar --- drivers/gpio/gpio-davinci.c| 74 +++- include/linux/platform_data/gpio-davinci.h |2 +- 2 files changed, 41 insertions(+)

[PATCH v5 4/7] gpio: davinci: remove unused variable intc_irq_num

2013-11-08 Thread Prabhakar Lad
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 references from the machine file. Signed-off-by: Lad, Prabhakar --- arch/arm/m

[PATCH v5 5/7] gpio: davinci: add OF support

2013-11-08 Thread Prabhakar Lad
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/. Signed-off-by: KV Sujith Signed-off-by: Philip Avinash [prabhakar.cse...@gmail.com: simplified the OF code, remov

[PATCH v5 2/7] gpio: davinci: use readl/writel instead of __raw_*

2013-11-08 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch replaces the __raw_readl/writel with readl and writel, Although the code runs on ARMv5 based SOCs, changing this will help copying the code for other uses. Signed-off-by: Lad, Prabhakar --- drivers/gpio/gpio-davinci.c | 36 ++-

[PATCH v5 7/7] ARM: davinci: da850 evm: add GPIO pinumux entries DT node

2013-11-08 Thread Prabhakar Lad
From: KV Sujith 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 --- arch/arm/boot/dts/da850-evm.dts | 20

[PATCH v5 1/7] gpio: davinci: remove unnecessary printk

2013-11-08 Thread Prabhakar Lad
From: "Lad, Prabhakar" the devm_*() helper prints error messages in case of errors no need to do the same in the driver. Signed-off-by: Lad, Prabhakar --- drivers/gpio/gpio-davinci.c |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpio/gpio-davinci.c b/driv

[PATCH v5 0/7] gpio: daVinci: cleanup and feature enhancement

2013-11-08 Thread Prabhakar Lad
From: KV Sujith This patch series does the following 1> Ports the driver to use irqdomain. 2> Adds dt binding support for gpio-davinci. 3> Adds DA850 dt support goio. Changes for v5: 1: Fixed review comments pointed by Grygorii. 2: Dropped the fixup patch This patch series is based on following

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

2013-11-07 Thread Prabhakar Lad
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. Reported-by: Grygorii Strashko Signed-off-by: Lad, Prabhakar --- drivers/gpio/gpio-

[PATCH 2/2] ARM: davinci: Fix number of resources passed to davinci_gpio_register() call

2013-11-07 Thread Prabhakar Lad
From: "Lad, Prabhakar" The davinci_gpio_register() function expects the number of resources as the second parameter, but size of resources was passed to it due to which it was causing unexpected behaviour. This patch fixes the same by passing the ARRAY_SIZE of resources. Reported-by: Grygorii St

[PATCH 0/2] DaVinci: GPIO: fixes

2013-11-07 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch series fixes gpio driver regestration and offset check for unbanked gpio. Lad, Prabhakar (2): gpio: davinci: Fix a check for unbanked gpio ARM: davinci: Fix number of resources passed to davinci_gpio_register() call arch/arm/mach-davinci/dm355.c |

Re: [PATCH v2] ths7303: Declare as static a private function

2013-11-07 Thread Prabhakar Lad
rning: no previous prototype for > ‘ths7303_setval’ [-Wmissing-prototypes] >int ths7303_setval(struct v4l2_subdev *sd, enum ths7303_filter_mode mode) > ^ > > Signed-off-by: Ricardo Ribalda Delgado > Acked-by: Laurent Pinchart Acked-by: Lad, Prabhakar Regards, --Prabhakar

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

2013-11-06 Thread Prabhakar Lad
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, Lad, Prabhakar >>> wrote: >>>

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

2013-11-06 Thread Prabhakar Lad
Hi Grygorii, On Tue, Nov 5, 2013 at 10:29 PM, Grygorii Strashko wrote: > On 11/05/2013 10:53 AM, Prabhakar Lad wrote:> Hi Grygorii, > >> >> Thanks for the review. >> >> On Mon, Nov 4, 2013 at 11:58 PM, Grygorii Strashko >> wrote: >>> Hi Prabha

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

2013-11-06 Thread Prabhakar Lad
; >> Signed-off-by: Lad, Prabhakar > > Is this a regression that should go in right now? > Yes it needs too. Regards, --Prabhakar Lad -- 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 4/6] gpio: davinci: add OF support

2013-11-05 Thread Prabhakar Lad
Hi Grygorii, Thanks for the review. On Mon, Nov 4, 2013 at 11:58 PM, Grygorii Strashko wrote: > Hi Prabhakar Lad, > > > On 11/02/2013 05:39 PM, Lad, Prabhakar wrote: >> >> From: KV Sujith >> >> This patch adds OF parser support for davinci gpio >> dr

Re: [PATCH v4 3/6] gpio: davinci: use irqdomain

2013-11-05 Thread Prabhakar Lad
Hi grygorii, Thanks for the review. On Mon, Nov 4, 2013 at 11:57 PM, Grygorii Strashko wrote: > Hi Prabhakar Lad, > > On 11/02/2013 05:39 PM, Lad, Prabhakar wrote: >> From: "Lad, Prabhakar" >> >> This patch converts the davinci gpio driver to use irqdomai

Re: [PATCH v3 1/3] gpio: davinci: add OF support

2013-11-02 Thread Prabhakar Lad
Hi Grygorii, On Mon, Oct 14, 2013 at 5:55 PM, Grygorii Strashko wrote: > Hi Prabhakar Lad, > > On 10/11/2013 07:18 PM, Prabhakar Lad wrote: >> Hi Linus, >> >> On 10/11/13, Linus Walleij wrote: >>> On Fri, Oct 11, 2013 at 4:59 PM, Prabhakar Lad >>>

Re: [PATCH] davinci_emac.c: Fix IFF_ALLMULTI setup

2013-10-21 Thread Prabhakar Lad
> > Tested with kernel 2.6.37. > It would have been better if you would have tested this on the latest kernel. Anyway David has pulled this patch in his tree. Regards, --Prabhakar Lad -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a messa

Re: [PATCH v3 1/3] gpio: davinci: add OF support

2013-10-11 Thread Prabhakar Lad
Hi Linus, On 10/11/13, Linus Walleij wrote: > On Fri, Oct 11, 2013 at 4:59 PM, Prabhakar Lad > wrote: >> On 10/11/13, Linus Walleij wrote: >>> On Fri, Oct 4, 2013 at 6:03 PM, Prabhakar Lad >>> wrote: > >>>> +- ti,davinci-gpio-irq-base: Base fr

Re: [PATCH v3 1/3] gpio: davinci: add OF support

2013-10-11 Thread Prabhakar Lad
Hi Linus , On 10/11/13, Linus Walleij wrote: > On Fri, Oct 4, 2013 at 6:03 PM, Prabhakar Lad > wrote: > >> This patch adds OF parser support for davinci gpio >> driver and also appropriate documentation in gpio-davinci.txt >> located at Documentation/devicetree/bindi

[PATCH] v4l: tuner-core: fix typo

2013-10-10 Thread Prabhakar Lad
From: "Lad, Prabhakar" Signed-off-by: Lad, Prabhakar --- drivers/media/v4l2-core/tuner-core.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/tuner-core.c b/drivers/media/v4l2-core/tuner-core.c index ddc9379..4b8a9a3 100644 --- a/drivers/media/v4l

Re: [PATCH] [media] davinci: vpfe: remove deprecated IRQF_DISABLED

2013-10-06 Thread Prabhakar Lad
On Sun, Oct 6, 2013 at 11:53 AM, Michael Opdenacker wrote: > This patch proposes to remove the use of the IRQF_DISABLED flag > > It's a NOOP since 2.6.35 and it will be removed one day. > > Signed-off-by: Michael Opdenacker Acked-by: Lad, Prabhakar Regrads, --Prabhakar L

[PATCH] of/of_reserved_mem.c: fix typo

2013-10-02 Thread Prabhakar Lad
From: "Lad, Prabhakar" Signed-off-by: Lad, Prabhakar --- drivers/of/of_reserved_mem.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/of/of_reserved_mem.c b/drivers/of/of_reserved_mem.c index 0fe40c7..c7a1ea3 100644 --- a/drivers/of/of_reserved_mem.c +++ b/drivers

Re: [PATCH v5] media: st-rc: Add ST remote control driver

2013-09-27 Thread Prabhakar Lad
gt; +{ > + struct st_rc_device *rc_dev = dev_get_drvdata(dev); > + > + if (device_may_wakeup(dev)) { > + if (!enable_irq_wake(rc_dev->irq)) > + rc_dev->irq_wake = 1; > + else > + re

Re: [PATCH] [media] davinci: remove deprecated IRQF_DISABLED

2013-09-12 Thread Prabhakar Lad
ad, Prabhakar Regrads, --Prabhakar Lad -- 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 v3 2/2] media: i2c: adv7343: add OF support

2013-09-04 Thread Prabhakar Lad
Hi Mark, On Mon, Sep 2, 2013 at 9:47 PM, Mark Rutland wrote: > On Wed, Aug 28, 2013 at 03:43:04AM +0100, Prabhakar Lad wrote: >> Hi Mark, >> >> On Tue, Aug 27, 2013 at 8:54 PM, Mark Rutland wrote: >> > [fixing up devicetree list address] >> > >>

Re: [PATCH v3 2/2] media: i2c: adv7343: add OF support

2013-08-27 Thread Prabhakar Lad
Hi Mark, On Tue, Aug 27, 2013 at 8:54 PM, Mark Rutland wrote: > [fixing up devicetree list address] > Thanks! > On Mon, Aug 26, 2013 at 03:41:45AM +0100, Prabhakar Lad wrote: >> Hi Sylwester, >> >> On Fri, Aug 23, 2013 at 11:33 PM, Sylwester Nawrocki >> wrote

Re: [PATCH v3 2/2] media: i2c: adv7343: add OF support

2013-08-25 Thread Prabhakar Lad
ad,adv7343-power-mode-dac: array configuring the power on/off DAC's 1..6, >> + 0 = OFF and 1 = ON, Default value when this >> + property is not specified is <0 0 0 0 0 0>. > > Name of the property is incorrect here.

Re: [PATCH 7/8] net: davinci_mdio: remove unnecessary dev_set_drvdata()

2013-08-21 Thread Prabhakar Lad
ar Regards, --Prabhakar Lad -- 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 V3 1/2] tps6507x-ts: Add DT support

2013-08-19 Thread Prabhakar Lad
goto error_ret; > + > + err = of_property_read_u16(node, "ti,product", > + &input_dev->id.product); > + if (err < 0) > + goto error_ret; > + > + err = of_property_read_u16(node, "ti,version", > + &input_dev->id.version); > + if (err < 0) > + goto error_ret; All the three above properties as per the documentation above are optional, but the code here makes them required property. May be what you can do is if property is not defined make it as zero as per documentation. Regards, --Prabhakar Lad -- 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 v3 7/7] net: davinci_mdio: use platform_{get,set}_drvdata()

2013-08-16 Thread Prabhakar Lad
o Chen Acked-by: Lad, Prabhakar Regards, --Prabhakar Lad -- 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 v4 5/5] ARM: davinci: da850: add OF_DEV_AUXDATA entry for eth0.

2013-08-15 Thread Prabhakar Lad
250.2", NULL), >> OF_DEV_AUXDATA("ti,davinci_mdio", 0x01e24000, "davinci_mdio.0", >> NULL), >> + OF_DEV_AUXDATA("ti,davinci-dm6467-emac", 0x01e2, >> "davinci_emac.1", >> + NULL), >> + > >

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

2013-08-15 Thread Prabhakar Lad
}; >> + eth0: emac@1e2 { > > >According to the ePAPR spec[1] section 2.2.2, the node should be named > "ethernet". > Thanks for pointing it, I'll fix and repost it. Regards, --Prabhakar Lad > [1] http://www.power.org/resources/downloads/Po

Re: [PATCH v5] media: i2c: tvp7002: add OF support

2013-08-12 Thread Prabhakar Lad
uld keep it as is, let me know if you still want me to change. >> + >> +- field-even-active: Active-high Field ID output polarity control of the >> bus. >> + Under normal operation, the field ID output is set to logic 1 for an odd >> field >> + (field 1)

Re: [PATCH v2 0/4] GPIO DT support for da850

2013-08-11 Thread Prabhakar Lad
gt; This patch series has dependency on [PATCH v2 0/7] Convert GPIO Davinci to > platform driver > > https://lkml.org/lkml/2013/6/14/120 > What is the status of this patch series is any one taking care of it or else I can take care of review comments and repost them. Regards, --Prabhakar Lad

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

2013-08-08 Thread Prabhakar Lad
On Thu, Aug 8, 2013 at 9:35 PM, Sekhar Nori wrote: > On Thursday 08 August 2013 04:02 PM, Prabhakar Lad wrote: >> Hi Sekhar, >> >> Sorry for the delayed response I was on leave and now back again up >> and running. >> >> On Wed, Jul 31, 2013 at 11:17 AM, Se

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

2013-08-08 Thread Prabhakar Lad
Hi Sekhar, Sorry for the delayed response I was on leave and now back again up and running. On Wed, Jul 31, 2013 at 11:17 AM, Sekhar Nori wrote: > On Sunday 23 June 2013 08:30 PM, Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> This patch makes a common fun

Re: [PATCH RFC FINAL v5] media: OF: add "sync-on-green-active" property

2013-07-26 Thread Prabhakar Lad
Hi Hans, Thanks for the review. On Fri, Jul 26, 2013 at 3:27 PM, Hans Verkuil wrote: > Hi Prabhakar, > > On 07/17/2013 05:47 PM, Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> This patch adds 'sync-on-green-active' property as part &g

Re: [PATCH v4] media: i2c: tvp7002: add OF support

2013-07-26 Thread Prabhakar Lad
ive state of Sync-on-green signal property of the >> + endpoint. >> + 0 = Normal Operation (Default) > > I would extend this a little bit: > > 0 = Normal Operation (Active Low, Default) > Ok. >> + 1 = Inverted operation >> + >> +- field-even-active: Active-

Re: [PATCH 0/5] Davinci VPBE use devres and some cleanup

2013-07-23 Thread Prabhakar Lad
list. [1] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source Regards, --Prabhakar Lad -- 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

Re: [PATCH 0/5] Davinci VPBE use devres and some cleanup

2013-07-23 Thread Prabhakar Lad
Hi Hans, On Sat, Jul 13, 2013 at 2:20 PM, Prabhakar Lad wrote: > From: "Lad, Prabhakar" > > This patch series replaces existing resource handling in the > driver with managed device resource. > > Lad, Prabhakar (5): > media: davinci: vpbe_venc: convert t

Re: [PATCH v2 5/5] media: davinci: vpbe: Replace printk with dev_*

2013-07-18 Thread Prabhakar Lad
Hi Joe, On Thu, Jul 18, 2013 at 9:10 PM, Joe Perches wrote: > On Thu, 2013-07-18 at 20:59 +0530, Prabhakar Lad wrote: >> Use the dev_* message logging API instead of raw printk. > [] >> diff --git a/drivers/media/platform/davinci/vpbe.c >> b/drivers/media

[PATCH v2 5/5] media: davinci: vpbe: Replace printk with dev_*

2013-07-18 Thread Prabhakar Lad
From: "Lad, Prabhakar" Use the dev_* message logging API instead of raw printk. Signed-off-by: Lad, Prabhakar --- Posting independent patch of the series, http://www.spinics.net/lists/linux-media/msg65701.html Changes for v2: 1: Fixed logging levels. drivers/media/platform/davinci/vpbe

Re: [PATCH v3] media: i2c: tvp7002: add OF support

2013-07-17 Thread Prabhakar Lad
Hi Sylwester, Thanks for the quick review. On Wed, Jul 17, 2013 at 10:09 PM, Sylwester Nawrocki wrote: > On 07/17/2013 06:20 PM, Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> add OF support for the tvp7002 driver. >> >> Signed-off-by: Lad,

Re: [PATCH 0/5] Davinci VPBE use devres and some cleanup

2013-07-17 Thread Prabhakar Lad
Hi Laurent, On Wed, Jul 17, 2013 at 5:51 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > On Saturday 13 July 2013 14:20:26 Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> This patch series replaces existing resource handling in the >> driver with

Re: [PATCH RFC FINAL v5] media: OF: add "sync-on-green-active" property

2013-07-17 Thread Prabhakar Lad
Hi Sylwester, On Wed, Jul 17, 2013 at 9:50 PM, Sylwester Nawrocki wrote: > On 07/17/2013 05:47 PM, Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> This patch adds 'sync-on-green-active' property as part >> of endpoint property. >> >&

[PATCH v3] media: i2c: tvp7002: add OF support

2013-07-17 Thread Prabhakar Lad
From: "Lad, Prabhakar" add OF support for the tvp7002 driver. Signed-off-by: Lad, Prabhakar --- This patch depends on https://patchwork.kernel.org/patch/2828800/ Changes for v3: 1: Fixed review comments pointed by Sylwester. .../devicetree/bindings/media/i2c/tvp7002.txt | 43

[PATCH RFC FINAL v5] media: OF: add "sync-on-green-active" property

2013-07-17 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch adds 'sync-on-green-active' property as part of endpoint property. Signed-off-by: Lad, Prabhakar --- Changes for v5: 1: Changed description for sync-on-green-active property in documentation file as suggested by Sylwester. Changes for v4: 1: Fix

Re: [PATCH RFC v4] media: OF: add "sync-on-green-active" property

2013-07-17 Thread Prabhakar Lad
t;> + or inverted operation. > > > Would you mind adding this entry after pclk-sample property description ? OK. > And how about describing it a bit more precisely and similarly to > VSYNC/HSYNC, > e.g. > > - sync-on-green-active: active state of Sync-on-green (SoG) signal, >

[PATCH RFC v4] media: OF: add "sync-on-green-active" property

2013-07-16 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch adds 'sync-on-green-active' property as part of endpoint property. Signed-off-by: Lad, Prabhakar --- Changes for v4: 1: Fixed review comments pointed by Sylwester. Changes for v3: 1: Fixed review comments pointed by Laurent and Sylwester. RFC v2 h

Re: [PATCH v2 2/2] media: i2c: tvp7002: add OF support

2013-07-15 Thread Prabhakar Lad
-on-green-active' property and parse it in v4l2_of_parse_parallel_bus() and add it as part of flags. and define the following flags in mediabus.h V4L2_MBUS_VIDEO_SOG_ACTIVE_{HIGH,LOW}. Hope you are OK with it. -- Regards, Prabhakar Lad -- To unsubscribe from this list: send the line "

Re: [PATCH v2] media: i2c: adv7343: add OF support

2013-07-15 Thread Prabhakar Lad
Hi Sylwester, Thanks for the review. On Mon, Jul 15, 2013 at 2:03 AM, Sylwester Nawrocki wrote: > Hi Prabhakar, > > > On 07/13/2013 01:12 PM, Prabhakar Lad wrote: >> >> From: "Lad, Prabhakar" >> >> add OF support for the adv7343 driver. >>

[PATCH v2] media: i2c: adv7343: add OF support

2013-07-13 Thread Prabhakar Lad
From: "Lad, Prabhakar" add OF support for the adv7343 driver. Signed-off-by: Lad, Prabhakar --- Changes for v2: 1: Fixed naming of properties. .../devicetree/bindings/media/i2c/adv7343.txt | 54 drivers/media/i2c/adv7343.c| 65 +

[PATCH 2/5] media: davinci: vpbe_osd: convert to devm_* api

2013-07-13 Thread Prabhakar Lad
From: "Lad, Prabhakar" Replace existing resource handling in the driver with managed device resource, this ensures more consistent error values and simplifies error paths. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpbe_osd.c | 45 +++-- 1 file c

[PATCH 1/5] media: davinci: vpbe_venc: convert to devm_* api

2013-07-13 Thread Prabhakar Lad
From: "Lad, Prabhakar" Replace existing resource handling in the driver with managed device resource, this ensures more consistent error values and simplifies error paths. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpbe_venc.c | 97 ++-- 1 file c

[PATCH 3/5] media: davinci: vpbe_display: convert to devm* api

2013-07-13 Thread Prabhakar Lad
From: "Lad, Prabhakar" Replace existing resource handling in the driver with managed device resource, this ensures more consistent error values and simplifies error paths. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpbe_display.c | 23 +++ 1 file cha

[PATCH 4/5] media: davinci: vpss: convert to devm* api

2013-07-13 Thread Prabhakar Lad
From: "Lad, Prabhakar" Replace existing resource handling in the driver with managed device resource, this ensures more consistent error values and simplifies error paths. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpss.c | 62 +++-- 1 file c

[PATCH 5/5] media: davinci: vpbe: Replace printk with dev_*

2013-07-13 Thread Prabhakar Lad
From: "Lad, Prabhakar" Use the dev_* message logging API instead of raw printk. Signed-off-by: Lad, Prabhakar --- drivers/media/platform/davinci/vpbe.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/platform/davinci/vpbe.c b/drivers/media/platform/d

[PATCH 0/5] Davinci VPBE use devres and some cleanup

2013-07-13 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch series replaces existing resource handling in the driver with managed device resource. Lad, Prabhakar (5): media: davinci: vpbe_venc: convert to devm_* api media: davinci: vpbe_osd: convert to devm_* api media: davinci: vpbe_display: convert to devm* api

Re: [PATCH v2 2/2] media: i2c: tvp7002: add OF support

2013-07-11 Thread Prabhakar Lad
On Fri, Jul 12, 2013 at 3:34 AM, Sylwester Nawrocki wrote: > On 07/11/2013 07:09 PM, Prabhakar Lad wrote: > [...] > >>>> diff --git a/Documentation/devicetree/bindings/media/i2c/tvp7002.txt >>>> b/Documentation/devicetree/bindings/media/i2c/tvp7002.txt >>>

Re: [PATCH RFC v3] media: OF: add video sync endpoint property

2013-07-11 Thread Prabhakar Lad
Hi Sylwester, On Fri, Jul 12, 2013 at 2:45 AM, Sylwester Nawrocki wrote: > On 07/11/2013 01:41 PM, Prabhakar Lad wrote: > [...] > >>>> diff --git a/drivers/media/v4l2-core/v4l2-of.c >>>> b/drivers/media/v4l2-core/v4l2-of.c >>>> index aa59639..1a54

Re: [PATCH v3 0/6] ARM: davinci: da850: add ethernet driver DT support

2013-07-11 Thread Prabhakar Lad
Hi Sekhar, On Sun, Jun 23, 2013 at 8:30 PM, Prabhakar Lad wrote: > From: "Lad, Prabhakar" > > This patch set enables Ethernet support through device tree model. > This patch set enables mii interface only and is being tested to boot via > rootfs. The rmii phy is present

Re: [PATCH v2 2/2] media: i2c: tvp7002: add OF support

2013-07-11 Thread Prabhakar Lad
Hi Sylwester, Thanks for the review. On Sun, Jun 30, 2013 at 9:27 PM, Sylwester Nawrocki wrote: > Hi, > > > On 06/22/2013 07:44 PM, Prabhakar Lad wrote: >> >> From: "Lad, Prabhakar" >> >> add OF support for the tvp7002 driver. >> >&g

Re: [PATCH RFC v3] media: OF: add video sync endpoint property

2013-07-11 Thread Prabhakar Lad
Hi Sylwester, Oops some how missed this mail, sorry for the late response. On Sun, Jun 30, 2013 at 9:23 PM, Sylwester Nawrocki wrote: > Hi, > > > On 06/22/2013 05:03 PM, Prabhakar Lad wrote: >> >> From: "Lad, Prabhakar" >> >> This patch

Re: [PATCH 2/2] video:da8xx-fb: Convert to devm_* api

2013-07-02 Thread Prabhakar Lad
Hi Darren, On Wed, Jun 26, 2013 at 5:41 PM, Tomi Valkeinen wrote: > On 26/06/13 15:06, Prabhakar Lad wrote: >> Hi Tomi, >> >> On Wed, Jun 26, 2013 at 5:30 PM, Tomi Valkeinen >> wrote: >>> On 26/06/13 14:56, Prabhakar Lad wrote: >>>> Hi Tomi,

Re: [PATCH 1/2] media: davinci: vpif: capture: add V4L2-async support

2013-06-26 Thread Prabhakar Lad
Hi Hans, Thanks for the review. On Thu, Jun 27, 2013 at 11:27 AM, Hans Verkuil wrote: > On Tue June 25 2013 17:17:34 Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> Add support for asynchronous subdevice probing, using the v4l2-async API. >> The legac

Re: [PATCH 2/2] video:da8xx-fb: Convert to devm_* api

2013-06-26 Thread Prabhakar Lad
Hi Tomi, On Wed, Jun 26, 2013 at 5:30 PM, Tomi Valkeinen wrote: > On 26/06/13 14:56, Prabhakar Lad wrote: >> Hi Tomi, >> >> On Wed, Jun 26, 2013 at 5:09 PM, Tomi Valkeinen >> wrote: >>> On 16/05/13 10:10, Lad Prabhakar wrote: >>>> From: Lad

Re: [PATCH 2/2] video:da8xx-fb: Convert to devm_* api

2013-06-26 Thread Prabhakar Lad
27;s something similar in Darren's da8xx-fb series. Can you check if > there are differences? > I don't see similar changes in his patch series. Darren while you are at it you can take this patch along you series. let me know how you plan to. Regards, --Prabhakar Lad -- To unsubsc

Re: [PATCH 2/2] video:da8xx-fb: Convert to devm_* api

2013-06-26 Thread Prabhakar Lad
lifies error paths. > > Signed-off-by: Lad, Prabhakar > --- Can you pick up this patch and drop 1/2. Regards, --Prabhakar Lad -- 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:

[PATCH v2 1/3] net: davinci: emac: Convert to devm_* api

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" Use devm_ioremap_resource instead of devm_request_mem_region()/devm_ioremap() and devm_request_irq() instead of request_irq(). This ensures more consistent error values and simplifies error paths. Signed-off-by: Lad, Prabhakar --- drivers/net/ethernet/ti/davinci_emac.c

[PATCH v2 3/3] net: davinci_mdio: gaurd the DT code with IS_ENABLED(CONFIG_OF)

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" guard the davinci_mdio_of_mtable table and davinci_mdio_probe_dt() with CONFIG_OF. Signed-off-by: Lad, Prabhakar --- drivers/net/ethernet/ti/davinci_mdio.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/ti/davinci_mdio.c

[PATCH v2 2/3] net: davinci_emac: simplify the OF parser code

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch cleans up the OF parser code, removes unnecessary checks on of_property_read_*() and guards davinci_emac_of_match table with CONFIG_OF. Signed-off-by: Lad, Prabhakar --- drivers/net/ethernet/ti/davinci_emac.c | 67 +++- 1 file cha

[PATCH v2 0/3] net: davinci: driver cleanup

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch series cleans up the davinci driver. This patch series applies on 3.10.rc7 and is boot tested on OMAP-L138 EVM with DT and non DT cases. Changes for v2: 1: Dropped patches which removed include files. Lad, Prabhakar (3): net: davinci: emac: Convert to devm_*

[PATCH 2/2] media: davinci: vpif: display: add V4L2-async support

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" Add support for asynchronous subdevice probing, using the v4l2-async API. The legacy synchronous mode is still supported too, which allows to gradually update drivers and platforms. Signed-off-by: Lad, Prabhakar Cc: Guennadi Liakhovetski Cc: Hans Verkuil Cc: Laurent Pin

[PATCH 0/2] media: davinci: vpif: capture/display support for async subdevice probing

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch series adds support for vpif capture and display driver to support asynchronously register subdevices. Need for this support: Currently bridge device drivers register devices for all subdevices synchronously, typically, during their probing. E.g. if an I2C CMOS

[PATCH 1/2] media: davinci: vpif: capture: add V4L2-async support

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" Add support for asynchronous subdevice probing, using the v4l2-async API. The legacy synchronous mode is still supported too, which allows to gradually update drivers and platforms. Signed-off-by: Prabhakar Lad Cc: Guennadi Liakhovetski Cc: Hans Verkuil C

Re: [PATCH RFC v3] media: OF: add video sync endpoint property

2013-06-25 Thread Prabhakar Lad
Hi Hans, On Mon, Jun 24, 2013 at 1:21 PM, Hans Verkuil wrote: > Hi Prabhakar, > > On Sat June 22 2013 17:03:03 Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> [snip] >> +#ifndef _DT_BINDINGS_VIDEO_INTERFACES_H >> +#define _DT_B

Re: [PATCH RFC v3] media: OF: add video sync endpoint property

2013-06-24 Thread Prabhakar Lad
Hi Hans, Thanks for the review. On Mon, Jun 24, 2013 at 1:21 PM, Hans Verkuil wrote: > Hi Prabhakar, > > On Sat June 22 2013 17:03:03 Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> This patch adds video sync properties as part of endpoint >> p

[PATCH v2] media: i2c: tvp514x: add support for asynchronous probing

2013-06-24 Thread Prabhakar Lad
From: "Lad, Prabhakar" Both synchronous and asynchronous tvp514x subdevice probing is supported by this patch. This patch also fixes the error path by calling media_entity_cleanup() on failure in probe when CONFIG_MEDIA_CONTROLLER is enabled. Signed-off-by: Prabhakar Lad Cc

[PATCH 2/2] media: i2c: tvp514x: remove manual setting of subdev name

2013-06-24 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch removes manual setting of subdev name in the probe, ideally subdev names must be unique. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil --- drivers/media/i2c/tvp514x.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/i2c/tvp514x.c b/drivers

[PATCH 1/2] media: i2c: tvp7002: remove manual setting of subdev name

2013-06-24 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch removes manual setting of subdev name in the probe, ideally subdev names must be unique. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil --- drivers/media/i2c/tvp7002.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/i2c/tvp7002.c b/drivers

[PATCH 0/2] tvp514x/tvp7002 remove manual setting of subdev names

2013-06-24 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch series removes manual setting of subdev names, as ideally the subdev names must be unique and this would not be the case if there are two devices. Lad, Prabhakar (2): media: i2c: tvp7002: remove manual setting of subdev name media: i2c: tvp514x: remove manua

Re: [PATCH] media: i2c: tvp514x: add support for asynchronous probing

2013-06-24 Thread Prabhakar Lad
Hi Hans, On Mon, Jun 24, 2013 at 2:09 PM, Hans Verkuil wrote: > On Mon June 24 2013 10:24:02 Prabhakar Lad wrote: >> Hi Hans, >> >> On Mon, Jun 24, 2013 at 12:41 PM, Hans Verkuil wrote: >> > On Sun June 23 2013 17:48:20 Prabhakar Lad wrote: >> >> Hi G

Re: [PATCH] media: i2c: tvp514x: add support for asynchronous probing

2013-06-24 Thread Prabhakar Lad
Hi Hans, On Mon, Jun 24, 2013 at 12:41 PM, Hans Verkuil wrote: > On Sun June 23 2013 17:48:20 Prabhakar Lad wrote: >> Hi Guennadi, >> >> Thanks for the review. >> >> On Sun, Jun 23, 2013 at 8:49 PM, Guennadi Liakhovetski >> wrote: >> > On Sat, 2

Re: [PATCH v2 1/2] media: i2c: tvp7002: add support for asynchronous probing

2013-06-24 Thread Prabhakar Lad
Hi Hans, On Mon, Jun 24, 2013 at 12:44 PM, Hans Verkuil wrote: > On Sat June 22 2013 19:44:14 Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> Both synchronous and asynchronous tvp7002 subdevice probing >> is supported by this patch. > > Can I merg

Re: [PATCH] media: i2c: tvp514x: add support for asynchronous probing

2013-06-23 Thread Prabhakar Lad
Hi Guennadi, Thanks for the review. On Sun, Jun 23, 2013 at 8:49 PM, Guennadi Liakhovetski wrote: > On Sat, 22 Jun 2013, Prabhakar Lad wrote: > >> From: "Lad, Prabhakar" >> >> Both synchronous and asynchronous tvp514x subdevice probing is supported by

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

2013-06-23 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch makes a common function for to configure emac and calls it appropriately in DT and non DT boot mode. The system configuration chip CFGCHIP3, controls the emac module. This patch appropriately configures this register for emac and sets DA850_MII_MDIO_CLKEN_PIN GPI

[PATCH v3 5/6] ARM: davinci: da850: add OF_DEV_AUXDATA entry for eth0.

2013-06-23 Thread Prabhakar Lad
From: "Lad, Prabhakar" Add OF_DEV_AUXDATA for eth0 driver in da850 board dt file to use emac clock. Signed-off-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: devicetr

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

2013-06-23 Thread Prabhakar Lad
From: "Lad, Prabhakar" Add eth0 device tree node information and pinmux for mii to da850 by providing interrupt details and local mac address of eth0. Signed-off-by: Lad, Prabhakar Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org Cc: davinci-linux-open-sou...@linux.davi

[PATCH v3 0/6] ARM: davinci: da850: add ethernet driver DT support

2013-06-23 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch set enables Ethernet support through device tree model. This patch set enables mii interface only and is being tested to boot via rootfs. The rmii phy is present on the i2c gpio expander chip (UI board) for which yet support needs to be added, once the DT support

[PATCH v3 3/6] ARM: davinci: da850: add OF_DEV_AUXDATA entry for mdio.

2013-06-23 Thread Prabhakar Lad
From: "Lad, Prabhakar" Add OF_DEV_AUXDATA for mdio driver in da850 board dt file to use mdio clock. Signed-off-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: devicetre

[PATCH v3 1/6] ARM: davinci: da8xx: fix clock lookup for mdio device

2013-06-23 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch removes the clock alias for mdio device and adds a entry in clock lookup table, this entry can now be used by both DT and NON DT case. Signed-off-by: Lad, Prabhakar Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org Cc: davinci-linux-open

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

2013-06-23 Thread Prabhakar Lad
From: "Lad, Prabhakar" Add mdio device tree node information to da850 by providing register details and bus frequency of mdio. Signed-off-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...@vg

[PATCH v2 2/2] media: i2c: tvp7002: add OF support

2013-06-22 Thread Prabhakar Lad
From: "Lad, Prabhakar" add OF support for the tvp7002 driver. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc: Sakari Ailus Cc: Grant Likely Cc: Rob Herring Cc: Rob Landley Cc: devicetree-

[PATCH v2 1/2] media: i2c: tvp7002: add support for asynchronous probing

2013-06-22 Thread Prabhakar Lad
From: "Lad, Prabhakar" Both synchronous and asynchronous tvp7002 subdevice probing is supported by this patch. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc: Sakari Ailus Cc: linux-kernel@v

[PATCH v2 0/2] media: i2c: tvp7002: feature enhancement

2013-06-22 Thread Prabhakar Lad
From: "Lad, Prabhakar" The first patch of the series add support for asynchronous probing and the second patch adds OF support to tvp7002 driver. Lad, Prabhakar (2): media: i2c: tvp7002: add support for asynchronous probing media: i2c: tvp7002: add OF support .../devicetree/bindings/media/

[PATCH RFC v3] media: OF: add video sync endpoint property

2013-06-22 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch adds video sync properties as part of endpoint properties and also support to parse them in the parser. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc: Sakar

[PATCH] media: i2c: tvp514x: add support for asynchronous probing

2013-06-22 Thread Prabhakar Lad
From: "Lad, Prabhakar" Both synchronous and asynchronous tvp514x subdevice probing is supported by this patch. Signed-off-by: Prabhakar Lad Cc: Guennadi Liakhovetski Cc: Laurent Pinchart Cc: Hans Verkuil Cc: Sakari Ailus Cc: Mauro Carvalho Chehab --- drivers/media/i2c/tvp514

[PATCH] media: i2c: adv7343: add support for asynchronous probing

2013-06-22 Thread Prabhakar Lad
From: "Lad, Prabhakar" Both synchronous and asynchronous adv7343 subdevice probing is supported by this patch. Signed-off-by: Lad, Prabhakar Cc: Guennadi Liakhovetski Cc: Laurent Pinchart Cc: Hans Verkuil Cc: Sakari Ailus Cc: Mauro Carvalho Chehab --- drivers/media/i2c/adv7343.c | 15 ++

<    1   2   3   4   5   6   7   >