From: Hans Verkuil <hverkuil@xs4all.nl>
To: Antti Palosaari <crope@iki.fi>, linux-media@vger.kernel.org
Subject: Re: [ATTN 0/9] SDR transmitter API
Date: Fri, 05 Jun 2015 11:50:17 +0200 [thread overview]
Message-ID: <557170D9.3010607@xs4all.nl> (raw)
In-Reply-To: <1432660090-19574-1-git-send-email-crope@iki.fi>
Hi Antti,
FYI: don't use ATTN as prefix: patchwork doesn't pick that up as a patch,
and if it doesn't appear there, then it is likely to be forgotten.
Can you repost it with the correct prefix? It's so much easier to process
for me if it ends up in patchwork...
Thanks!
Hans
On 05/26/2015 07:08 PM, Antti Palosaari wrote:
> That patch set adds V4L2 API support for SDR transmitters, with real
> life working example driver - HackRF.
>
> It also contains small change for old SDR API as I decided to rename
> tuner type from V4L2_TUNER_ADC to V4L2_TUNER_SDR. ADC is SDR receiver
> and DAC is SDR transmitter, so I eventually though it is better to
> rename ADC to common term than add new type for DAC. Old type works
> of course, it is defined also in order to avoid API breakage.
>
> Most of the V4L implementation stuff is pretty trivial, ~copy & paste
> from the SDR receiver.
>
> New capability flag V4L2_CAP_SDR_OUTPUT is added to indicate device
> is SDR transmitter.
>
> Old capability flag V4L2_CAP_MODULATOR is used to indicate there is
> 'tuner' to set radio frequency for transmitter. That capability flag
> name is pretty misleading in a case of SDR as SDR does not have
> hardware modulator at all - but as it is existing flag it is hard to
> change anymore (V4L API uses term TUNER for radio receiver and
> MODULATOR for radio transmitter).
>
> New v4l2 buffer type V4L2_BUF_TYPE_SDR_OUTPUT.
>
> Transmitter format is negotiated similarly than receiver.
>
> regards
> Antti
>
>
> Antti Palosaari (9):
> v4l2: rename V4L2_TUNER_ADC to V4L2_TUNER_SDR
> v4l2: add RF gain control
> DocBook: document tuner RF gain control
> v4l2: add support for SDR transmitter
> DocBook: document SDR transmitter
> hackrf: add control for RF amplifier
> hackrf: switch to single function which configures everything
> hackrf: add support for transmitter
> hackrf: do not set human readable name for formats
>
> Documentation/DocBook/media/v4l/compat.xml | 20 +
> Documentation/DocBook/media/v4l/controls.xml | 19 +
> Documentation/DocBook/media/v4l/dev-sdr.xml | 32 +-
> Documentation/DocBook/media/v4l/io.xml | 10 +-
> Documentation/DocBook/media/v4l/pixfmt.xml | 2 +-
> Documentation/DocBook/media/v4l/v4l2.xml | 9 +
> Documentation/DocBook/media/v4l/vidioc-g-fmt.xml | 2 +-
> .../DocBook/media/v4l/vidioc-querycap.xml | 6 +
> drivers/media/usb/hackrf/hackrf.c | 1019 ++++++++++++++------
> drivers/media/v4l2-core/v4l2-ctrls.c | 4 +
> drivers/media/v4l2-core/v4l2-dev.c | 14 +-
> drivers/media/v4l2-core/v4l2-ioctl.c | 31 +-
> drivers/media/v4l2-core/videobuf-core.c | 4 +-
> include/media/v4l2-ioctl.h | 8 +
> include/trace/events/v4l2.h | 1 +
> include/uapi/linux/v4l2-controls.h | 2 +
> include/uapi/linux/videodev2.h | 10 +-
> 17 files changed, 898 insertions(+), 295 deletions(-)
>
next prev parent reply other threads:[~2015-06-05 9:50 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-26 17:08 [ATTN 0/9] SDR transmitter API Antti Palosaari
2015-05-26 17:08 ` [ATTN 1/9] v4l2: rename V4L2_TUNER_ADC to V4L2_TUNER_SDR Antti Palosaari
2015-05-26 17:08 ` [ATTN 2/9] v4l2: add RF gain control Antti Palosaari
2015-05-26 17:08 ` [ATTN 3/9] DocBook: document tuner " Antti Palosaari
2015-05-26 17:08 ` [ATTN 4/9] v4l2: add support for SDR transmitter Antti Palosaari
2015-05-26 17:08 ` [ATTN 5/9] DocBook: document " Antti Palosaari
2015-05-26 17:08 ` [ATTN 6/9] hackrf: add control for RF amplifier Antti Palosaari
2015-05-26 17:08 ` [ATTN 7/9] hackrf: switch to single function which configures everything Antti Palosaari
2015-05-26 17:08 ` [ATTN 8/9] hackrf: add support for transmitter Antti Palosaari
2015-05-26 17:08 ` [ATTN 9/9] hackrf: do not set human readable name for formats Antti Palosaari
2015-06-05 9:50 ` Hans Verkuil [this message]
2015-06-05 10:25 ` [ATTN 0/9] SDR transmitter API Antti Palosaari
2015-06-05 10:30 ` 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=557170D9.3010607@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=crope@iki.fi \
--cc=linux-media@vger.kernel.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