Re: [PATCH v3 0/2] ARM: davinci: da850/omap-l138: add support for VPIF driver

[Sekhar Nori <nsekhar@ti.com>]

Hi Prabhakar,

On 9/11/2012 11:55 AM, Prabhakar Lad wrote:
> This patch series adds support for VPIF
> capture and display driver on da850/omap-l138.
> Enables SD capture and display.
> 
> This patch series is dependent on the following patch:
> https://patchwork.kernel.org/patch/1332311/
> 
> Changes for v3:
> 1: Clubbed the code for DA850_UI_SD_VIDEO_PORT config
>    as pointed by Sekhar.
> 2: Define  the resource structure outside the function as pointed
>    by Sekhar.
> 
> Changes for v2:
> 1: Avoid breaking of print messages.
> 2: Removed the handlers which just returned zero (which did nothing).
> 3: Clubbed the code where ever possible for DA850_UI_SD_VIDEO_PORT
>    config option.
> 4: Removed the dma_declare_coherent_memory() calls and used
>    global CMA.
> 5: Added the base address in increasing order.
> 
> Manjunath Hadli (2):
>   ARM: da850/omap-l138: Add SoC related definitions for VPIF
>   ARM: da850/omap-l138: Add EVM specific code for VPIF to work
> 
> Manjunath Hadli (2):
>   ARM: da850/omap-l138: Add SoC related definitions for VPIF
>   ARM: da850/omap-l138: Add EVM specific code for VPIF to work
> 
>  arch/arm/mach-davinci/Kconfig              |    7 ++
>  arch/arm/mach-davinci/board-da850-evm.c    |  156 ++++++++++++++++++++++++++++
>  arch/arm/mach-davinci/da850.c              |  152 +++++++++++++++++++++++++++
>  arch/arm/mach-davinci/include/mach/da8xx.h |   11 ++
>  arch/arm/mach-davinci/include/mach/mux.h   |   42 ++++++++
>  arch/arm/mach-davinci/include/mach/psc.h   |    1 +
>  6 files changed, 369 insertions(+), 0 deletions(-)

The patches look good to me. In case you want to queue them through the
media tree to manage the dependencies, feel free to add my:

Acked-by: Sekhar Nori <nsekhar@ti.com>

Note that the subject like prefix for DA850 soc patches should be

"ARM: davinci: da850: .."

and that for DA850 EVM patches should be:

"ARM: davinci: da850 evm: .."

instead of what you have used here. Please fix these if you are queuing
them and take care of this in future.

Thanks,
Sekhar