Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 11:52:01 +0100 Alexis Ballier wrote: > > No, this would imply that the pointer to the opaque data is in > > AVFrame.data[3], and all other pointers are ignored. So you have > > only 1 pointer. AVFrame.linesize has no meaning either in this case. > > this won't work with NV12(T

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 11:23:49 +0100 wm4 wrote: > > I didn't know such "opaque" things existed; it probably makes more > > sense with it indeed. > > > > How should I do it ? add .flags = AV_PIX_FMT_FLAG_HWACCEL and be > > done ? I would like to keep 'av_pix_fmt_count_planes()' working for > > it at

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 10:50:42 +0100 Jean-Baptiste Kempf wrote: > > Now, if I want to use the fimc device to apply some effects I can't > > because the decoder is magically using it behind my back. > > How is that relevant? Just block it. Block what ? automatic conversion ? what fmt should i adver

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread wm4
On Fri, 21 Nov 2014 11:15:08 +0100 Alexis Ballier wrote: > On Fri, 21 Nov 2014 10:51:52 +0100 > wm4 wrote: > > > On Fri, 21 Nov 2014 10:43:15 +0100 > > Alexis Ballier wrote: > > > > > On Fri, 21 Nov 2014 10:15:29 +0100 > > > Jean-Baptiste Kempf wrote: > > > > > > > On 21 Nov, Alexis Ballier

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 10:51:52 +0100 wm4 wrote: > On Fri, 21 Nov 2014 10:43:15 +0100 > Alexis Ballier wrote: > > > On Fri, 21 Nov 2014 10:15:29 +0100 > > Jean-Baptiste Kempf wrote: > > > > > On 21 Nov, Alexis Ballier wrote : > > > > > So you have a format outputted that is of no use, except if

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread wm4
On Fri, 21 Nov 2014 10:43:15 +0100 Alexis Ballier wrote: > On Fri, 21 Nov 2014 10:15:29 +0100 > Jean-Baptiste Kempf wrote: > > > On 21 Nov, Alexis Ballier wrote : > > > > So you have a format outputted that is of no use, except if you > > > > use the filter. > > > > > > On MFCv5 yes; I don't a

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Jean-Baptiste Kempf
On 21 Nov, Alexis Ballier wrote : > On Fri, 21 Nov 2014 10:38:37 +0100 > Jean-Baptiste Kempf wrote: > > > On 21 Nov, Alexis Ballier wrote : > > > I think it's safe to assume that if you have a SOC with a HW dec > > > that outputs only this insane and uncommon format, the SOC also has > > > a HW f

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Jean-Baptiste Kempf
On 21 Nov, Alexis Ballier wrote : > On Fri, 21 Nov 2014 10:15:29 +0100 > Jean-Baptiste Kempf wrote: > > > On 21 Nov, Alexis Ballier wrote : > > > > So you have a format outputted that is of no use, except if you > > > > use the filter. > > > > > > On MFCv5 yes; I don't assume because I'm working

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 10:38:37 +0100 Jean-Baptiste Kempf wrote: > On 21 Nov, Alexis Ballier wrote : > > I think it's safe to assume that if you have a SOC with a HW dec > > that outputs only this insane and uncommon format, the SOC also has > > a HW filter that can convert it. > > Then use this fi

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 10:15:29 +0100 Jean-Baptiste Kempf wrote: > On 21 Nov, Alexis Ballier wrote : > > > So you have a format outputted that is of no use, except if you > > > use the filter. > > > > On MFCv5 yes; I don't assume because I'm working on this that it is > > the only one :) > > So, b

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Benoit Fouet
Hi, - Mail original - [...] > I think it's safe to assume that if you have a SOC with a HW dec that > outputs only this insane and uncommon format, the SOC also has a HW > filter that can convert it. This is a good argument IMHO. On a side note, I don't think it would be that complicate

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Jean-Baptiste Kempf
On 21 Nov, Alexis Ballier wrote : > I think it's safe to assume that if you have a SOC with a HW dec that > outputs only this insane and uncommon format, the SOC also has a HW > filter that can convert it. Then use this filter to output a standard format. > > Sorry, but that's far from clear. >

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 10:02:36 +0100 Jean-Baptiste Kempf wrote: > On 21 Nov, Alexis Ballier wrote : > > > > How do you put this in an AVFrame then ? > > > > > > Preferably not at all. Is there a need to? Only the end result > > > (nv12 I suppose) needs to be in an AVFrame. > > > > It can be done

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Jean-Baptiste Kempf
On 21 Nov, Alexis Ballier wrote : > > So you have a format outputted that is of no use, except if you use > > the filter. > > On MFCv5 yes; I don't assume because I'm working on this that it is the > only one :) So, basically, noone can display it, reencode or do anything with it, without the fil

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 09:58:40 +0100 Jean-Baptiste Kempf wrote: > On 21 Nov, Alexis Ballier wrote : > > On Thu, 20 Nov 2014 18:40:18 +0100 > > Jean-Baptiste Kempf wrote: > > > > > On 20 Nov, Alexis Ballier wrote : > > > > This is the only format supported by MFC5 HW decoders (e.g. > > > > Samsung

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Jean-Baptiste Kempf
On 21 Nov, Alexis Ballier wrote : > > > How do you put this in an AVFrame then ? > > > > Preferably not at all. Is there a need to? Only the end result (nv12 I > > suppose) needs to be in an AVFrame. > > It can be done but is ugly IMHO: > > - I was under the impression that lavc decoder conventi

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Jean-Baptiste Kempf
On 21 Nov, Alexis Ballier wrote : > On Thu, 20 Nov 2014 18:40:18 +0100 > Jean-Baptiste Kempf wrote: > > > On 20 Nov, Alexis Ballier wrote : > > > This is the only format supported by MFC5 HW decoders (e.g. Samsung > > > exynos 4412). > > > > Why not convert it to a normal format? > > > > That

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread Alexis Ballier
On Fri, 21 Nov 2014 09:01:43 +0100 wm4 wrote: > On Fri, 21 Nov 2014 08:45:47 +0100 > Alexis Ballier wrote: > > > On Thu, 20 Nov 2014 18:42:19 +0100 > > wm4 wrote: > > > I think this should be rejected. It's far too special to be > > > useful in general, and it's not even pixel- or line-address

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-21 Thread wm4
On Fri, 21 Nov 2014 08:45:47 +0100 Alexis Ballier wrote: > On Thu, 20 Nov 2014 18:42:19 +0100 > wm4 wrote: > > I think this should be rejected. It's far too special to be useful in > > general, and it's not even pixel- or line-addressable (Z-shaped tiles, > > seriously?). It's pretty much a raw

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-20 Thread Alexis Ballier
On Thu, 20 Nov 2014 18:40:18 +0100 Jean-Baptiste Kempf wrote: > On 20 Nov, Alexis Ballier wrote : > > This is the only format supported by MFC5 HW decoders (e.g. Samsung > > exynos 4412). > > Why not convert it to a normal format? > That is exactly what the m2m filter is for: on 4412 you have

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-20 Thread Alexis Ballier
On Thu, 20 Nov 2014 18:42:19 +0100 wm4 wrote: > I think this should be rejected. It's far too special to be useful in > general, and it's not even pixel- or line-addressable (Z-shaped tiles, > seriously?). It's pretty much a raw codec, not a pixel format. How do you put this in an AVFrame then ?

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-20 Thread wm4
On Thu, 20 Nov 2014 17:51:54 +0100 Alexis Ballier wrote: > This is the only format supported by MFC5 HW decoders (e.g. Samsung exynos > 4412). > --- > libavutil/pixdesc.c | 12 > libavutil/pixfmt.h | 1 + > 2 files changed, 13 insertions(+) > > diff --git a/libavutil/pixdesc.c b

Re: [FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-20 Thread Jean-Baptiste Kempf
On 20 Nov, Alexis Ballier wrote : > This is the only format supported by MFC5 HW decoders (e.g. Samsung exynos > 4412). Why not convert it to a normal format? With my kindest regards, -- Jean-Baptiste Kempf http://www.jbkempf.com/ - +33 672 704 734 Sent from my Electronic Device __

[FFmpeg-devel] [PATCH 3/6] Add AV_PIX_FMT_NV12T.

2014-11-20 Thread Alexis Ballier
This is the only format supported by MFC5 HW decoders (e.g. Samsung exynos 4412). --- libavutil/pixdesc.c | 12 libavutil/pixfmt.h | 1 + 2 files changed, 13 insertions(+) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index 648d014..426dfd4 100644 --- a/libavutil/pixdesc.