| Message ID | 20210611203104.1692-18-dcnieho@gmail.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | avdevice (mostly dshow) enhancements | expand |
| Context | Check | Description |
|---|---|---|
| andriy/x86_make | success | Make finished |
| andriy/x86_make_fate | success | Make fate finished |
| andriy/PPC64_make | success | Make finished |
| andriy/PPC64_make_fate | success | Make fate finished |
On Fri, Jun 11, 2021 at 10:30:48PM +0200, Diederick Niehorster wrote: > Enabled discovering a DirectShow device's color range, space, primaries, > transfer characteristics and chroma location, if the device exposes that > information. Sets them in the stream's codecpars. > > Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> > Co-authored-by: Valerii Zapodovnikov <val.zapod.vz@gmail.com> > Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> > --- > libavdevice/dshow.c | 250 +++++++++++++++++++++++++++++++++++++++++++- > 1 file changed, 249 insertions(+), 1 deletion(-) breaks build on mingw64 make CC libavdevice/dshow.o src/libavdevice/dshow.c: In function ‘dshow_get_device_media_types’: src/libavdevice/dshow.c:415:23: warning: unused variable ‘ctx’ [-Wunused-variable] struct dshow_ctx *ctx = avctx->priv_data; ^~~ src/libavdevice/dshow.c: In function ‘dshow_get_device_list’: src/libavdevice/dshow.c:662:23: warning: unused variable ‘ctx’ [-Wunused-variable] struct dshow_ctx *ctx = avctx->priv_data; ^~~ src/libavdevice/dshow.c: In function ‘dshow_cycle_formats’: src/libavdevice/dshow.c:731:13: error: unknown type name ‘DXVA_ExtendedFormat’; did you mean ‘DXVA2_ExtendedFormat’? DXVA_ExtendedFormat *extended_format_info = NULL; ^~~~~~~~~~~~~~~~~~~ DXVA2_ExtendedFormat src/libavdevice/dshow.c:744:41: error: ‘AMCONTROL_COLORINFO_PRESENT’ undeclared (first use in this function) if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) ^~~~~~~~~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:744:41: note: each undeclared identifier is reported only once for each function it appears in src/libavdevice/dshow.c:745:45: error: ‘DXVA_ExtendedFormat’ undeclared (first use in this function); did you mean ‘DXVA2_ExtendedFormat’? extended_format_info = (DXVA_ExtendedFormat*)&v->dwControlFlags; ^~~~~~~~~~~~~~~~~~~ DXVA2_ExtendedFormat src/libavdevice/dshow.c:745:65: error: expected expression before ‘)’ token extended_format_info = (DXVA_ExtendedFormat*)&v->dwControlFlags; ^ src/libavdevice/dshow.c:768:69: warning: passing argument 1 of ‘dshow_color_range’ from incompatible pointer type [-Wincompatible-pointer-types] enum AVColorRange col_range = dshow_color_range(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:77:26: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorRange dshow_color_range(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:769:69: warning: passing argument 1 of ‘dshow_color_space’ from incompatible pointer type [-Wincompatible-pointer-types] enum AVColorSpace col_space = dshow_color_space(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:101:26: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorSpace dshow_color_space(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:770:76: warning: passing argument 1 of ‘dshow_color_primaries’ from incompatible pointer type [-Wincompatible-pointer-types] enum AVColorPrimaries col_prim = dshow_color_primaries(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:156:30: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorPrimaries dshow_color_primaries(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:771:82: warning: passing argument 1 of ‘dshow_color_trc’ from incompatible pointer type [-Wincompatible-pointer-types] enum AVColorTransferCharacteristic col_trc = dshow_color_trc(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:193:43: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorTransferCharacteristic dshow_color_trc(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~ src/libavdevice/dshow.c:772:73: warning: passing argument 1 of ‘dshow_chroma_loc’ from incompatible pointer type [-Wincompatible-pointer-types] enum AVChromaLocation chroma_loc = dshow_chroma_loc(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:248:30: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVChromaLocation dshow_chroma_loc(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~ src/libavdevice/dshow.c: In function ‘dshow_add_device’: src/libavdevice/dshow.c:1474:9: error: unknown type name ‘DXVA_ExtendedFormat’; did you mean ‘DXVA2_ExtendedFormat’? DXVA_ExtendedFormat* extended_format_info = NULL; ^~~~~~~~~~~~~~~~~~~ DXVA2_ExtendedFormat src/libavdevice/dshow.c:1484:37: error: ‘AMCONTROL_COLORINFO_PRESENT’ undeclared (first use in this function) if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) ^~~~~~~~~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:1485:41: error: ‘DXVA_ExtendedFormat’ undeclared (first use in this function); did you mean ‘DXVA2_ExtendedFormat’? extended_format_info = (DXVA_ExtendedFormat*)&v->dwControlFlags; ^~~~~~~~~~~~~~~~~~~ DXVA2_ExtendedFormat src/libavdevice/dshow.c:1485:61: error: expected expression before ‘)’ token extended_format_info = (DXVA_ExtendedFormat*)&v->dwControlFlags; ^ src/libavdevice/dshow.c:1505:50: warning: passing argument 1 of ‘dshow_color_range’ from incompatible pointer type [-Wincompatible-pointer-types] par->color_range = dshow_color_range(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:77:26: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorRange dshow_color_range(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:1506:50: warning: passing argument 1 of ‘dshow_color_space’ from incompatible pointer type [-Wincompatible-pointer-types] par->color_space = dshow_color_space(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:101:26: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorSpace dshow_color_space(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:1507:58: warning: passing argument 1 of ‘dshow_color_primaries’ from incompatible pointer type [-Wincompatible-pointer-types] par->color_primaries = dshow_color_primaries(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:156:30: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorPrimaries dshow_color_primaries(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:1508:46: warning: passing argument 1 of ‘dshow_color_trc’ from incompatible pointer type [-Wincompatible-pointer-types] par->color_trc = dshow_color_trc(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:193:43: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVColorTransferCharacteristic dshow_color_trc(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~ src/libavdevice/dshow.c:1509:53: warning: passing argument 1 of ‘dshow_chroma_loc’ from incompatible pointer type [-Wincompatible-pointer-types] par->chroma_location = dshow_chroma_loc(extended_format_info); ^~~~~~~~~~~~~~~~~~~~ src/libavdevice/dshow.c:248:30: note: expected ‘DXVA2_ExtendedFormat * {aka struct _DXVA2_ExtendedFormat *}’ but argument is of type ‘int *’ static enum AVChromaLocation dshow_chroma_loc(DXVA2_ExtendedFormat* fmt_info) ^~~~~~~~~~~~~~~~ src/ffbuild/common.mak:67: recipe for target 'libavdevice/dshow.o' failed make: *** [libavdevice/dshow.o] Error 1 [...]
On 6/14/2021 1:56 PM, Michael Niedermayer wrote: > On Fri, Jun 11, 2021 at 10:30:48PM +0200, Diederick Niehorster wrote: >> Enabled discovering a DirectShow device's color range, space, primaries, >> transfer characteristics and chroma location, if the device exposes that >> information. Sets them in the stream's codecpars. >> >> Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> >> Co-authored-by: Valerii Zapodovnikov <val.zapod.vz@gmail.com> >> Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> >> --- >> libavdevice/dshow.c | 250 +++++++++++++++++++++++++++++++++++++++++++- >> 1 file changed, 249 insertions(+), 1 deletion(-) > > breaks build on mingw64 > > make > CC libavdevice/dshow.o > src/libavdevice/dshow.c: In function ‘dshow_get_device_media_types’: > src/libavdevice/dshow.c:415:23: warning: unused variable ‘ctx’ [-Wunused-variable] > struct dshow_ctx *ctx = avctx->priv_data; > ^~~ > src/libavdevice/dshow.c: In function ‘dshow_get_device_list’: > src/libavdevice/dshow.c:662:23: warning: unused variable ‘ctx’ [-Wunused-variable] > struct dshow_ctx *ctx = avctx->priv_data; > ^~~ > src/libavdevice/dshow.c: In function ‘dshow_cycle_formats’: > src/libavdevice/dshow.c:731:13: error: unknown type name ‘DXVA_ExtendedFormat’; did you mean ‘DXVA2_ExtendedFormat’? > DXVA_ExtendedFormat *extended_format_info = NULL; > ^~~~~~~~~~~~~~~~~~~ > DXVA2_ExtendedFormat > src/libavdevice/dshow.c:744:41: error: ‘AMCONTROL_COLORINFO_PRESENT’ undeclared (first use in this function) > if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) > ^~~~~~~~~~~~~~~~~~~~~~~~~~~ $ grep -r AMCONTROL_COLORINFO_PRESENT /mingw64/x86_64-w64-mingw32/include/* /mingw64/x86_64-w64-mingw32/include/dvdmedia.h:#define AMCONTROL_COLORINFO_PRESENT 0x00000080 Can you see if you can update your mingw-w64 toolchain, or at least the headers/crt? This is not the first time you get an error with new code. Old mingw headers releases lacked a lot of API and are not representative of what a sane toolchain is meant to have.
On Mon, Jun 14, 2021 at 02:50:32PM -0300, James Almer wrote: > On 6/14/2021 1:56 PM, Michael Niedermayer wrote: > > On Fri, Jun 11, 2021 at 10:30:48PM +0200, Diederick Niehorster wrote: > > > Enabled discovering a DirectShow device's color range, space, primaries, > > > transfer characteristics and chroma location, if the device exposes that > > > information. Sets them in the stream's codecpars. > > > > > > Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> > > > Co-authored-by: Valerii Zapodovnikov <val.zapod.vz@gmail.com> > > > Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> > > > --- > > > libavdevice/dshow.c | 250 +++++++++++++++++++++++++++++++++++++++++++- > > > 1 file changed, 249 insertions(+), 1 deletion(-) > > > > breaks build on mingw64 > > > > make > > CC libavdevice/dshow.o > > src/libavdevice/dshow.c: In function ‘dshow_get_device_media_types’: > > src/libavdevice/dshow.c:415:23: warning: unused variable ‘ctx’ [-Wunused-variable] > > struct dshow_ctx *ctx = avctx->priv_data; > > ^~~ > > src/libavdevice/dshow.c: In function ‘dshow_get_device_list’: > > src/libavdevice/dshow.c:662:23: warning: unused variable ‘ctx’ [-Wunused-variable] > > struct dshow_ctx *ctx = avctx->priv_data; > > ^~~ > > src/libavdevice/dshow.c: In function ‘dshow_cycle_formats’: > > src/libavdevice/dshow.c:731:13: error: unknown type name ‘DXVA_ExtendedFormat’; did you mean ‘DXVA2_ExtendedFormat’? > > DXVA_ExtendedFormat *extended_format_info = NULL; > > ^~~~~~~~~~~~~~~~~~~ > > DXVA2_ExtendedFormat > > src/libavdevice/dshow.c:744:41: error: ‘AMCONTROL_COLORINFO_PRESENT’ undeclared (first use in this function) > > if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~ > > $ grep -r AMCONTROL_COLORINFO_PRESENT /mingw64/x86_64-w64-mingw32/include/* > /mingw64/x86_64-w64-mingw32/include/dvdmedia.h:#define > AMCONTROL_COLORINFO_PRESENT 0x00000080 > > Can you see if you can update your mingw-w64 toolchain, or at least the > headers/crt? This is not the first time you get an error with new code. > Old mingw headers releases lacked a lot of API and are not representative of > what a sane toolchain is meant to have. Assuming that ubuntu didnt fail to update its mingw installation on OS update This is a standard installation for mingw64 from 18.04 LTS packages I think you are missing my point here as you are asking me to upgrade. What would i achieve by upgrading ? Isnt that equivalent of not testing build with ubuntu 18 LTS anymore ? I think testing does make sense unless you belive theres something funky wrong with my setup? And if you want to drop support for some specific versions of platforms, question would be which platforms would that affect and it would require a test in configure not failure to build in a random source file. The later can be a pain for novice users. Thanks [...]
On Tue, Jun 15, 2021 at 10:18 AM Michael Niedermayer <michael@niedermayer.cc> wrote: > > On Mon, Jun 14, 2021 at 02:50:32PM -0300, James Almer wrote: > > On 6/14/2021 1:56 PM, Michael Niedermayer wrote: > > > On Fri, Jun 11, 2021 at 10:30:48PM +0200, Diederick Niehorster wrote: > > > > Enabled discovering a DirectShow device's color range, space, primaries, > > > > transfer characteristics and chroma location, if the device exposes that > > > > information. Sets them in the stream's codecpars. > > > > > > > > Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> > > > > Co-authored-by: Valerii Zapodovnikov <val.zapod.vz@gmail.com> > > > > Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> > > > > --- > > > > libavdevice/dshow.c | 250 +++++++++++++++++++++++++++++++++++++++++++- > > > > 1 file changed, 249 insertions(+), 1 deletion(-) > > > > > > breaks build on mingw64 > > > > > > make > > > CC libavdevice/dshow.o > > > src/libavdevice/dshow.c: In function ‘dshow_get_device_media_types’: > > > src/libavdevice/dshow.c:415:23: warning: unused variable ‘ctx’ [-Wunused-variable] > > > struct dshow_ctx *ctx = avctx->priv_data; > > > ^~~ > > > src/libavdevice/dshow.c: In function ‘dshow_get_device_list’: > > > src/libavdevice/dshow.c:662:23: warning: unused variable ‘ctx’ [-Wunused-variable] > > > struct dshow_ctx *ctx = avctx->priv_data; > > > ^~~ > > > src/libavdevice/dshow.c: In function ‘dshow_cycle_formats’: > > > src/libavdevice/dshow.c:731:13: error: unknown type name ‘DXVA_ExtendedFormat’; did you mean ‘DXVA2_ExtendedFormat’? > > > DXVA_ExtendedFormat *extended_format_info = NULL; > > > ^~~~~~~~~~~~~~~~~~~ > > > DXVA2_ExtendedFormat > > > src/libavdevice/dshow.c:744:41: error: ‘AMCONTROL_COLORINFO_PRESENT’ undeclared (first use in this function) > > > if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) > > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > > $ grep -r AMCONTROL_COLORINFO_PRESENT /mingw64/x86_64-w64-mingw32/include/* > > /mingw64/x86_64-w64-mingw32/include/dvdmedia.h:#define > > AMCONTROL_COLORINFO_PRESENT 0x00000080 > > > > Can you see if you can update your mingw-w64 toolchain, or at least the > > headers/crt? This is not the first time you get an error with new code. > > Old mingw headers releases lacked a lot of API and are not representative of > > what a sane toolchain is meant to have. > > Assuming that ubuntu didnt fail to update its mingw installation on OS update > This is a standard installation for mingw64 from 18.04 LTS packages > > I think you are missing my point here as you are asking me to upgrade. > What would i achieve by upgrading ? > Isnt that equivalent of not testing build with ubuntu 18 LTS anymore ? > > I think testing does make sense unless you belive theres something funky wrong > with my setup? > And if you want to drop support for some specific versions of platforms, question > would be which platforms would that affect and it would require a test in > configure not failure to build in a random source file. The later can be a pain > for novice users. Looks like i missed some DXVA_... ones, and i didn't know AMCONTROL_COLORINFO_PRESENT was also an issue. I will try to set up a mingw (instead of msvc that i am using) build environment. But i'll fix this in any case for the next version of patches (will be later, in waiting for reviews of other bits). On patchworks everything built correctly by the way, is that not using MinGW? All the best, Dee
On 6/15/2021 5:18 AM, Michael Niedermayer wrote: > On Mon, Jun 14, 2021 at 02:50:32PM -0300, James Almer wrote: >> On 6/14/2021 1:56 PM, Michael Niedermayer wrote: >>> On Fri, Jun 11, 2021 at 10:30:48PM +0200, Diederick Niehorster wrote: >>>> Enabled discovering a DirectShow device's color range, space, primaries, >>>> transfer characteristics and chroma location, if the device exposes that >>>> information. Sets them in the stream's codecpars. >>>> >>>> Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> >>>> Co-authored-by: Valerii Zapodovnikov <val.zapod.vz@gmail.com> >>>> Signed-off-by: Diederick Niehorster <dcnieho@gmail.com> >>>> --- >>>> libavdevice/dshow.c | 250 +++++++++++++++++++++++++++++++++++++++++++- >>>> 1 file changed, 249 insertions(+), 1 deletion(-) >>> >>> breaks build on mingw64 >>> >>> make >>> CC libavdevice/dshow.o >>> src/libavdevice/dshow.c: In function ‘dshow_get_device_media_types’: >>> src/libavdevice/dshow.c:415:23: warning: unused variable ‘ctx’ [-Wunused-variable] >>> struct dshow_ctx *ctx = avctx->priv_data; >>> ^~~ >>> src/libavdevice/dshow.c: In function ‘dshow_get_device_list’: >>> src/libavdevice/dshow.c:662:23: warning: unused variable ‘ctx’ [-Wunused-variable] >>> struct dshow_ctx *ctx = avctx->priv_data; >>> ^~~ >>> src/libavdevice/dshow.c: In function ‘dshow_cycle_formats’: >>> src/libavdevice/dshow.c:731:13: error: unknown type name ‘DXVA_ExtendedFormat’; did you mean ‘DXVA2_ExtendedFormat’? >>> DXVA_ExtendedFormat *extended_format_info = NULL; >>> ^~~~~~~~~~~~~~~~~~~ >>> DXVA2_ExtendedFormat >>> src/libavdevice/dshow.c:744:41: error: ‘AMCONTROL_COLORINFO_PRESENT’ undeclared (first use in this function) >>> if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) >>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~ >> >> $ grep -r AMCONTROL_COLORINFO_PRESENT /mingw64/x86_64-w64-mingw32/include/* >> /mingw64/x86_64-w64-mingw32/include/dvdmedia.h:#define >> AMCONTROL_COLORINFO_PRESENT 0x00000080 >> >> Can you see if you can update your mingw-w64 toolchain, or at least the >> headers/crt? This is not the first time you get an error with new code. >> Old mingw headers releases lacked a lot of API and are not representative of >> what a sane toolchain is meant to have. > > Assuming that ubuntu didnt fail to update its mingw installation on OS update > This is a standard installation for mingw64 from 18.04 LTS packages That is unfortunate, but i guess not unexpected since mingw-w64 evolves pretty fast and headers get new API with every release. > > I think you are missing my point here as you are asking me to upgrade. > What would i achieve by upgrading ? Getting a recent enough release of the headers + CRT. Old releases lacked a lot of API, even stuff available in msvc for years. Even nightly is way ahead of tagged releases. > Isnt that equivalent of not testing build with ubuntu 18 LTS anymore ? Yes, but i was not aware you were using a distro package. > > I think testing does make sense unless you belive theres something funky wrong > with my setup? No, it's just three years old by now, but that's how LTS distros go, so... > And if you want to drop support for some specific versions of platforms, question > would be which platforms would that affect and it would require a test in > configure not failure to build in a random source file. The later can be a pain > for novice users. I guess a configure option to look for the API used by this patch would be needed, making this new code optional. > > Thanks > > [...] > > > _______________________________________________ > 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". >
On Wed, Jun 16, 2021 at 12:23 AM James Almer <jamrial@gmail.com> wrote: > > I guess a configure option to look for the API used by this patch would > be needed, making this new code optional. No need luckily, i have a workaround for all but the AMCONTROL_COLORINFO_PRESENT symbol. I'll define that one if not already defined. Cheers, Dee
diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c index add46e82ac..b3fcd9444e 100644 --- a/libavdevice/dshow.c +++ b/libavdevice/dshow.c @@ -29,6 +29,26 @@ #include "libavcodec/raw.h" #include "objidl.h" #include "shlwapi.h" +// NB: technically, we should include dxva.h and use +// DXVA_ExtendedFormat, but that type is not defined in +// the MinGW headers. The DXVA2_ExtendedFormat and the +// contents of its fields is identical to +// DXVA_ExtendedFormat (see https://docs.microsoft.com/en-us/windows/win32/medfound/extended-color-information#color-space-in-media-types) +// and is provided by MinGW as well, so we use that +// instead. NB also that per the Microsoft docs, the +// lowest 8 bits of the structure, i.e. the SampleFormat +// field, contain AMCONTROL_xxx flags instead of sample +// format information, and should thus not be used. +// NB further that various values in the structure's +// fields (e.g. BT.2020 color space) are not provided +// for either of the DXVA structs, but are provided in +// the flags of the corresponding fields of Media Foundation. +// These may be provided by DirectShow devices (e.g. LAVFilters +// does so). So we use those values here too (the equivalence is +// indicated by Microsoft example code: https://docs.microsoft.com/en-us/windows/win32/api/dxva2api/ns-dxva2api-dxva2_videodesc) +typedef DWORD D3DFORMAT; // dxva2api.h header needs these types defined before include apparently in WinSDK (not MinGW). +typedef DWORD D3DPOOL; +#include "dxva2api.h" static enum AVPixelFormat dshow_pixfmt(DWORD biCompression, WORD biBitCount) @@ -54,6 +74,192 @@ static enum AVPixelFormat dshow_pixfmt(DWORD biCompression, WORD biBitCount) return avpriv_find_pix_fmt(avpriv_get_raw_pix_fmt_tags(), biCompression); // all others } +static enum AVColorRange dshow_color_range(DXVA2_ExtendedFormat* fmt_info) +{ + switch (fmt_info->NominalRange) + { + case DXVA2_NominalRange_Unknown: + return AVCOL_RANGE_UNSPECIFIED; + case DXVA2_NominalRange_Normal: // equal to DXVA2_NominalRange_0_255 + return AVCOL_RANGE_JPEG; + case DXVA2_NominalRange_Wide: // equal to DXVA2_NominalRange_16_235 + return AVCOL_RANGE_MPEG; + case DXVA2_NominalRange_48_208: + // not an ffmpeg color range + return AVCOL_RANGE_UNSPECIFIED; + + // values from MediaFoundation SDK (mfobjects.h) + case 4: // MFNominalRange_64_127 + // not an ffmpeg color range + return AVCOL_RANGE_UNSPECIFIED; + + default: + return DXVA2_NominalRange_Unknown; + } +} + +static enum AVColorSpace dshow_color_space(DXVA2_ExtendedFormat* fmt_info) +{ + enum AVColorSpace ret = AVCOL_SPC_UNSPECIFIED; + + switch (fmt_info->VideoTransferMatrix) + { + case DXVA2_VideoTransferMatrix_BT709: + ret = AVCOL_SPC_BT709; + break; + case DXVA2_VideoTransferMatrix_BT601: + ret = AVCOL_SPC_BT470BG; + break; + case DXVA2_VideoTransferMatrix_SMPTE240M: + ret = AVCOL_SPC_SMPTE240M; + break; + + // values from MediaFoundation SDK (mfobjects.h) + case 4: // MFVideoTransferMatrix_BT2020_10 + case 5: // MFVideoTransferMatrix_BT2020_12 + if (fmt_info->VideoTransferFunction==12) // MFVideoTransFunc_2020_const + ret = AVCOL_SPC_BT2020_CL; + else + ret = AVCOL_SPC_BT2020_NCL; + break; + } + + if (ret == AVCOL_SPC_UNSPECIFIED) + { + // if color space not known from transfer matrix, + // fall back to using primaries to guess color space + switch (fmt_info->VideoPrimaries) + { + case DXVA2_VideoPrimaries_BT709: + ret = AVCOL_SPC_BT709; + break; + case DXVA2_VideoPrimaries_BT470_2_SysM: + ret = AVCOL_SPC_FCC; + break; + case DXVA2_VideoPrimaries_BT470_2_SysBG: + case DXVA2_VideoPrimaries_EBU3213: // this is PAL + ret = AVCOL_SPC_BT470BG; + break; + case DXVA2_VideoPrimaries_SMPTE170M: + case DXVA2_VideoPrimaries_SMPTE_C: + ret = AVCOL_SPC_SMPTE170M; + break; + case DXVA2_VideoPrimaries_SMPTE240M: + ret = AVCOL_SPC_SMPTE240M; + break; + } + } + + return ret; +} + +static enum AVColorPrimaries dshow_color_primaries(DXVA2_ExtendedFormat* fmt_info) +{ + switch (fmt_info->VideoPrimaries) + { + case DXVA2_VideoPrimaries_Unknown: + return AVCOL_PRI_UNSPECIFIED; + case DXVA2_VideoPrimaries_reserved: + return AVCOL_PRI_RESERVED; + case DXVA2_VideoPrimaries_BT709: + return AVCOL_PRI_BT709; + case DXVA2_VideoPrimaries_BT470_2_SysM: + return AVCOL_PRI_BT470M; + case DXVA2_VideoPrimaries_BT470_2_SysBG: + case DXVA2_VideoPrimaries_EBU3213: // this is PAL + return AVCOL_PRI_BT470BG; + case DXVA2_VideoPrimaries_SMPTE170M: + case DXVA2_VideoPrimaries_SMPTE_C: + return AVCOL_PRI_SMPTE170M; + case DXVA2_VideoPrimaries_SMPTE240M: + return AVCOL_PRI_SMPTE240M; + + // values from MediaFoundation SDK (mfobjects.h) + case 9: // MFVideoPrimaries_BT2020 + return AVCOL_PRI_BT2020; + case 10: // MFVideoPrimaries_XYZ + return AVCOL_PRI_SMPTE428; + case 11: // MFVideoPrimaries_DCI_P3 + return AVCOL_PRI_SMPTE431; + case 12: // MFVideoPrimaries_ACES (Academy Color Encoding System) + // not an FFmpeg color primary + return AVCOL_PRI_UNSPECIFIED; + + default: + return AVCOL_PRI_UNSPECIFIED; + } +} + +static enum AVColorTransferCharacteristic dshow_color_trc(DXVA2_ExtendedFormat* fmt_info) +{ + switch (fmt_info->VideoTransferFunction) + { + case DXVA2_VideoTransFunc_Unknown: + return AVCOL_TRC_UNSPECIFIED; + case DXVA2_VideoTransFunc_10: + return AVCOL_TRC_LINEAR; + case DXVA2_VideoTransFunc_18: + // not an FFmpeg transfer characteristic + return AVCOL_TRC_UNSPECIFIED; + case DXVA2_VideoTransFunc_20: + // not an FFmpeg transfer characteristic + return AVCOL_TRC_UNSPECIFIED; + case DXVA2_VideoTransFunc_22: + return AVCOL_TRC_GAMMA22; + case DXVA2_VideoTransFunc_709: + return AVCOL_TRC_BT709; + case DXVA2_VideoTransFunc_240M: + return AVCOL_TRC_SMPTE240M; + case DXVA2_VideoTransFunc_sRGB: + return AVCOL_TRC_IEC61966_2_1; + case DXVA2_VideoTransFunc_28: + return AVCOL_TRC_GAMMA28; + + // values from MediaFoundation SDK (mfobjects.h) + case 9: // MFVideoTransFunc_Log_100 + return AVCOL_TRC_LOG; + case 10: // MFVideoTransFunc_Log_316 + return AVCOL_TRC_LOG_SQRT; + case 11: // MFVideoTransFunc_709_sym + // not an FFmpeg transfer characteristic + return AVCOL_TRC_UNSPECIFIED; + case 12: // MFVideoTransFunc_2020_const + case 13: // MFVideoTransFunc_2020 + if (fmt_info->VideoTransferMatrix==5) // MFVideoTransferMatrix_BT2020_12 + return AVCOL_TRC_BT2020_12; + else + return AVCOL_TRC_BT2020_10; + case 14: // MFVideoTransFunc_26 + // not an FFmpeg transfer characteristic + return AVCOL_TRC_UNSPECIFIED; + case 15: // MFVideoTransFunc_2084 + return AVCOL_TRC_SMPTEST2084; + case 16: // MFVideoTransFunc_HLG + return AVCOL_TRC_ARIB_STD_B67; + case 17: // MFVideoTransFunc_10_rel + // not an FFmpeg transfer characteristic? Undocumented also by MS + return AVCOL_TRC_UNSPECIFIED; + + default: + return AVCOL_TRC_UNSPECIFIED; + } +} + +static enum AVChromaLocation dshow_chroma_loc(DXVA2_ExtendedFormat* fmt_info) +{ + if (fmt_info->VideoChromaSubsampling == DXVA2_VideoChromaSubsampling_Cosited) // that is: (DXVA2_VideoChromaSubsampling_Horizontally_Cosited | DXVA2_VideoChromaSubsampling_Vertically_Cosited | DXVA2_VideoChromaSubsampling_Vertically_AlignedChromaPlanes) + return AVCHROMA_LOC_TOPLEFT; + else if (fmt_info->VideoChromaSubsampling == DXVA2_VideoChromaSubsampling_MPEG1) // that is: DXVA2_VideoChromaSubsampling_Vertically_AlignedChromaPlanes + return AVCHROMA_LOC_CENTER; + else if (fmt_info->VideoChromaSubsampling == DXVA2_VideoChromaSubsampling_MPEG2) // that is: (DXVA2_VideoChromaSubsampling_Horizontally_Cosited | DXVA2_VideoChromaSubsampling_Vertically_AlignedChromaPlanes) + return AVCHROMA_LOC_LEFT; + else if (fmt_info->VideoChromaSubsampling == DXVA2_VideoChromaSubsampling_DV_PAL) // that is: (DXVA2_VideoChromaSubsampling_Horizontally_Cosited | DXVA2_VideoChromaSubsampling_Vertically_Cosited) + return AVCHROMA_LOC_TOPLEFT; + else + // unknown + return AVCHROMA_LOC_UNSPECIFIED; +} + static int dshow_read_close(AVFormatContext *s) { @@ -522,6 +728,7 @@ dshow_cycle_formats(AVFormatContext *avctx, enum dshowDeviceType devtype, VIDEO_STREAM_CONFIG_CAPS *vcaps = caps; BITMAPINFOHEADER *bih; int64_t *fr; + DXVA_ExtendedFormat *extended_format_info = NULL; const AVCodecTag *const tags[] = { avformat_get_riff_video_tags(), NULL }; #if DSHOWDEBUG ff_print_VIDEO_STREAM_CONFIG_CAPS(vcaps); @@ -534,6 +741,8 @@ dshow_cycle_formats(AVFormatContext *avctx, enum dshowDeviceType devtype, VIDEOINFOHEADER2 *v = (void *) type->pbFormat; fr = &v->AvgTimePerFrame; bih = &v->bmiHeader; + if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) + extended_format_info = (DXVA_ExtendedFormat*)&v->dwControlFlags; } else { goto next; } @@ -550,11 +759,40 @@ dshow_cycle_formats(AVFormatContext *avctx, enum dshowDeviceType devtype, } else { av_log(avctx, AV_LOG_INFO, " pixel_format=%s", av_get_pix_fmt_name(pix_fmt)); } - av_log(avctx, AV_LOG_INFO, " min s=%ldx%ld fps=%g max s=%ldx%ld fps=%g\n", + av_log(avctx, AV_LOG_INFO, " min s=%ldx%ld fps=%g max s=%ldx%ld fps=%g", vcaps->MinOutputSize.cx, vcaps->MinOutputSize.cy, 1e7 / vcaps->MaxFrameInterval, vcaps->MaxOutputSize.cx, vcaps->MaxOutputSize.cy, 1e7 / vcaps->MinFrameInterval); + if (extended_format_info) { + enum AVColorRange col_range = dshow_color_range(extended_format_info); + enum AVColorSpace col_space = dshow_color_space(extended_format_info); + enum AVColorPrimaries col_prim = dshow_color_primaries(extended_format_info); + enum AVColorTransferCharacteristic col_trc = dshow_color_trc(extended_format_info); + enum AVChromaLocation chroma_loc = dshow_chroma_loc(extended_format_info); + if (col_range != AVCOL_RANGE_UNSPECIFIED || col_space != AVCOL_SPC_UNSPECIFIED || col_prim != AVCOL_PRI_UNSPECIFIED || col_trc != AVCOL_TRC_UNSPECIFIED) { + const char* range = av_color_range_name(col_range); + const char* space = av_color_space_name(col_space); + const char* prim = av_color_primaries_name(col_prim); + const char* trc = av_color_transfer_name(col_trc); + av_log(avctx, AV_LOG_INFO, " (%s, %s/%s/%s", + range ? range : "unknown", + space ? space : "unknown", + prim ? prim : "unknown", + trc ? trc : "unknown"); + if (chroma_loc != AVCHROMA_LOC_UNSPECIFIED) { + const char* chroma = av_chroma_location_name(chroma_loc); + av_log(avctx, AV_LOG_INFO, ", %s", chroma ? chroma : "unknown"); + } + av_log(avctx, AV_LOG_INFO, ")"); + } + else if (chroma_loc != AVCHROMA_LOC_UNSPECIFIED) { + const char* chroma = av_chroma_location_name(chroma_loc); + av_log(avctx, AV_LOG_INFO, "(%s)", chroma ? chroma : "unknown"); + } + } + + av_log(avctx, AV_LOG_INFO, "\n"); continue; } if (ctx->video_codec_id != AV_CODEC_ID_RAWVIDEO) { @@ -1233,6 +1471,7 @@ dshow_add_device(AVFormatContext *avctx, if (devtype == VideoDevice) { BITMAPINFOHEADER *bih = NULL; AVRational time_base; + DXVA_ExtendedFormat* extended_format_info = NULL; if (IsEqualGUID(&type.formattype, &FORMAT_VideoInfo)) { VIDEOINFOHEADER *v = (void *) type.pbFormat; @@ -1242,6 +1481,8 @@ dshow_add_device(AVFormatContext *avctx, VIDEOINFOHEADER2 *v = (void *) type.pbFormat; time_base = (AVRational) { v->AvgTimePerFrame, 10000000 }; bih = &v->bmiHeader; + if (v->dwControlFlags & AMCONTROL_COLORINFO_PRESENT) + extended_format_info = (DXVA_ExtendedFormat*)&v->dwControlFlags; } if (!bih) { av_log(avctx, AV_LOG_ERROR, "Could not get media type.\n"); @@ -1260,6 +1501,13 @@ dshow_add_device(AVFormatContext *avctx, av_log(avctx, AV_LOG_DEBUG, "attempt to use full range for HDYC...\n"); par->color_range = AVCOL_RANGE_MPEG; // just in case it needs this... } + if (extended_format_info) { + par->color_range = dshow_color_range(extended_format_info); + par->color_space = dshow_color_space(extended_format_info); + par->color_primaries = dshow_color_primaries(extended_format_info); + par->color_trc = dshow_color_trc(extended_format_info); + par->chroma_location = dshow_chroma_loc(extended_format_info); + } if (par->format == AV_PIX_FMT_NONE) { const AVCodecTag *const tags[] = { avformat_get_riff_video_tags(), NULL }; par->codec_id = av_codec_get_id(tags, bih->biCompression);