From: Arnaud Patard (Rtp) <arnaud.patard@rtp-net.org>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/7] Add a mfd IPUv3 driver
Date: Thu, 17 Feb 2011 18:10:24 +0000 [thread overview]
Message-ID: <871v36pm5r.fsf@lechat.rtp-net.org> (raw)
In-Reply-To: <1297865452-32181-2-git-send-email-s.hauer@pengutronix.de> (Sascha Hauer's message of "Wed, 16 Feb 2011 15:10:46 +0100")
Sascha Hauer <s.hauer@pengutronix.de> writes:
Hi,
> The IPU is the Image Processing Unit found on i.MX51/53 SoCs. It
> features several units for image processing, this patch adds support
> for the units needed for Framebuffer support, namely:
>
> - Display Controller (dc)
> - Display Interface (di)
> - Display Multi Fifo Controller (dmfc)
> - Display Processor (dp)
> - Image DMA Controller (idmac)
>
> This patch is based on the Freescale driver, but follows a different
> approach. The Freescale code implements logical idmac channels and
> the handling of the subunits is hidden in common idmac code pathes
> in big switch/case statements. This patch instead just provides code
> and resource management for the different subunits. The user, in this
> case the framebuffer driver, decides how the different units play
> together.
>
> The IPU has other units missing in this patch:
>
> - CMOS Sensor Interface (csi)
> - Video Deinterlacer (vdi)
> - Sensor Multi FIFO Controler (smfc)
> - Image Converter (ic)
> - Image Rotator (irt)
>
> So expect more files to come in this directory.
>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: linux-kernel@vger.kernel.org
> Cc: linux-fbdev@vger.kernel.org
> Cc: Paul Mundt <lethal@linux-sh.org>
> Cc: Samuel Ortiz <sameo@linux.intel.com>
> ---
> arch/arm/plat-mxc/include/mach/ipu-v3.h | 49 +++
> drivers/video/Kconfig | 2 +
> drivers/video/Makefile | 1 +
> drivers/video/imx-ipu-v3/Makefile | 3 +
> drivers/video/imx-ipu-v3/ipu-common.c | 708 +++++++++++++++++++++++++++++++
> drivers/video/imx-ipu-v3/ipu-cpmem.c | 612 ++++++++++++++++++++++++++
> drivers/video/imx-ipu-v3/ipu-dc.c | 364 ++++++++++++++++
> drivers/video/imx-ipu-v3/ipu-di.c | 550 ++++++++++++++++++++++++
> drivers/video/imx-ipu-v3/ipu-dmfc.c | 355 ++++++++++++++++
> drivers/video/imx-ipu-v3/ipu-dp.c | 476 +++++++++++++++++++++
> drivers/video/imx-ipu-v3/ipu-prv.h | 216 ++++++++++
> include/linux/mfd/imx-ipu-v3.h | 219 ++++++++++
> 12 files changed, 3555 insertions(+), 0 deletions(-)
> create mode 100644 arch/arm/plat-mxc/include/mach/ipu-v3.h
> create mode 100644 drivers/video/imx-ipu-v3/Makefile
> create mode 100644 drivers/video/imx-ipu-v3/ipu-common.c
> create mode 100644 drivers/video/imx-ipu-v3/ipu-cpmem.c
> create mode 100644 drivers/video/imx-ipu-v3/ipu-dc.c
> create mode 100644 drivers/video/imx-ipu-v3/ipu-di.c
> create mode 100644 drivers/video/imx-ipu-v3/ipu-dmfc.c
> create mode 100644 drivers/video/imx-ipu-v3/ipu-dp.c
> create mode 100644 drivers/video/imx-ipu-v3/ipu-prv.h
> create mode 100644 include/linux/mfd/imx-ipu-v3.h
[...]
> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
> index 6bafb51b..ffdb37a 100644
> --- a/drivers/video/Kconfig
> +++ b/drivers/video/Kconfig
> @@ -26,6 +26,8 @@ source "drivers/gpu/drm/Kconfig"
>
> source "drivers/gpu/stub/Kconfig"
>
> +source "drivers/video/imx-ipu-v3/Kconfig"
> +
I don't see such a Kconfig file in this patch. Got lost while moving
from mfd to video ?
> config VGASTATE
> tristate
> default n
> diff --git a/drivers/video/Makefile b/drivers/video/Makefile
> index 8c8fabd..f4921ab 100644
> --- a/drivers/video/Makefile
> +++ b/drivers/video/Makefile
> @@ -153,6 +153,7 @@ obj-$(CONFIG_FB_BFIN_T350MCQB) += bfin-t350mcqb-fb.o
> obj-$(CONFIG_FB_BFIN_7393) += bfin_adv7393fb.o
> obj-$(CONFIG_FB_MX3) += mx3fb.o
> obj-$(CONFIG_FB_DA8XX) += da8xx-fb.o
> +obj-$(CONFIG_MFD_IMX_IPU_V3) += imx-ipu-v3/
Now that files are in drivers/video, do we want to keep MFD in the name ?
Arnaud
next prev parent reply other threads:[~2011-02-17 18:10 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1297865452-32181-1-git-send-email-s.hauer@pengutronix.de>
2011-02-16 14:10 ` [PATCH 2/7] fb: export fb mode db table Sascha Hauer
2011-02-16 14:10 ` [PATCH 3/7] Add i.MX5 framebuffer driver Sascha Hauer
2011-02-18 9:22 ` Jason Chen
2011-02-18 9:52 ` Sascha Hauer
2011-02-19 2:22 ` Jason Chen
[not found] ` <1297865452-32181-2-git-send-email-s.hauer@pengutronix.de>
2011-02-17 18:10 ` Arnaud Patard [this message]
2011-02-18 9:49 ` [PATCH 1/7] Add a mfd IPUv3 driver Sascha Hauer
2011-02-18 12:07 ` Samuel Ortiz
2011-02-21 5:38 ` Jason Chen
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=871v36pm5r.fsf@lechat.rtp-net.org \
--to=arnaud.patard@rtp-net.org \
--cc=linux-arm-kernel@lists.infradead.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).