RE: [PATCH v17 00/13] davinci vpbe: dm6446 v4l2 driver

2011-03-22 Thread Hadli, Manjunath
Sekhar, Kevin, 
 These patches have gone through considerable reviews. 
Could you please ACK from your end?


Thanks and Regards,
-Manju

On Tue, Mar 15, 2011 at 19:26:28, Hadli, Manjunath wrote:
 version17:
 The more important among the patch history from previous comments 1. 
 Replacing _raw_readl() with readl().
 2. Removal of platform resource overlap.
 3. Removal of unused macros.
 4. Fixing the module params typo.
 5. Minor changes in the GPL licensing header.
 6. Removed the initializer for field inversion parameter.
 7. Changing the Field inversion #ifdef to platform 
based implementation.
 8. Interchanged platform and board specific patches due to dependencies.
 
 Manjunath Hadli (13):
   davinci vpbe: V4L2 display driver for DM644X SoC
   davinci vpbe: VPBE display driver
   davinci vpbe: OSD(On Screen Display) block
   davinci vpbe: VENC( Video Encoder) implementation
   davinci vpbe: Build infrastructure for VPBE driver
   davinci vpbe: Readme text for Dm6446 vpbe
   davinci: move DM64XX_VDD3P3V_PWDN to devices.c
   davinci: eliminate use of IO_ADDRESS() on sysmod
   davinci: dm644x: Replace register base value with a defined macro
   davinci: dm644x: change vpfe capture structure variables for
 consistency
   davinci: dm644x: move vpfe init from soc to board specific files
   davinci: dm644x: add support for v4l2 video display
   davinci: dm644x EVM: add support for VPBE display
 
  Documentation/video4linux/README.davinci-vpbe |   93 ++
  arch/arm/mach-davinci/board-dm644x-evm.c  |  131 ++-
  arch/arm/mach-davinci/devices.c   |   24 +-
  arch/arm/mach-davinci/dm355.c |1 +
  arch/arm/mach-davinci/dm365.c |1 +
  arch/arm/mach-davinci/dm644x.c|  172 ++-
  arch/arm/mach-davinci/dm646x.c|1 +
  arch/arm/mach-davinci/include/mach/dm644x.h   |7 +-
  arch/arm/mach-davinci/include/mach/hardware.h |7 +-
  drivers/media/video/davinci/Kconfig   |   22 +
  drivers/media/video/davinci/Makefile  |2 +
  drivers/media/video/davinci/vpbe.c|  826 ++
  drivers/media/video/davinci/vpbe_display.c| 2084 
 +
  drivers/media/video/davinci/vpbe_osd.c| 1216 ++
  drivers/media/video/davinci/vpbe_osd_regs.h   |  364 +
  drivers/media/video/davinci/vpbe_venc.c   |  556 +++
  drivers/media/video/davinci/vpbe_venc_regs.h  |  177 +++
  include/media/davinci/vpbe.h  |  185 +++
  include/media/davinci/vpbe_display.h  |  146 ++
  include/media/davinci/vpbe_osd.h  |  397 +
  include/media/davinci/vpbe_types.h|   91 ++
  include/media/davinci/vpbe_venc.h |   41 +
  22 files changed, 6500 insertions(+), 44 deletions(-)  create mode 100644 
 Documentation/video4linux/README.davinci-vpbe
  create mode 100644 drivers/media/video/davinci/vpbe.c
  create mode 100644 drivers/media/video/davinci/vpbe_display.c
  create mode 100644 drivers/media/video/davinci/vpbe_osd.c
  create mode 100644 drivers/media/video/davinci/vpbe_osd_regs.h
  create mode 100644 drivers/media/video/davinci/vpbe_venc.c
  create mode 100644 drivers/media/video/davinci/vpbe_venc_regs.h
  create mode 100644 include/media/davinci/vpbe.h  create mode 100644 
 include/media/davinci/vpbe_display.h
  create mode 100644 include/media/davinci/vpbe_osd.h  create mode 100644 
 include/media/davinci/vpbe_types.h
  create mode 100644 include/media/davinci/vpbe_venc.h
 
 

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH v17 00/13] davinci vpbe: dm6446 v4l2 driver

2011-03-22 Thread Nori, Sekhar
Manju,

On Tue, Mar 22, 2011 at 12:23:14, Hadli, Manjunath wrote:
 Sekhar, Kevin, 
  These patches have gone through considerable reviews. 
 Could you please ACK from your end?

I have some minor comments which I have already posted and
once you fix those you can add:

Acked-by: Sekhar Nori nsek...@ti.com

to the platform patches.

 
 On Tue, Mar 15, 2011 at 19:26:28, Hadli, Manjunath wrote:
  version17:
  The more important among the patch history from previous comments 1. 
  Replacing _raw_readl() with readl().

This is not valid.

Thanks,
Sekhar

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html