All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nicolas Dufresne <nicolas.dufresne@gmail.com>
To: tiffany lin <tiffany.lin@mediatek.com>,
	Hans Verkuil <hverkuil@xs4all.nl>
Cc: Pawel Osciak <pawel@osciak.com>,
	Hans Verkuil <hans.verkuil@cisco.com>,
	daniel.thompson@linaro.org, Rob Herring <robh+dt@kernel.org>,
	Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Daniel Kurtz <djkurtz@chromium.org>,
	Pawel Osciak <posciak@chromium.org>,
	Eddie Huang <eddie.huang@mediatek.com>,
	Yingjoe Chen <yingjoe.chen@mediatek.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-media@vger.kernel.org, linux-mediatek@lists.infradead.org,
	PoChun.Lin@mediatek.com
Subject: Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver
Date: Mon, 18 Apr 2016 13:48:17 -0400	[thread overview]
Message-ID: <1461001697.2719.7.camel@gmail.com> (raw)
In-Reply-To: <1460967754.7861.61.camel@mtksdaap41>

Le lundi 18 avril 2016 à 16:22 +0800, tiffany lin a écrit :
> > > We are plaining to remove m2m framework in th feature, although
> we think
> > 
> > Remove it for just the decoder driver or both encoder and decoder?
> > 
> Remove it from decoder driver.

Did you look at how CODA handle it (drivers/media/platform/coda/coda-
common.c) ? I don't know any detail, but they do have the same issue
and use both v4l2_m2m_fop_poll and v4l2_m2m_fop_mmap.

cheers,
Nicolas

WARNING: multiple messages have this Message-ID (diff)
From: nicolas.dufresne@gmail.com (Nicolas Dufresne)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver
Date: Mon, 18 Apr 2016 13:48:17 -0400	[thread overview]
Message-ID: <1461001697.2719.7.camel@gmail.com> (raw)
In-Reply-To: <1460967754.7861.61.camel@mtksdaap41>

Le lundi 18 avril 2016 ? 16:22 +0800, tiffany lin a ?crit?:
> > > We are plaining to remove m2m framework in th feature, although
> we think
> >?
> > Remove it for just the decoder driver or both encoder and decoder?
> >?
> Remove it from decoder driver.

Did you look at how CODA handle it (drivers/media/platform/coda/coda-
common.c) ? I don't know any detail, but they do have the same issue
and use both?v4l2_m2m_fop_poll and?v4l2_m2m_fop_mmap.

cheers,
Nicolas

  reply	other threads:[~2016-04-18 17:48 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-13 12:01 [PATCH 0/7] Add MT8173 Video Decoder Driver Tiffany Lin
2016-04-13 12:01 ` Tiffany Lin
2016-04-13 12:01 ` Tiffany Lin
2016-04-13 12:01 ` [PATCH 1/7] [media]: v4l: add Mediatek MT21 video block format Tiffany Lin
2016-04-13 12:01   ` Tiffany Lin
2016-04-13 12:01   ` Tiffany Lin
2016-04-13 12:01   ` [PATCH 2/7] dt-bindings: Add a binding for Mediatek Video Decoder Tiffany Lin
2016-04-13 12:01     ` Tiffany Lin
2016-04-13 12:01     ` Tiffany Lin
2016-04-13 12:01     ` [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver Tiffany Lin
2016-04-13 12:01       ` Tiffany Lin
2016-04-13 12:01       ` Tiffany Lin
2016-04-13 12:01       ` [PATCH 4/7] [media] vcodec: mediatek: Add Mediatek H264 " Tiffany Lin
2016-04-13 12:01         ` Tiffany Lin
2016-04-13 12:01         ` Tiffany Lin
2016-04-13 12:01         ` [PATCH 5/7] [media] vcodec: mediatek: Add Mediatek VP8 " Tiffany Lin
2016-04-13 12:01           ` Tiffany Lin
2016-04-13 12:01           ` Tiffany Lin
2016-04-13 12:01           ` [PATCH 6/7] [media] vcodec: mediatek: Add Mediatek VP9 " Tiffany Lin
2016-04-13 12:01             ` Tiffany Lin
2016-04-13 12:01             ` Tiffany Lin
2016-04-13 12:01             ` [PATCH 7/7] arm64: dts: mediatek: Add Video Encoder for MT8173 Tiffany Lin
2016-04-13 12:01               ` Tiffany Lin
2016-04-13 12:01               ` Tiffany Lin
2016-04-15 14:27       ` [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver Hans Verkuil
2016-04-15 14:27         ` Hans Verkuil
2016-04-15 14:27         ` Hans Verkuil
2016-04-18  4:00         ` tiffany lin
2016-04-18  4:00           ` tiffany lin
2016-04-18  4:00           ` tiffany lin
2016-04-18  5:40         ` tiffany lin
2016-04-18  5:40           ` tiffany lin
2016-04-18  5:40           ` tiffany lin
2016-04-18  7:32           ` Hans Verkuil
2016-04-18  7:32             ` Hans Verkuil
2016-04-18  7:32             ` Hans Verkuil
2016-04-18  8:22             ` tiffany lin
2016-04-18  8:22               ` tiffany lin
2016-04-18  8:22               ` tiffany lin
2016-04-18 17:48               ` Nicolas Dufresne [this message]
2016-04-18 17:48                 ` Nicolas Dufresne
2016-04-19  6:46                 ` tiffany lin
2016-04-19  6:46                   ` tiffany lin
2016-04-19  6:46                   ` tiffany lin
2016-04-14 16:15     ` [PATCH 2/7] dt-bindings: Add a binding for Mediatek Video Decoder Rob Herring
2016-04-14 16:15       ` Rob Herring
2016-04-14 16:15       ` Rob Herring
2016-04-13 14:23   ` [PATCH 1/7] [media]: v4l: add Mediatek MT21 video block format Nicolas Dufresne
2016-04-13 14:23     ` Nicolas Dufresne
2016-04-14  6:13     ` tiffany lin
2016-04-14  6:13       ` tiffany lin
2016-04-14  6:13       ` tiffany lin

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=1461001697.2719.7.camel@gmail.com \
    --to=nicolas.dufresne@gmail.com \
    --cc=PoChun.Lin@mediatek.com \
    --cc=daniel.thompson@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=djkurtz@chromium.org \
    --cc=eddie.huang@mediatek.com \
    --cc=hans.verkuil@cisco.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=mchehab@osg.samsung.com \
    --cc=nicolas@ndufresne.ca \
    --cc=pawel@osciak.com \
    --cc=posciak@chromium.org \
    --cc=robh+dt@kernel.org \
    --cc=tiffany.lin@mediatek.com \
    --cc=yingjoe.chen@mediatek.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.