From: JiUn Yu <jiun.yu@samsung.com>
To: Tomasz Stanislawski <t.stanislaws@samsung.com>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Cc: "Marek Szyprowski" <m.szyprowski@samsung.com>,
"Kyungmin Park" <kyungmin.park@samsung.com>,
"hverkuil@xs4all.nl" <hverkuil@xs4all.nl>,
선경일 <ki.sun@samsung.com>, 김영락 <younglak1004.kim@samsung.com>,
이일호 <ilho215.lee@samsung.com>, 김경환 <kh.k.kim@samsung.com>
Subject: Re: [PATCH v4 0/3] TV driver for Samsung S5P platform (media part)
Date: Sun, 29 May 2011 11:33:27 +0000 (GMT) [thread overview]
Message-ID: <599405.4311306668807678.JavaMail.weblogic@epml25> (raw)
> 5. Mixer & Video Processor driver. It is called 's5p-mixer' because of
> historical reasons. It was decided combine VP and MXR drivers into one because
> of shared interrupt and very similar interface via V4L2 nodes. The driver is a
> realization of many-to-many relation between multiple input layers and multiple
> outputs. All shared resources are kept in struct mxr_device. It provides
> utilities for management and synchronization of access to resources and
> reference counting. The outputs are obtained from HDMI/SDO private data. One
> layer is a single video node. Simple inheritance is applied because there only
> little difference between layer's types. Every layer type implements set of
> ops. There are different ops for Mixer layers and other for VP layer.
I agreed with subdev of hdmi, hdmiphy, sdo and videoDAC. It is very flexible in case of adding new interface or removing current interface.
But 's5p-mixer' driver is not flexible. So, If new scaler is added instead of VP or mixer is someting changed,
I think current architecture of tvout driver can't support.
How about separating vp and mixer driver?
next reply other threads:[~2011-05-29 11:33 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-29 11:33 JiUn Yu [this message]
2011-05-31 10:20 ` [PATCH v4 0/3] TV driver for Samsung S5P platform (media part) Tomasz Stanislawski
2011-06-02 0:57 ` Kyungmin Park
-- strict thread matches above, loose matches on Subject: below --
2011-05-24 13:09 Tomasz Stanislawski
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=599405.4311306668807678.JavaMail.weblogic@epml25 \
--to=jiun.yu@samsung.com \
--cc=hverkuil@xs4all.nl \
--cc=ilho215.lee@samsung.com \
--cc=kh.k.kim@samsung.com \
--cc=ki.sun@samsung.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-media@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=t.stanislaws@samsung.com \
--cc=younglak1004.kim@samsung.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox