From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Prabhakar Lad <prabhakar.csengg@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>,
LMML <linux-media@vger.kernel.org>,
DLOS <davinci-linux-open-source@linux.davincidsp.com>,
Hans Verkuil <hans.verkuil@cisco.com>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4 00/11] media: davinci: vpif driver cleanup
Date: Mon, 17 Jun 2013 19:20:54 +0200 [thread overview]
Message-ID: <2109154.PkSYdnQTas@avalon> (raw)
In-Reply-To: <1371482451-18314-1-git-send-email-prabhakar.csengg@gmail.com>
Hi Prabhakar,
Thank you for the patches.
On Monday 17 June 2013 20:50:40 Prabhakar Lad wrote:
> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
>
> This patch series cleans the VPIF driver, uses devm_* api wherever
> required and uses module_platform_driver() to simplify the code.
>
> This patch series applies on http://git.linuxtv.org/hverkuil/media_tree.git/
> shortlog/refs/heads/for-v3.11 and is tested on OMAP-L138 EVM.
>
> Changes for v2:
> 1: Rebased on v3.11 branch of Hans.
> 2: Dropped the patches which removed headers as mentioned by Laurent.
>
> Changes for v3:
> 1: Splitted the patches logically as mentioned by Laurent.
> 2: Fixed review comments pointed by Laurent.
> 3: Included Ack's.
>
> Changes for v4:
> 1: Rebased on v3.11 branch of Hans.
> 2: Fixed review comments pointed by Laurent and Sergei.
> 3: Included Ack's.
> 4: Removed unnecessary loop for IRQ resource.
For the whole series,
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Lad, Prabhakar (11):
> media: davinci: vpif: remove unwanted header mach/hardware.h and sort
> the includes alphabetically
> media: davinci: vpif: Convert to devm_* api
> media: davinci: vpif: remove unnecessary braces around defines
> media: davinci: vpif_capture: move the freeing of irq and global
> variables to remove()
> media: davinci: vpif_capture: use module_platform_driver()
> media: davinci: vpif_capture: Convert to devm_* api
> media: davinci: vpif_capture: remove unnecessary loop for IRQ
> resource
> media: davinci: vpif_display: move the freeing of irq and global
> variables to remove()
> media: davinci: vpif_display: use module_platform_driver()
> media: davinci: vpif_display: Convert to devm_* api
> media: davinci: vpif_display: remove unnecessary loop for IRQ
> resource
>
> drivers/media/platform/davinci/vpif.c | 45 ++++-----------
> drivers/media/platform/davinci/vpif_capture.c | 76 +++++-----------------
> drivers/media/platform/davinci/vpif_display.c | 65 +++++----------------
> 3 files changed, 39 insertions(+), 147 deletions(-)
--
Regards,
Laurent Pinchart
prev parent reply other threads:[~2013-06-17 17:20 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-17 15:20 [PATCH v4 00/11] media: davinci: vpif driver cleanup Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 01/11] media: davinci: vpif: remove unwanted header mach/hardware.h and sort the includes alphabetically Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 02/11] media: davinci: vpif: Convert to devm_* api Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 03/11] media: davinci: vpif: remove unnecessary braces around defines Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 04/11] media: davinci: vpif_capture: move the freeing of irq and global variables to remove() Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 05/11] media: davinci: vpif_capture: use module_platform_driver() Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 06/11] media: davinci: vpif_capture: Convert to devm_* api Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 07/11] media: davinci: vpif_capture: remove unnecessary loop for IRQ resource Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 08/11] media: davinci: vpif_display: move the freeing of irq and global variables to remove() Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 09/11] media: davinci: vpif_display: use module_platform_driver() Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 10/11] media: davinci: vpif_display: Convert to devm_* api Prabhakar Lad
2013-06-17 15:20 ` [PATCH v4 11/11] media: davinci: vpif_display: remove unnecessary loop for IRQ resource Prabhakar Lad
2013-06-17 17:20 ` Laurent Pinchart [this message]
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=2109154.PkSYdnQTas@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=davinci-linux-open-source@linux.davincidsp.com \
--cc=hans.verkuil@cisco.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@redhat.com \
--cc=prabhakar.csengg@gmail.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.