[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <201305231138.12523.hverkuil@xs4all.nl>
Date: Thu, 23 May 2013 11:38:12 +0200
From: Hans Verkuil <hverkuil@...all.nl>
To: Lad Prabhakar <prabhakar.csengg@...il.com>
Cc: DLOS <davinci-linux-open-source@...ux.davincidsp.com>,
LMML <linux-media@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>,
Mauro Carvalho Chehab <mchehab@...hat.com>,
Hans Verkuil <hans.verkuil@...co.com>,
Laurent Pinchart <laurent.pinchart@...asonboard.com>
Subject: Re: [PATCH 0/7] media: davinci: vpif trivial cleanup
On Thu 16 May 2013 14:58:15 Lad Prabhakar wrote:
> From: Lad, Prabhakar <prabhakar.csengg@...il.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 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists