Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Devin Heitmueller <devin.heitmueller@ltnglobal.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] Request for assistance with adding new pixel format (NV12_8L128) in FFmpeg
Date: Wed, 8 Feb 2023 13:11:38 -0500
Message-ID: <CAHGibzHeo726Y_zkwswFcQyW8CMFOX4JMjb95bC2HjtPCnkDZA@mail.gmail.com> (raw)
In-Reply-To: <3818751.3MK9gZLdrh@basile.remlab.net>

Hi Remi,

On Wed, Feb 8, 2023 at 12:38 PM Rémi Denis-Courmont <remi@remlab.net> wrote:
> Uh, not to deny that, but tiled formats are in no way an embedded thing, as
> you imply. They are also used by desktop GPUs. And that is not really
> relevant, I would argue, to the discussion, either way.

I don't disagree that tiled formats can be found on desktop GPUs.  My
only point was that the importance of being able to pass through the
native format is much greater with embedded SOCs since in many cases
on desktops you have enough CPU power to get away with doing a
conversion.

Also, you're more likely to find that desktop GPUs which support tiled
formats also support formats that are likely to be supported by ffmpeg
today.  This is less likely to be the case with embedded SOCs.

> > Of course ffmpeg could choose
> > to ignore them, but it would effectively prevent it from being used on
> > those platforms (and it's pretty essential to use the hardware blocks
> > to do any real video processing).
>
> Insofar as the format is *only* used by a single module, I don't see the need
> to assign it a pixel format in libavutil. You could just as well define a
> generic V4L format that would be used for all weird V4L formats that nothing
> other than V4L code understands.

In many cases the lack of support for a pixel format may be the reason
that more platforms aren't supported.  Somebody has got to be the
first.  Suggesting that it's only used by a single module may be the
effect of not supporting the pixel format, not the other way around.

This is not to suggest that like most developers I don't hate the fact
that there are so many different pixel formats out there, and with
each new format proposed for addition to ffmpeg we have the same
arguments.

> Ditto tiled DRM pixel formats for that matter. And while V4L is maybe not
> there yet, DRM has introduced format modifiers that anyway preclude any simple
> enumeration. My point being, there needs to be a way to further specify
> framework-specific formats outside of the avutil pixel format list, in any
> case.

I can't really offer an opinion on the implementation detail regarding
the avutil pixel format list.  I suspect the OP is open to alternative
approaches as long as the original requirement is met.

Devin

-- 
Devin Heitmueller, Senior Software Engineer
LTN Global Communications
o: +1 (301) 363-1001
w: https://ltnglobal.com  e: devin.heitmueller@ltnglobal.com
_______________________________________________
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-02-08 18:12 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-06 15:43 Le Bao Tin Ha
2023-02-06 18:15 ` Kieran Kunhya
2023-02-08  9:47   ` Le Bao Tin Ha
2023-02-08 13:07     ` Jean-Baptiste Kempf
2023-02-08 13:17       ` Hendrik Leppkes
2023-02-08 14:27       ` Devin Heitmueller
2023-02-08 17:38         ` Rémi Denis-Courmont
2023-02-08 18:11           ` Devin Heitmueller [this message]
2023-02-09  0:00           ` Philip Langdale

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=CAHGibzHeo726Y_zkwswFcQyW8CMFOX4JMjb95bC2HjtPCnkDZA@mail.gmail.com \
    --to=devin.heitmueller@ltnglobal.com \
    --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