Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Lynne <dev@lynne.ee>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH 5/5] avformat: add sdr support
Date: Sun, 18 Jun 2023 15:45:37 +0200 (CEST)
Message-ID: <NYDkFe7--3-9@lynne.ee> (raw)
In-Reply-To: <CA+anqdw8R05ghTC5+iHoS-H3sjxz9aG-7=Wv4Z8NMxe4Fom_=Q@mail.gmail.com>

Jun 18, 2023, 15:30 by h.leppkes@gmail.com:

> On Sun, Jun 18, 2023 at 2:59 PM Lynne <dev@lynne.ee> wrote:
>
>>
>> Jun 17, 2023, 20:37 by michael@niedermayer.cc:
>>
>> > On Sat, Jun 17, 2023 at 08:08:11PM +0200, Paul B Mahol wrote:
>> > [...]
>> >
>> >> Which library handles compressed stuff?
>> >>
>> >
>> > For digital stuff like DAB/DVB/...
>> >
>> > sdrdemux in libavformat will demodulate, do error correction then return  AVPacket
>> > with H.264 / Mpeg2 video or AAC or wahetever
>> > That then will get passed to libavcodec, same as with any other demuxer.
>> >
>> > This code is not written yet. There is just analog AM demodulation. I posted
>> > this patch as soon as the first usecase worked
>> >
>> > usecase was "listen to random AM radio stations in ffplay" :)
>> > and it can also grab all AM stations in a 10Mhz window and demodulate all
>> > (that 10mhz is what my hw and usb bandwidth can do max)
>> >
>> > I intend to add FM demodulation, because it seems a fun thing to try to do.
>> >
>> > someone will need to add DAB/DVB support, I would be happy if someone else
>> > comes forth to do that. If noone does ill think about if i want to do it
>> > or not once FM is done and once this is in git.
>> >
>> > The initial reaction from some was a bit spicy, so i need to think
>> > how much fun this is in FFmpeg.
>> > But one thing i can say for sure, is, if i cannot use this in ffplay
>> > then it makes no sense for me to put more time into it
>> >
>> > We could implement seeking to next/previous stations differently but
>> > it seems a bit like creating extra work for everyone.
>> > I mean a new API in sdrdemux then ffplay needs to support that and then
>> > other players would need to support it and so on. While really the
>> > seeking on a realtime input has no other use
>> >
>>
>> I like the new functionality, but I too think that libavformat
>> should just source the data from a device, and that libavfilter
>> should demodulate.
>>
>
> And how do you deal with the prospect of future digital audio,
> outputting a compressed bitstream?
>

I don't think anyone has thought that far ahead yet. The goal of the
patch is to decode pure analog audio.

If I may theorize for a moment:
AVFrame.format = AV_SAMPLE_FMT_COMPRESSED | AV_CODEC_ID_AAC,
swr would return AVERROR(ENOSUP).
an encoder would do the decoding to a wrapped_avframe?
a new lavc api to decode avframes?
There was some work to unify avpackets into avframes in the libav days AFAIK.
Nothing good at all, pure guesswork.

Before someone starts bikeshedding, let me point out that
having lavfi demodulate does make the current patch better IMO
and that any discussion on compressed frames should be handled
*in a different thread*.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

  reply	other threads:[~2023-06-18 13:45 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-16 22:20 [FFmpeg-devel] [PATCH 0/5] " Michael Niedermayer
2023-06-16 22:20 ` [FFmpeg-devel] [PATCH 1/5] avutil/tx_template: extend to 2M Michael Niedermayer
2023-07-22 15:40   ` Michael Niedermayer
2023-07-23 17:48     ` Tomas Härdin
2023-07-24  8:59       ` Michael Niedermayer
2023-07-24 11:06         ` Leo Izen
2023-07-24 12:39           ` Thilo Borgmann
2023-07-24 13:04             ` Rémi Denis-Courmont
2023-07-24 15:45               ` Thilo Borgmann
2023-07-24  8:35     ` Andreas Rheinhardt
2023-07-24  8:57       ` Paul B Mahol
2023-07-24 18:46         ` Tomas Härdin
2023-07-25  6:14           ` Paul B Mahol
2023-06-16 22:20 ` [FFmpeg-devel] [PATCH 2/5] avcodec/pcm: allow Changing parameters Michael Niedermayer
2023-06-16 22:20 ` [FFmpeg-devel] [PATCH 3/5] avcodec/kbdwin: Support arbitrary sized windows Michael Niedermayer
2023-06-16 22:20 ` [FFmpeg-devel] [PATCH 4/5] avcodec: Rename ff_kbd_window_init() as it will be needed from outside libavcodec Michael Niedermayer
2023-06-16 22:20 ` [FFmpeg-devel] [PATCH 5/5] avformat: add sdr support Michael Niedermayer
2023-06-17  6:16   ` Paul B Mahol
2023-06-17  8:46     ` Michael Niedermayer
2023-06-17 11:08       ` Paul B Mahol
2023-06-17 12:48         ` Michael Niedermayer
2023-06-17 18:08           ` Paul B Mahol
2023-06-17 18:37             ` Michael Niedermayer
2023-06-17 18:43               ` Michael Niedermayer
2023-06-18 11:46               ` Paul B Mahol
2023-06-18 12:36                 ` Kieran Kunhya
2023-06-24 10:22                   ` Tomas Härdin
2023-06-18 12:59               ` Lynne
2023-06-18 13:30                 ` Hendrik Leppkes
2023-06-18 13:45                   ` Lynne [this message]
2023-06-18 13:48                 ` Michael Niedermayer
2023-06-17 13:30   ` Leo Izen
2023-06-17 18:10     ` Michael Niedermayer

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=NYDkFe7--3-9@lynne.ee \
    --to=dev@lynne.ee \
    --cc=ffmpeg-devel@ffmpeg.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

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git