Re: [v3] media: davinci: kconfig: fix incorrect selects

2013-05-23 Thread Hans Verkuil
On Mon 13 May 2013 12:41:26 Prabhakar Lad wrote: Hi Paul, On Mon, May 13, 2013 at 3:35 PM, Paul Bolle pebo...@tiscali.nl wrote: Prabhakar, On Mon, 2013-05-13 at 15:27 +0530, Prabhakar Lad wrote: Good catch! the dependency can be dropped now. Great. Are you planning to post a

Re: [v3] media: davinci: kconfig: fix incorrect selects

2013-05-23 Thread Prabhakar Lad
Hi Hans, On Thu, May 23, 2013 at 3:53 PM, Hans Verkuil hverk...@xs4all.nl wrote: On Mon 13 May 2013 12:41:26 Prabhakar Lad wrote: Hi Paul, On Mon, May 13, 2013 at 3:35 PM, Paul Bolle pebo...@tiscali.nl wrote: Prabhakar, On Mon, 2013-05-13 at 15:27 +0530, Prabhakar Lad wrote: Good

Re: [v3] media: davinci: kconfig: fix incorrect selects

2013-05-13 Thread Paul Bolle
On Tue, 2013-03-12 at 09:14 +, Sekhar Nori wrote: drivers/media/platform/davinci/Kconfig uses selects where it should be using 'depends on'. This results in warnings of the following sort when doing randconfig builds. warning: (VIDEO_DM6446_CCDC VIDEO_DM355_CCDC VIDEO_ISIF

Re: [v3] media: davinci: kconfig: fix incorrect selects

2013-05-13 Thread Prabhakar Lad
Hi Paul, On Mon, May 13, 2013 at 3:11 PM, Paul Bolle pebo...@tiscali.nl wrote: On Tue, 2013-03-12 at 09:14 +, Sekhar Nori wrote: [Snip] This patch has only been build tested; I have tried to not break any existing assumptions. I do not have the setup to test video, so any test reports

Re: [v3] media: davinci: kconfig: fix incorrect selects

2013-05-13 Thread Paul Bolle
Prabhakar, On Mon, 2013-05-13 at 15:27 +0530, Prabhakar Lad wrote: Good catch! the dependency can be dropped now. Great. Are you planning to post a patch for it or shall I do it ? I don't mind submitting that trivial patch. However, it's probably better if you do that. I can only state that

Re: [v3] media: davinci: kconfig: fix incorrect selects

2013-05-13 Thread Prabhakar Lad
Hi Paul, On Mon, May 13, 2013 at 3:35 PM, Paul Bolle pebo...@tiscali.nl wrote: Prabhakar, On Mon, 2013-05-13 at 15:27 +0530, Prabhakar Lad wrote: Good catch! the dependency can be dropped now. Great. Are you planning to post a patch for it or shall I do it ? I don't mind submitting that

Re: [PATCH v3] media: davinci: kconfig: fix incorrect selects

2013-04-08 Thread Prabhakar Lad
Hi Sekhar, On Tue, Mar 12, 2013 at 2:44 PM, Sekhar Nori nsek...@ti.com wrote: drivers/media/platform/davinci/Kconfig uses selects where it should be using 'depends on'. This results in warnings of the following sort when doing randconfig builds. warning: (VIDEO_DM6446_CCDC VIDEO_DM355_CCDC

Re: [PATCH v3] media: davinci: kconfig: fix incorrect selects

2013-04-08 Thread Sekhar Nori
On 4/8/2013 1:39 PM, Prabhakar Lad wrote: Hi Sekhar, config VIDEO_DAVINCI_VPBE_DISPLAY - tristate DM644X/DM365/DM355 VPBE HW module - depends on ARCH_DAVINCI_DM644x || ARCH_DAVINCI_DM355 || ARCH_DAVINCI_DM365 - select VIDEO_VPSS_SYSTEM + tristate TI DaVinci VPBE

Re: [PATCH v3] media: davinci: kconfig: fix incorrect selects

2013-03-18 Thread Sekhar Nori
On 3/16/2013 2:06 PM, Prabhakar Lad wrote: Hi Sekhar, Thanks for the patch! On Tue, Mar 12, 2013 at 2:44 PM, Sekhar Nori nsek...@ti.com wrote: drivers/media/platform/davinci/Kconfig uses selects where it should be using 'depends on'. This results in warnings of the following sort when

Re: [PATCH v3] media: davinci: kconfig: fix incorrect selects

2013-03-16 Thread Prabhakar Lad
Hi Sekhar, Thanks for the patch! On Tue, Mar 12, 2013 at 2:44 PM, Sekhar Nori nsek...@ti.com wrote: drivers/media/platform/davinci/Kconfig uses selects where it should be using 'depends on'. This results in warnings of the following sort when doing randconfig builds. warning: