Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

2016-09-07 Thread Burt P.
applied On Mon, Sep 5, 2016 at 6:18 AM, Burt P wrote: > * use the actual sample rate > * use a more sensible frequency for the tone > * update fate test result > > Signed-off-by: Burt P > --- > libavfilter/af_hdcd.c | 9 ++--- >

Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

2016-09-06 Thread Carl Eugen Hoyos
2016-09-06 14:01 GMT+02:00 Burt P. : > In this patch, I took the opportunity to remove a hard-coded rate and > use one from the framework. Thanks for explaining! Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

2016-09-06 Thread Burt P.
On Tue, Sep 6, 2016 at 6:12 AM, Carl Eugen Hoyos wrote: > 2016-09-06 13:02 GMT+02:00 Burt P. : >> On Tue, Sep 6, 2016 at 5:22 AM, Carl Eugen Hoyos wrote: >>> 2016-09-05 13:18 GMT+02:00 Burt P : * use the actual

Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

2016-09-06 Thread Carl Eugen Hoyos
2016-09-06 13:02 GMT+02:00 Burt P. : > On Tue, Sep 6, 2016 at 5:22 AM, Carl Eugen Hoyos wrote: >> 2016-09-05 13:18 GMT+02:00 Burt P : >>> * use the actual sample rate >> >> Is hdcd supposed to work for sample_rates >> different from 44100? >

Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

2016-09-06 Thread Burt P.
On Tue, Sep 6, 2016 at 5:22 AM, Carl Eugen Hoyos wrote: > 2016-09-05 13:18 GMT+02:00 Burt P : >> * use the actual sample rate > > Is hdcd supposed to work for sample_rates > different from 44100? As I understand it, yes. The PM Model 2 supported code

Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

2016-09-06 Thread Carl Eugen Hoyos
2016-09-05 13:18 GMT+02:00 Burt P : > * use the actual sample rate Is hdcd supposed to work for sample_rates different from 44100? > * use a more sensible frequency for the tone > * update fate test result Can't this be split? Carl Eugen

[FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

2016-09-05 Thread Burt P
* use the actual sample rate * use a more sensible frequency for the tone * update fate test result Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 9 ++--- tests/fate/filter-audio.mak | 2 +- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git