Re: [PATCH v4 00/11] media: davinci: vpif driver cleanup

From: Laurent Pinchart
Date: Mon Jun 17 2013 - 13:20:46 EST


Hi Prabhakar,

Thank you for the patches.

On Monday 17 June 2013 20:50:40 Prabhakar Lad wrote:
> From: "Lad, Prabhakar" <prabhakar.csengg@xxxxxxxxx>
>
> 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@xxxxxxxxxxxxxxxx>

> 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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/