Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-28 Thread Marton Balint
On Mon, 25 Sep 2017, Marton Balint wrote: On Thu, 21 Sep 2017, Gildas Fargeas wrote: Hey, do you need more stuff changed on this patch ? No, just waiting for other pending decklink patches to push (and test) them at once. Finally applied. Thanks, Marton

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-25 Thread Marton Balint
On Thu, 21 Sep 2017, Gildas Fargeas wrote: Hey, do you need more stuff changed on this patch ? No, just waiting for other pending decklink patches to push (and test) them at once. Regards, Marton 2017-09-07 14:46 GMT+02:00 Gildas Fargeas : Ok, I changed the

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-21 Thread Gildas Fargeas
Hey, do you need more stuff changed on this patch ? 2017-09-07 14:46 GMT+02:00 Gildas Fargeas : > Ok, I changed the texi, added the bitrate and turned rgb48 value to rgb10. > > --- > doc/indevs.texi | 22 ++-- >

[FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-07 Thread Gildas Fargeas
Ok, I changed the texi, added the bitrate and turned rgb48 value to rgb10. --- doc/indevs.texi | 22 ++-- libavdevice/decklink_common.cpp | 2 +- libavdevice/decklink_common_c.h | 1 + libavdevice/decklink_dec.cpp| 46

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-07 Thread Gildas Fargeas
Sorry I completely forgot about the texi doc ! I agree with you regarding rgb48. It feels misleading. r210 is fine with me. I'll send the changes in a few minutes. 2017-09-06 18:47 GMT+02:00 Marton Balint : > > > On Wed, 6 Sep 2017, Gildas Fargeas wrote: > > Alright, this should

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-06 Thread Marton Balint
On Wed, 6 Sep 2017, Gildas Fargeas wrote: Alright, this should do the trick: - use named const for raw_format option - deprecate bm_v210 - bumped avdevice micro version --- libavdevice/decklink_common.cpp | 2 +- libavdevice/decklink_common_c.h | 1 + libavdevice/decklink_dec.cpp| 43

[FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-06 Thread Gildas Fargeas
Alright, this should do the trick: - use named const for raw_format option - deprecate bm_v210 - bumped avdevice micro version --- libavdevice/decklink_common.cpp | 2 +- libavdevice/decklink_common_c.h | 1 + libavdevice/decklink_dec.cpp| 43 ++---

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-05 Thread Marton Balint
On Mon, 4 Sep 2017, Gildas Fargeas wrote: Thanks for the feedback. Here are the changes: - renamed pixelformat_option in raw_format - restored bm_v210 options (if set, it overrides the raw_format option) - removed list_pixelformats and only mention supported raw codes in help. - restore

[FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-04 Thread Gildas Fargeas
Thanks for the feedback. Here are the changes: - renamed pixelformat_option in raw_format - restored bm_v210 options (if set, it overrides the raw_format option) - removed list_pixelformats and only mention supported raw codes in help. - restore previous handling of output Regarding the special

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-02 Thread Marton Balint
On Fri, 1 Sep 2017, Gildas Fargeas wrote: I removed the option bm_v210 as it is replaced with a more complete pixel format selection option. Well, you can't do that, you can only deprecate an option and keep compatibility for some time. E.g. if bm_v210 is set then force pixel format to

[FFmpeg-devel] [PATCH] avdevice/decklink: new options 'list_pixelformats' and 'pixelformat_code' to allow pixelformat selection by code

2017-09-01 Thread Gildas Fargeas
I removed the option bm_v210 as it is replaced with a more complete pixel format selection option. The main objective for this patch was to enable RGB capture. The pixel format codes are directly copied from the Decklink API. Signed-off-by: Gildas Fargeas ---