linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Ran Shalit <ranshalit@gmail.com>, Steven Toth <stoth@kernellabs.com>
Cc: linux-media@vger.kernel.org
Subject: Re: PCIe capture driver
Date: Tue, 27 Oct 2015 07:21:52 +0900	[thread overview]
Message-ID: <562EA780.7070706@xs4all.nl> (raw)
In-Reply-To: <CAJ2oMh++Ed43esZi3jnO7SZtc6ySmkmxaydEGPU=PY=UCxhGig@mail.gmail.com>



On 10/27/2015 02:04, Ran Shalit wrote:
> On Mon, Oct 26, 2015 at 1:46 PM, Steven Toth <stoth@kernellabs.com> wrote:
>>> No, use V4L2. What you do with the frame after it has been captured
>>> into memory has no relevance to the API you use to capture into memory.
>>
>> Ran, I've built many open and closed source Linux drivers over the
>> last 10 years - so I can speak with authority on this.
>>
>> Hans is absolutely correct, don't make the mistake of going
>> proprietary with your API. Take advantage of the massive amount of
>> video related frameworks the kernel has to offer. It will get you to
>> market faster, assuming your goal is to build a driver that is open
>> source. If your licensing prohibits an open source driver solution,
>> you'll have no choice but to build your own proprietary API.
>>
>> --
>> Steven Toth - Kernel Labs
>> http://www.kernellabs.com
> 
> Hi,
> 
> Thank you very much for these valuable comments.
> If I may ask one more on this issue:
> Is there an example in linux tree, for a pci device which is used both
> as a capture and a display device ? (I've made a search but did not
> find any)
> The PCIe device we are using will be both a capture device and output
> video device (for display).

The cobalt driver (drivers/media/pci/cobalt) does exactly that: multiple HDMI inputs and an optional HDMI output (through a daughterboard).

Please note: using V4L2 for an output only makes sense if you will be outputting video, if the goal is to output a graphical desktop then the drm/kms API is much more suitable.

Regards,

	Hans

  parent reply	other threads:[~2015-10-26 22:22 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-19 20:26 PCIe capture driver Ran Shalit
2015-10-20  6:23 ` Hans Verkuil
2015-10-20  6:38   ` Ran Shalit
2015-10-23 21:57   ` Ran Shalit
2015-10-24  9:38     ` Hans Verkuil
2015-10-25 20:16       ` Ran Shalit
2015-10-25 22:55         ` Hans Verkuil
2015-10-26 11:46           ` Steven Toth
2015-10-26 17:04             ` Ran Shalit
2015-10-26 17:10               ` Jean-Michel Hautbois
2015-10-26 17:22               ` Steven Toth
2015-10-26 22:21               ` Hans Verkuil [this message]
2015-10-27 13:56                 ` Ran Shalit
2015-10-27 22:50                   ` Hans Verkuil
2015-10-30  9:18                     ` Ran Shalit
2015-11-11  6:04                     ` Ran Shalit
2015-11-11  7:22                       ` Hans Verkuil

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=562EA780.7070706@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=ranshalit@gmail.com \
    --cc=stoth@kernellabs.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;
as well as URLs for NNTP newsgroup(s).