From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Manjunath Hadli <manjunath.hadli@ti.com>
Cc: LMML <linux-media@vger.kernel.org>,
dlos <davinci-linux-open-source@linux.davincidsp.com>
Subject: Re: [PATCH v2 0/8] RFC for Media Controller capture driver for DM365
Date: Sun, 18 Sep 2011 11:35:46 -0300 [thread overview]
Message-ID: <4E7601C2.9030900@redhat.com> (raw)
In-Reply-To: <1314630439-1122-1-git-send-email-manjunath.hadli@ti.com>
Em 29-08-2011 12:07, Manjunath Hadli escreveu:
> changes from last patch set:
> 1. Made changes based on Sakari's feedback mainly:
> a. returned early to allow unindenting
> b. reformatting to shift the code to left where possible
> c. changed hex numbers to lower case
> d. corrected the defines according to module names.
> e. magic numbers removed.
> f. changed non-integer returning functions to void
> g. removed unwanted paranthses.
> h. fixed error codes.
> i. fixed some RESET_BIt code to what it was intended for.
> 2. reorganized the header files to move the kernel-only headers
> along with the c files and interface headers in the include folder.
Manju,
Please be sure to send me a pull request when you think this driver
is ready for merge. The first submission I'm noticing for this driver
was back on 2009, and still today, nobody sent me a git pull request
for it. Two years seems too much time to solve the pending issues
and sending a pull request for me to merge it!
>
> Manjunath Hadli (6):
> davinci: vpfe: add dm3xx IPIPEIF hardware support module
> davinci: vpfe: add support for CCDC hardware for dm365
> davinci: vpfe: add ccdc driver with media controller interface
> davinci: vpfe: add v4l2 video driver support
> davinci: vpfe: v4l2 capture driver with media interface
> davinci: vpfe: build infrastructure for dm365
>
> Nagabhushana Netagunte (2):
> davinci: vpfe: add IPIPE hardware layer support
> davinci: vpfe: add IPIPE support for media controller driver
>
> drivers/media/video/davinci/Kconfig | 46 +-
> drivers/media/video/davinci/Makefile | 17 +-
> drivers/media/video/davinci/ccdc_hw_device.h | 10 +-
> drivers/media/video/davinci/ccdc_types.h | 43 +
> drivers/media/video/davinci/dm365_ccdc.c | 1519 ++++++++++
> drivers/media/video/davinci/dm365_ccdc.h | 88 +
> drivers/media/video/davinci/dm365_ccdc_regs.h | 309 ++
> drivers/media/video/davinci/dm365_def_para.c | 486 +++
> drivers/media/video/davinci/dm365_def_para.h | 39 +
> drivers/media/video/davinci/dm365_ipipe.c | 3966 +++++++++++++++++++++++++
> drivers/media/video/davinci/dm365_ipipe.h | 300 ++
> drivers/media/video/davinci/dm365_ipipe_hw.c | 949 ++++++
> drivers/media/video/davinci/dm365_ipipe_hw.h | 539 ++++
> drivers/media/video/davinci/dm3xx_ipipeif.c | 317 ++
> drivers/media/video/davinci/dm3xx_ipipeif.h | 258 ++
> drivers/media/video/davinci/imp_common.h | 85 +
> drivers/media/video/davinci/imp_hw_if.h | 178 ++
> drivers/media/video/davinci/vpfe_capture.c | 793 +++++
> drivers/media/video/davinci/vpfe_capture.h | 104 +
> drivers/media/video/davinci/vpfe_ccdc.c | 813 +++++
> drivers/media/video/davinci/vpfe_ccdc.h | 85 +
> drivers/media/video/davinci/vpfe_video.c | 1712 +++++++++++
> drivers/media/video/davinci/vpfe_video.h | 142 +
> include/linux/davinci_vpfe.h | 1223 ++++++++
> include/linux/dm365_ccdc.h | 664 +++++
> include/linux/dm3xx_ipipeif.h | 64 +
> include/media/davinci/vpfe.h | 91 +
> 27 files changed, 14829 insertions(+), 11 deletions(-)
> create mode 100644 drivers/media/video/davinci/ccdc_types.h
> create mode 100644 drivers/media/video/davinci/dm365_ccdc.c
> create mode 100644 drivers/media/video/davinci/dm365_ccdc.h
> create mode 100644 drivers/media/video/davinci/dm365_ccdc_regs.h
> create mode 100644 drivers/media/video/davinci/dm365_def_para.c
> create mode 100644 drivers/media/video/davinci/dm365_def_para.h
> create mode 100644 drivers/media/video/davinci/dm365_ipipe.c
> create mode 100644 drivers/media/video/davinci/dm365_ipipe.h
> create mode 100644 drivers/media/video/davinci/dm365_ipipe_hw.c
> create mode 100644 drivers/media/video/davinci/dm365_ipipe_hw.h
> create mode 100644 drivers/media/video/davinci/dm3xx_ipipeif.c
> create mode 100644 drivers/media/video/davinci/dm3xx_ipipeif.h
> create mode 100644 drivers/media/video/davinci/imp_common.h
> create mode 100644 drivers/media/video/davinci/imp_hw_if.h
> create mode 100644 drivers/media/video/davinci/vpfe_capture.c
> create mode 100644 drivers/media/video/davinci/vpfe_capture.h
> create mode 100644 drivers/media/video/davinci/vpfe_ccdc.c
> create mode 100644 drivers/media/video/davinci/vpfe_ccdc.h
> create mode 100644 drivers/media/video/davinci/vpfe_video.c
> create mode 100644 drivers/media/video/davinci/vpfe_video.h
> create mode 100644 include/linux/davinci_vpfe.h
> create mode 100644 include/linux/dm365_ccdc.h
> create mode 100644 include/linux/dm3xx_ipipeif.h
> create mode 100644 include/media/davinci/vpfe.h
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2011-09-18 14:35 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-29 15:07 [PATCH v2 0/8] RFC for Media Controller capture driver for DM365 Manjunath Hadli
2011-08-29 15:07 ` [PATCH v2 1/8] davinci: vpfe: add dm3xx IPIPEIF hardware support module Manjunath Hadli
2011-09-01 20:47 ` Sylwester Nawrocki
2011-09-09 13:20 ` Hadli, Manjunath
2011-08-29 15:07 ` [PATCH v2 2/8] davinci: vpfe: add IPIPE hardware layer support Manjunath Hadli
2011-08-29 15:07 ` [PATCH v2 3/8] davinci: vpfe: add IPIPE support for media controller driver Manjunath Hadli
2011-09-27 14:01 ` Sakari Ailus
2011-10-17 14:16 ` Hadli, Manjunath
2011-08-29 15:07 ` [PATCH v2 4/8] davinci: vpfe: add support for CCDC hardware for dm365 Manjunath Hadli
2011-09-01 21:23 ` Sylwester Nawrocki
2011-09-09 13:30 ` Hadli, Manjunath
2011-09-09 16:39 ` Sylwester Nawrocki
2011-08-29 15:07 ` [PATCH v2 5/8] davinci: vpfe: add ccdc driver with media controller interface Manjunath Hadli
2011-08-29 15:07 ` [PATCH v2 6/8] davinci: vpfe: add v4l2 video driver support Manjunath Hadli
2011-08-29 15:07 ` [PATCH v2 7/8] davinci: vpfe: v4l2 capture driver with media interface Manjunath Hadli
2011-08-29 15:07 ` [PATCH v2 8/8] davinci: vpfe: build infrastructure for dm365 Manjunath Hadli
2011-08-29 15:20 ` [PATCH v2 0/8] RFC for Media Controller capture driver for DM365 Hadli, Manjunath
2011-08-31 21:30 ` Sakari Ailus
2011-09-09 13:40 ` Hadli, Manjunath
2011-09-09 16:19 ` Sakari Ailus
2011-09-10 6:41 ` Hadli, Manjunath
2011-09-12 11:59 ` Sakari Ailus
2011-09-12 14:18 ` Hadli, Manjunath
2011-09-14 19:02 ` Hadli, Manjunath
2011-09-16 21:05 ` Sakari Ailus
2011-09-18 14:35 ` Mauro Carvalho Chehab [this message]
2011-09-19 5:09 ` Hadli, Manjunath
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4E7601C2.9030900@redhat.com \
--to=mchehab@redhat.com \
--cc=davinci-linux-open-source@linux.davincidsp.com \
--cc=linux-media@vger.kernel.org \
--cc=manjunath.hadli@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.