Re: [PATCH 0/7] media: davinci: vpif trivial cleanup

From: Hans Verkuil
Date: Thu May 23 2013 - 05:38:33 EST


On Thu 16 May 2013 14:58:15 Lad Prabhakar 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 3.10.rc1 and is tested on OMAP-L138.

Can you repost this taken into account Laurent's comments regarding the
unwanted header includes?

Thanks,

Hans

>
> Lad, Prabhakar (7):
> media: davinci: vpif: remove unwanted header includes
> media: davinci: vpif: Convert to devm_* api
> media: davinci: vpif: remove unnecessary braces around defines
> media: davinci: vpif_capture: remove unwanted header inclusion and
> sort them alphabetically
> media: davinci: vpif_capture: Convert to devm_* api
> media: davinci: vpif_display: remove unwanted header inclusion and
> sort them alphabetically
> media: davinci: vpif_display: Convert to devm_* api
>
> drivers/media/platform/davinci/vpif.c | 42 ++---------
> drivers/media/platform/davinci/vpif_capture.c | 96 +++++--------------------
> drivers/media/platform/davinci/vpif_capture.h | 6 +-
> drivers/media/platform/davinci/vpif_display.c | 85 ++++------------------
> drivers/media/platform/davinci/vpif_display.h | 6 +-
> 5 files changed, 46 insertions(+), 189 deletions(-)
>
>
--
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/