From 60b9785530dfc290a4c8763214938eb367e68cbe Mon Sep 17 00:00:00 2001 From: Loren Merritt Date: Thu, 26 Jul 2012 22:51:13 +0000 Subject: vf_hqdn3d: cosmetics Change code style to match the rest of libav. Signed-off-by: Ronald S. Bultje --- libavfilter/vf_hqdn3d.c | 308 ++++++++++++++++++++++++------------------------ 1 file changed, 152 insertions(+), 156 deletions(-) (limited to 'libavfilter') diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 9e01606041..1fa70c6e8f 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -32,165 +32,161 @@ #include "video.h" typedef struct { - int Coefs[4][512*16]; - unsigned int *Line; - unsigned short *Frame[3]; + int coefs[4][512*16]; + uint32_t *line; + uint16_t *frame_prev[3]; int hsub, vsub; } HQDN3DContext; -static inline unsigned int LowPassMul(unsigned int PrevMul, unsigned int CurrMul, int *Coef) +static inline uint32_t lowpass(unsigned int prev, unsigned int cur, int *coef) { - // int dMul= (PrevMul&0xFFFFFF)-(CurrMul&0xFFFFFF); - int dMul= PrevMul-CurrMul; - unsigned int d=((dMul+0x10007FF)>>12); - return CurrMul + Coef[d]; + int dmul = prev-cur; + unsigned int d = (dmul+0x10007FF)>>12; // 0x1000 to convert to unsigned, 7FF for rounding + return cur + coef[d]; } -static void deNoiseTemporal(unsigned char *FrameSrc, - unsigned char *FrameDest, - unsigned short *FrameAnt, - int W, int H, int sStride, int dStride, - int *Temporal) +static void denoise_temporal(uint8_t *src, uint8_t *dst, + uint16_t *frame_ant, + int w, int h, int sstride, int dstride, + int *temporal) { - long X, Y; - unsigned int PixelDst; - - for (Y = 0; Y < H; Y++) { - for (X = 0; X < W; X++) { - PixelDst = LowPassMul(FrameAnt[X]<<8, FrameSrc[X]<<16, Temporal); - FrameAnt[X] = ((PixelDst+0x1000007F)>>8); - FrameDest[X]= ((PixelDst+0x10007FFF)>>16); + long x, y; + uint32_t pixel; + + for (y = 0; y < h; y++) { + for (x = 0; x < w; x++) { + pixel = lowpass(frame_ant[x]<<8, src[x]<<16, temporal); + frame_ant[x] = ((pixel+0x1000007F)>>8); + dst[x]= ((pixel+0x10007FFF)>>16); } - FrameSrc += sStride; - FrameDest += dStride; - FrameAnt += W; + src += sstride; + dst += dstride; + frame_ant += w; } } -static void deNoiseSpacial(unsigned char *Frame, - unsigned char *FrameDest, - unsigned int *LineAnt, - int W, int H, int sStride, int dStride, - int *Horizontal, int *Vertical) +static void denoise_spatial(uint8_t *src, uint8_t *dst, + uint32_t *line_ant, + int w, int h, int sstride, int dstride, + int *horizontal, int *vertical) { - long X, Y; - long sLineOffs = 0, dLineOffs = 0; - unsigned int PixelAnt; - unsigned int PixelDst; + long x, y; + long sline_offs = 0, dline_offs = 0; + uint32_t pixel_ant; + uint32_t pixel; /* First pixel has no left nor top neighbor. */ - PixelDst = LineAnt[0] = PixelAnt = Frame[0]<<16; - FrameDest[0]= ((PixelDst+0x10007FFF)>>16); + pixel = line_ant[0] = pixel_ant = src[0]<<16; + dst[0]= ((pixel+0x10007FFF)>>16); /* First line has no top neighbor, only left. */ - for (X = 1; X < W; X++) { - PixelDst = LineAnt[X] = LowPassMul(PixelAnt, Frame[X]<<16, Horizontal); - FrameDest[X]= ((PixelDst+0x10007FFF)>>16); + for (x = 1; x < w; x++) { + pixel = line_ant[x] = lowpass(pixel_ant, src[x]<<16, horizontal); + dst[x]= ((pixel+0x10007FFF)>>16); } - for (Y = 1; Y < H; Y++) { - unsigned int PixelAnt; - sLineOffs += sStride, dLineOffs += dStride; + for (y = 1; y < h; y++) { + uint32_t pixel_ant; + sline_offs += sstride, dline_offs += dstride; /* First pixel on each line doesn't have previous pixel */ - PixelAnt = Frame[sLineOffs]<<16; - PixelDst = LineAnt[0] = LowPassMul(LineAnt[0], PixelAnt, Vertical); - FrameDest[dLineOffs]= ((PixelDst+0x10007FFF)>>16); + pixel_ant = src[sline_offs]<<16; + pixel = line_ant[0] = lowpass(line_ant[0], pixel_ant, vertical); + dst[dline_offs]= ((pixel+0x10007FFF)>>16); - for (X = 1; X < W; X++) { - unsigned int PixelDst; + for (x = 1; x < w; x++) { + uint32_t pixel; /* The rest are normal */ - PixelAnt = LowPassMul(PixelAnt, Frame[sLineOffs+X]<<16, Horizontal); - PixelDst = LineAnt[X] = LowPassMul(LineAnt[X], PixelAnt, Vertical); - FrameDest[dLineOffs+X]= ((PixelDst+0x10007FFF)>>16); + pixel_ant = lowpass(pixel_ant, src[sline_offs+x]<<16, horizontal); + pixel = line_ant[x] = lowpass(line_ant[x], pixel_ant, vertical); + dst[dline_offs+x]= ((pixel+0x10007FFF)>>16); } } } -static void deNoise(unsigned char *Frame, - unsigned char *FrameDest, - unsigned int *LineAnt, - unsigned short **FrameAntPtr, - int W, int H, int sStride, int dStride, - int *Horizontal, int *Vertical, int *Temporal) +static void denoise(uint8_t *src, uint8_t *dst, + uint32_t *line_ant, uint16_t **frame_ant_ptr, + int w, int h, int sstride, int dstride, + int *horizontal, int *vertical, int *temporal) { - long X, Y; - long sLineOffs = 0, dLineOffs = 0; - unsigned int PixelAnt; - unsigned int PixelDst; - unsigned short* FrameAnt=(*FrameAntPtr); - - if (!FrameAnt) { - (*FrameAntPtr) = FrameAnt = av_malloc(W*H*sizeof(unsigned short)); - for (Y = 0; Y < H; Y++) { - unsigned short* dst=&FrameAnt[Y*W]; - unsigned char* src=Frame+Y*sStride; - for (X = 0; X < W; X++) dst[X]=src[X]<<8; + long x, y; + long sline_offs = 0, dline_offs = 0; + uint32_t pixel_ant; + uint32_t pixel; + uint16_t *frame_ant = *frame_ant_ptr; + + if (!frame_ant) { + *frame_ant_ptr = frame_ant = av_malloc(w*h*sizeof(uint16_t)); + for (y = 0; y < h; y++) { + uint16_t *frame_dst = frame_ant+y*w; + uint8_t *frame_src = src+y*sstride; + for (x = 0; x < w; x++) + frame_dst[x] = frame_src[x]<<8; } } - if (!Horizontal[0] && !Vertical[0]) { - deNoiseTemporal(Frame, FrameDest, FrameAnt, - W, H, sStride, dStride, Temporal); + if (!horizontal[0] && !vertical[0]) { + denoise_temporal(src, dst, frame_ant, + w, h, sstride, dstride, temporal); return; } - if (!Temporal[0]) { - deNoiseSpacial(Frame, FrameDest, LineAnt, - W, H, sStride, dStride, Horizontal, Vertical); + if (!temporal[0]) { + denoise_spatial(src, dst, line_ant, + w, h, sstride, dstride, horizontal, vertical); return; } /* First pixel has no left nor top neighbor. Only previous frame */ - LineAnt[0] = PixelAnt = Frame[0]<<16; - PixelDst = LowPassMul(FrameAnt[0]<<8, PixelAnt, Temporal); - FrameAnt[0] = ((PixelDst+0x1000007F)>>8); - FrameDest[0]= ((PixelDst+0x10007FFF)>>16); + line_ant[0] = pixel_ant = src[0]<<16; + pixel = lowpass(frame_ant[0]<<8, pixel_ant, temporal); + frame_ant[0] = ((pixel+0x1000007F)>>8); + dst[0]= ((pixel+0x10007FFF)>>16); /* First line has no top neighbor. Only left one for each pixel and * last frame */ - for (X = 1; X < W; X++) { - LineAnt[X] = PixelAnt = LowPassMul(PixelAnt, Frame[X]<<16, Horizontal); - PixelDst = LowPassMul(FrameAnt[X]<<8, PixelAnt, Temporal); - FrameAnt[X] = ((PixelDst+0x1000007F)>>8); - FrameDest[X]= ((PixelDst+0x10007FFF)>>16); + for (x = 1; x < w; x++) { + line_ant[x] = pixel_ant = lowpass(pixel_ant, src[x]<<16, horizontal); + pixel = lowpass(frame_ant[x]<<8, pixel_ant, temporal); + frame_ant[x] = ((pixel+0x1000007F)>>8); + dst[x]= ((pixel+0x10007FFF)>>16); } - for (Y = 1; Y < H; Y++) { - unsigned int PixelAnt; - unsigned short* LinePrev=&FrameAnt[Y*W]; - sLineOffs += sStride, dLineOffs += dStride; + for (y = 1; y < h; y++) { + uint32_t pixel_ant; + uint16_t *line_prev = frame_ant+y*w; + sline_offs += sstride, dline_offs += dstride; /* First pixel on each line doesn't have previous pixel */ - PixelAnt = Frame[sLineOffs]<<16; - LineAnt[0] = LowPassMul(LineAnt[0], PixelAnt, Vertical); - PixelDst = LowPassMul(LinePrev[0]<<8, LineAnt[0], Temporal); - LinePrev[0] = ((PixelDst+0x1000007F)>>8); - FrameDest[dLineOffs]= ((PixelDst+0x10007FFF)>>16); - - for (X = 1; X < W; X++) { - unsigned int PixelDst; + pixel_ant = src[sline_offs]<<16; + line_ant[0] = lowpass(line_ant[0], pixel_ant, vertical); + pixel = lowpass(line_prev[0]<<8, line_ant[0], temporal); + line_prev[0] = ((pixel+0x1000007F)>>8); + dst[dline_offs]= ((pixel+0x10007FFF)>>16); + + for (x = 1; x < w; x++) { + uint32_t pixel; /* The rest are normal */ - PixelAnt = LowPassMul(PixelAnt, Frame[sLineOffs+X]<<16, Horizontal); - LineAnt[X] = LowPassMul(LineAnt[X], PixelAnt, Vertical); - PixelDst = LowPassMul(LinePrev[X]<<8, LineAnt[X], Temporal); - LinePrev[X] = ((PixelDst+0x1000007F)>>8); - FrameDest[dLineOffs+X]= ((PixelDst+0x10007FFF)>>16); + pixel_ant = lowpass(pixel_ant, src[sline_offs+x]<<16, horizontal); + line_ant[x] = lowpass(line_ant[x], pixel_ant, vertical); + pixel = lowpass(line_prev[x]<<8, line_ant[x], temporal); + line_prev[x] = ((pixel+0x1000007F)>>8); + dst[dline_offs+x]= ((pixel+0x10007FFF)>>16); } } } -static void PrecalcCoefs(int *Ct, double Dist25) +static void precalc_coefs(int *ct, double dist25) { int i; - double Gamma, Simil, C; + double gamma, simil, C; - Gamma = log(0.25) / log(1.0 - Dist25/255.0 - 0.00001); + gamma = log(0.25) / log(1.0 - dist25/255.0 - 0.00001); for (i = -255*16; i <= 255*16; i++) { - Simil = 1.0 - FFABS(i) / (16*255.0); - C = pow(Simil, Gamma) * 65536.0 * i / 16.0; - Ct[16*256+i] = lrint(C); + simil = 1.0 - FFABS(i) / (16*255.0); + C = pow(simil, gamma) * 65536.0 * i / 16.0; + ct[16*256+i] = lrint(C); } - Ct[0] = !!Dist25; + ct[0] = !!dist25; } #define PARAM1_DEFAULT 4.0 @@ -200,57 +196,57 @@ static void PrecalcCoefs(int *Ct, double Dist25) static int init(AVFilterContext *ctx, const char *args) { HQDN3DContext *hqdn3d = ctx->priv; - double LumSpac, LumTmp, ChromSpac, ChromTmp; - double Param1, Param2, Param3, Param4; + double lum_spac, lum_tmp, chrom_spac, chrom_tmp; + double param1, param2, param3, param4; - LumSpac = PARAM1_DEFAULT; - ChromSpac = PARAM2_DEFAULT; - LumTmp = PARAM3_DEFAULT; - ChromTmp = LumTmp * ChromSpac / LumSpac; + lum_spac = PARAM1_DEFAULT; + chrom_spac = PARAM2_DEFAULT; + lum_tmp = PARAM3_DEFAULT; + chrom_tmp = lum_tmp * chrom_spac / lum_spac; if (args) { switch (sscanf(args, "%lf:%lf:%lf:%lf", - &Param1, &Param2, &Param3, &Param4)) { + ¶m1, ¶m2, ¶m3, ¶m4)) { case 1: - LumSpac = Param1; - ChromSpac = PARAM2_DEFAULT * Param1 / PARAM1_DEFAULT; - LumTmp = PARAM3_DEFAULT * Param1 / PARAM1_DEFAULT; - ChromTmp = LumTmp * ChromSpac / LumSpac; + lum_spac = param1; + chrom_spac = PARAM2_DEFAULT * param1 / PARAM1_DEFAULT; + lum_tmp = PARAM3_DEFAULT * param1 / PARAM1_DEFAULT; + chrom_tmp = lum_tmp * chrom_spac / lum_spac; break; case 2: - LumSpac = Param1; - ChromSpac = Param2; - LumTmp = PARAM3_DEFAULT * Param1 / PARAM1_DEFAULT; - ChromTmp = LumTmp * ChromSpac / LumSpac; + lum_spac = param1; + chrom_spac = param2; + lum_tmp = PARAM3_DEFAULT * param1 / PARAM1_DEFAULT; + chrom_tmp = lum_tmp * chrom_spac / lum_spac; break; case 3: - LumSpac = Param1; - ChromSpac = Param2; - LumTmp = Param3; - ChromTmp = LumTmp * ChromSpac / LumSpac; + lum_spac = param1; + chrom_spac = param2; + lum_tmp = param3; + chrom_tmp = lum_tmp * chrom_spac / lum_spac; break; case 4: - LumSpac = Param1; - ChromSpac = Param2; - LumTmp = Param3; - ChromTmp = Param4; + lum_spac = param1; + chrom_spac = param2; + lum_tmp = param3; + chrom_tmp = param4; break; } } av_log(ctx, AV_LOG_VERBOSE, "ls:%lf cs:%lf lt:%lf ct:%lf\n", - LumSpac, ChromSpac, LumTmp, ChromTmp); - if (LumSpac < 0 || ChromSpac < 0 || isnan(ChromTmp)) { + lum_spac, chrom_spac, lum_tmp, chrom_tmp); + if (lum_spac < 0 || chrom_spac < 0 || isnan(chrom_tmp)) { av_log(ctx, AV_LOG_ERROR, "Invalid negative value for luma or chroma spatial strength, " "or resulting value for chroma temporal strength is nan.\n"); return AVERROR(EINVAL); } - PrecalcCoefs(hqdn3d->Coefs[0], LumSpac); - PrecalcCoefs(hqdn3d->Coefs[1], LumTmp); - PrecalcCoefs(hqdn3d->Coefs[2], ChromSpac); - PrecalcCoefs(hqdn3d->Coefs[3], ChromTmp); + precalc_coefs(hqdn3d->coefs[0], lum_spac); + precalc_coefs(hqdn3d->coefs[1], lum_tmp); + precalc_coefs(hqdn3d->coefs[2], chrom_spac); + precalc_coefs(hqdn3d->coefs[3], chrom_tmp); return 0; } @@ -259,10 +255,10 @@ static void uninit(AVFilterContext *ctx) { HQDN3DContext *hqdn3d = ctx->priv; - av_freep(&hqdn3d->Line); - av_freep(&hqdn3d->Frame[0]); - av_freep(&hqdn3d->Frame[1]); - av_freep(&hqdn3d->Frame[2]); + av_freep(&hqdn3d->line); + av_freep(&hqdn3d->frame_prev[0]); + av_freep(&hqdn3d->frame_prev[1]); + av_freep(&hqdn3d->frame_prev[2]); } static int query_formats(AVFilterContext *ctx) @@ -283,8 +279,8 @@ static int config_input(AVFilterLink *inlink) hqdn3d->hsub = av_pix_fmt_descriptors[inlink->format].log2_chroma_w; hqdn3d->vsub = av_pix_fmt_descriptors[inlink->format].log2_chroma_h; - hqdn3d->Line = av_malloc(inlink->w * sizeof(*hqdn3d->Line)); - if (!hqdn3d->Line) + hqdn3d->line = av_malloc(inlink->w * sizeof(*hqdn3d->line)); + if (!hqdn3d->line) return AVERROR(ENOMEM); return 0; @@ -305,24 +301,24 @@ static int end_frame(AVFilterLink *inlink) int ch = inpic->video->h >> hqdn3d->vsub; int ret; - deNoise(inpic->data[0], outpic->data[0], - hqdn3d->Line, &hqdn3d->Frame[0], inpic->video->w, inpic->video->h, + denoise(inpic->data[0], outpic->data[0], + hqdn3d->line, &hqdn3d->frame_prev[0], inpic->video->w, inpic->video->h, inpic->linesize[0], outpic->linesize[0], - hqdn3d->Coefs[0], - hqdn3d->Coefs[0], - hqdn3d->Coefs[1]); - deNoise(inpic->data[1], outpic->data[1], - hqdn3d->Line, &hqdn3d->Frame[1], cw, ch, + hqdn3d->coefs[0], + hqdn3d->coefs[0], + hqdn3d->coefs[1]); + denoise(inpic->data[1], outpic->data[1], + hqdn3d->line, &hqdn3d->frame_prev[1], cw, ch, inpic->linesize[1], outpic->linesize[1], - hqdn3d->Coefs[2], - hqdn3d->Coefs[2], - hqdn3d->Coefs[3]); - deNoise(inpic->data[2], outpic->data[2], - hqdn3d->Line, &hqdn3d->Frame[2], cw, ch, + hqdn3d->coefs[2], + hqdn3d->coefs[2], + hqdn3d->coefs[3]); + denoise(inpic->data[2], outpic->data[2], + hqdn3d->line, &hqdn3d->frame_prev[2], cw, ch, inpic->linesize[2], outpic->linesize[2], - hqdn3d->Coefs[2], - hqdn3d->Coefs[2], - hqdn3d->Coefs[3]); + hqdn3d->coefs[2], + hqdn3d->coefs[2], + hqdn3d->coefs[3]); if ((ret = ff_draw_slice(outlink, 0, inpic->video->h, 1)) < 0 || (ret = ff_end_frame(outlink)) < 0) -- cgit v1.2.3 From fb44e7401f44ce9c1dded07d992c57bd82ff666c Mon Sep 17 00:00:00 2001 From: Loren Merritt Date: Thu, 26 Jul 2012 22:51:12 +0000 Subject: factor identical ff_inplace_start_frame out of two filters Signed-off-by: Ronald S. Bultje --- libavfilter/vf_delogo.c | 38 +------------------------------------- libavfilter/vf_gradfun.c | 37 +------------------------------------ libavfilter/video.c | 36 ++++++++++++++++++++++++++++++++++++ libavfilter/video.h | 1 + 4 files changed, 39 insertions(+), 73 deletions(-) (limited to 'libavfilter') diff --git a/libavfilter/vf_delogo.c b/libavfilter/vf_delogo.c index 343585fc5e..e8b5431caf 100644 --- a/libavfilter/vf_delogo.c +++ b/libavfilter/vf_delogo.c @@ -214,42 +214,6 @@ static av_cold int init(AVFilterContext *ctx, const char *args) return 0; } -static int start_frame(AVFilterLink *inlink, AVFilterBufferRef *inpicref) -{ - AVFilterLink *outlink = inlink->dst->outputs[0]; - AVFilterBufferRef *outpicref = NULL, *for_next_filter; - int ret = 0; - - if (inpicref->perms & AV_PERM_PRESERVE) { - outpicref = ff_get_video_buffer(outlink, AV_PERM_WRITE, - outlink->w, outlink->h); - if (!outpicref) - return AVERROR(ENOMEM); - - avfilter_copy_buffer_ref_props(outpicref, inpicref); - outpicref->video->w = outlink->w; - outpicref->video->h = outlink->h; - } else { - outpicref = avfilter_ref_buffer(inpicref, ~0); - if (!outpicref) - return AVERROR(ENOMEM); - } - - for_next_filter = avfilter_ref_buffer(outpicref, ~0); - if (for_next_filter) - ret = ff_start_frame(outlink, for_next_filter); - else - ret = AVERROR(ENOMEM); - - if (ret < 0) { - avfilter_unref_bufferp(&outpicref); - return ret; - } - - outlink->out_buf = outpicref; - return 0; -} - static int null_draw_slice(AVFilterLink *link, int y, int h, int slice_dir) { return 0; @@ -296,7 +260,7 @@ AVFilter avfilter_vf_delogo = { .inputs = (const AVFilterPad[]) {{ .name = "default", .type = AVMEDIA_TYPE_VIDEO, .get_video_buffer = ff_null_get_video_buffer, - .start_frame = start_frame, + .start_frame = ff_inplace_start_frame, .draw_slice = null_draw_slice, .end_frame = end_frame, .min_perms = AV_PERM_WRITE | AV_PERM_READ, diff --git a/libavfilter/vf_gradfun.c b/libavfilter/vf_gradfun.c index 52dcb7092f..f8896b5e57 100644 --- a/libavfilter/vf_gradfun.c +++ b/libavfilter/vf_gradfun.c @@ -180,41 +180,6 @@ static int config_input(AVFilterLink *inlink) return 0; } -static int start_frame(AVFilterLink *inlink, AVFilterBufferRef *inpicref) -{ - AVFilterLink *outlink = inlink->dst->outputs[0]; - AVFilterBufferRef *outpicref = NULL, *for_next_filter; - int ret = 0; - - if (inpicref->perms & AV_PERM_PRESERVE) { - outpicref = ff_get_video_buffer(outlink, AV_PERM_WRITE, outlink->w, outlink->h); - if (!outpicref) - return AVERROR(ENOMEM); - - avfilter_copy_buffer_ref_props(outpicref, inpicref); - outpicref->video->w = outlink->w; - outpicref->video->h = outlink->h; - } else { - outpicref = avfilter_ref_buffer(inpicref, ~0); - if (!outpicref) - return AVERROR(ENOMEM); - } - - for_next_filter = avfilter_ref_buffer(outpicref, ~0); - if (for_next_filter) - ret = ff_start_frame(outlink, for_next_filter); - else - ret = AVERROR(ENOMEM); - - if (ret < 0) { - avfilter_unref_bufferp(&outpicref); - return ret; - } - - outlink->out_buf = outpicref; - return 0; -} - static int null_draw_slice(AVFilterLink *link, int y, int h, int slice_dir) { return 0; @@ -261,7 +226,7 @@ AVFilter avfilter_vf_gradfun = { .inputs = (const AVFilterPad[]) {{ .name = "default", .type = AVMEDIA_TYPE_VIDEO, .config_props = config_input, - .start_frame = start_frame, + .start_frame = ff_inplace_start_frame, .draw_slice = null_draw_slice, .end_frame = end_frame, .min_perms = AV_PERM_READ, }, diff --git a/libavfilter/video.c b/libavfilter/video.c index 6e5063778a..ebbbc346bc 100644 --- a/libavfilter/video.c +++ b/libavfilter/video.c @@ -168,6 +168,42 @@ int ff_null_start_frame(AVFilterLink *link, AVFilterBufferRef *picref) return ff_start_frame(link->dst->outputs[0], buf_out); } +// for filters that support (but don't require) outpic==inpic +int ff_inplace_start_frame(AVFilterLink *inlink, AVFilterBufferRef *inpicref) +{ + AVFilterLink *outlink = inlink->dst->outputs[0]; + AVFilterBufferRef *outpicref = NULL, *for_next_filter; + int ret = 0; + + if ((inpicref->perms & AV_PERM_WRITE) && !(inpicref->perms & AV_PERM_PRESERVE)) { + outpicref = avfilter_ref_buffer(inpicref, ~0); + if (!outpicref) + return AVERROR(ENOMEM); + } else { + outpicref = ff_get_video_buffer(outlink, AV_PERM_WRITE, outlink->w, outlink->h); + if (!outpicref) + return AVERROR(ENOMEM); + + avfilter_copy_buffer_ref_props(outpicref, inpicref); + outpicref->video->w = outlink->w; + outpicref->video->h = outlink->h; + } + + for_next_filter = avfilter_ref_buffer(outpicref, ~0); + if (for_next_filter) + ret = ff_start_frame(outlink, for_next_filter); + else + ret = AVERROR(ENOMEM); + + if (ret < 0) { + avfilter_unref_bufferp(&outpicref); + return ret; + } + + outlink->out_buf = outpicref; + return 0; +} + static int default_start_frame(AVFilterLink *inlink, AVFilterBufferRef *picref) { AVFilterLink *outlink = NULL; diff --git a/libavfilter/video.h b/libavfilter/video.h index ef4a1a9c32..348240c6e6 100644 --- a/libavfilter/video.h +++ b/libavfilter/video.h @@ -39,6 +39,7 @@ AVFilterBufferRef *ff_null_get_video_buffer(AVFilterLink *link, int perms, int w AVFilterBufferRef *ff_get_video_buffer(AVFilterLink *link, int perms, int w, int h); +int ff_inplace_start_frame(AVFilterLink *link, AVFilterBufferRef *picref); int ff_null_start_frame(AVFilterLink *link, AVFilterBufferRef *picref); int ff_null_draw_slice(AVFilterLink *link, int y, int h, int slice_dir); int ff_null_end_frame(AVFilterLink *link); -- cgit v1.2.3 From 85e228c71d44ce36f7c0f51d1cf9b63e84a082f3 Mon Sep 17 00:00:00 2001 From: Loren Merritt Date: Thu, 26 Jul 2012 22:51:14 +0000 Subject: vf_hqdn3d: simplify and optimize 14% faster on penryn, 2% on sandybridge, 9% on bulldozer Signed-off-by: Ronald S. Bultje --- libavfilter/vf_hqdn3d.c | 157 ++++++++++++++++-------------------------------- 1 file changed, 51 insertions(+), 106 deletions(-) (limited to 'libavfilter') diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 1fa70c6e8f..bb81b79f1d 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -1,6 +1,7 @@ /* * Copyright (c) 2003 Daniel Moreno * Copyright (c) 2010 Baptiste Coudurier + * Copyright (c) 2012 Loren Merritt * * This file is part of Libav, ported from MPlayer. * @@ -51,13 +52,13 @@ static void denoise_temporal(uint8_t *src, uint8_t *dst, int *temporal) { long x, y; - uint32_t pixel; + uint32_t tmp; for (y = 0; y < h; y++) { for (x = 0; x < w; x++) { - pixel = lowpass(frame_ant[x]<<8, src[x]<<16, temporal); - frame_ant[x] = ((pixel+0x1000007F)>>8); - dst[x]= ((pixel+0x10007FFF)>>16); + tmp = lowpass(frame_ant[x]<<8, src[x]<<16, temporal); + frame_ant[x] = (tmp+0x7F)>>8; + dst[x] = (tmp+0x7FFF)>>16; } src += sstride; dst += dstride; @@ -66,111 +67,66 @@ static void denoise_temporal(uint8_t *src, uint8_t *dst, } static void denoise_spatial(uint8_t *src, uint8_t *dst, - uint32_t *line_ant, + uint32_t *line_ant, uint16_t *frame_ant, int w, int h, int sstride, int dstride, - int *horizontal, int *vertical) + int *spatial, int *temporal) { long x, y; - long sline_offs = 0, dline_offs = 0; uint32_t pixel_ant; - uint32_t pixel; + uint32_t tmp; - /* First pixel has no left nor top neighbor. */ - pixel = line_ant[0] = pixel_ant = src[0]<<16; - dst[0]= ((pixel+0x10007FFF)>>16); - - /* First line has no top neighbor, only left. */ - for (x = 1; x < w; x++) { - pixel = line_ant[x] = lowpass(pixel_ant, src[x]<<16, horizontal); - dst[x]= ((pixel+0x10007FFF)>>16); + /* First line has no top neighbor. Only left one for each tmp and + * last frame */ + pixel_ant = src[0]<<16; + for (x = 0; x < w; x++) { + line_ant[x] = tmp = pixel_ant = lowpass(pixel_ant, src[x]<<16, spatial); + tmp = lowpass(frame_ant[x]<<8, tmp, temporal); + frame_ant[x] = (tmp+0x7F)>>8; + dst[x] = (tmp+0x7FFF)>>16; } for (y = 1; y < h; y++) { - uint32_t pixel_ant; - sline_offs += sstride, dline_offs += dstride; - /* First pixel on each line doesn't have previous pixel */ - pixel_ant = src[sline_offs]<<16; - pixel = line_ant[0] = lowpass(line_ant[0], pixel_ant, vertical); - dst[dline_offs]= ((pixel+0x10007FFF)>>16); - - for (x = 1; x < w; x++) { - uint32_t pixel; - /* The rest are normal */ - pixel_ant = lowpass(pixel_ant, src[sline_offs+x]<<16, horizontal); - pixel = line_ant[x] = lowpass(line_ant[x], pixel_ant, vertical); - dst[dline_offs+x]= ((pixel+0x10007FFF)>>16); + src += sstride; + dst += dstride; + frame_ant += w; + pixel_ant = src[0]<<16; + for (x = 0; x < w-1; x++) { + line_ant[x] = tmp = lowpass(line_ant[x], pixel_ant, spatial); + pixel_ant = lowpass(pixel_ant, src[x+1]<<16, spatial); + tmp = lowpass(frame_ant[x]<<8, tmp, temporal); + frame_ant[x] = (tmp+0x7F)>>8; + dst[x] = (tmp+0x7FFF)>>16; } + line_ant[x] = tmp = lowpass(line_ant[x], pixel_ant, spatial); + tmp = lowpass(frame_ant[x]<<8, tmp, temporal); + frame_ant[x] = (tmp+0x7F)>>8; + dst[x] = (tmp+0x7FFF)>>16; } } static void denoise(uint8_t *src, uint8_t *dst, uint32_t *line_ant, uint16_t **frame_ant_ptr, int w, int h, int sstride, int dstride, - int *horizontal, int *vertical, int *temporal) + int *spatial, int *temporal) { long x, y; - long sline_offs = 0, dline_offs = 0; - uint32_t pixel_ant; - uint32_t pixel; uint16_t *frame_ant = *frame_ant_ptr; - if (!frame_ant) { + uint8_t *frame_src = src; *frame_ant_ptr = frame_ant = av_malloc(w*h*sizeof(uint16_t)); - for (y = 0; y < h; y++) { - uint16_t *frame_dst = frame_ant+y*w; - uint8_t *frame_src = src+y*sstride; + for (y = 0; y < h; y++, src += sstride, frame_ant += w) for (x = 0; x < w; x++) - frame_dst[x] = frame_src[x]<<8; - } + frame_ant[x] = src[x]<<8; + src = frame_src; + frame_ant = *frame_ant_ptr; } - if (!horizontal[0] && !vertical[0]) { + if (spatial[0]) + denoise_spatial(src, dst, line_ant, frame_ant, + w, h, sstride, dstride, spatial, temporal); + else denoise_temporal(src, dst, frame_ant, w, h, sstride, dstride, temporal); - return; - } - if (!temporal[0]) { - denoise_spatial(src, dst, line_ant, - w, h, sstride, dstride, horizontal, vertical); - return; - } - - /* First pixel has no left nor top neighbor. Only previous frame */ - line_ant[0] = pixel_ant = src[0]<<16; - pixel = lowpass(frame_ant[0]<<8, pixel_ant, temporal); - frame_ant[0] = ((pixel+0x1000007F)>>8); - dst[0]= ((pixel+0x10007FFF)>>16); - - /* First line has no top neighbor. Only left one for each pixel and - * last frame */ - for (x = 1; x < w; x++) { - line_ant[x] = pixel_ant = lowpass(pixel_ant, src[x]<<16, horizontal); - pixel = lowpass(frame_ant[x]<<8, pixel_ant, temporal); - frame_ant[x] = ((pixel+0x1000007F)>>8); - dst[x]= ((pixel+0x10007FFF)>>16); - } - - for (y = 1; y < h; y++) { - uint32_t pixel_ant; - uint16_t *line_prev = frame_ant+y*w; - sline_offs += sstride, dline_offs += dstride; - /* First pixel on each line doesn't have previous pixel */ - pixel_ant = src[sline_offs]<<16; - line_ant[0] = lowpass(line_ant[0], pixel_ant, vertical); - pixel = lowpass(line_prev[0]<<8, line_ant[0], temporal); - line_prev[0] = ((pixel+0x1000007F)>>8); - dst[dline_offs]= ((pixel+0x10007FFF)>>16); - - for (x = 1; x < w; x++) { - uint32_t pixel; - /* The rest are normal */ - pixel_ant = lowpass(pixel_ant, src[sline_offs+x]<<16, horizontal); - line_ant[x] = lowpass(line_ant[x], pixel_ant, vertical); - pixel = lowpass(line_prev[x]<<8, line_ant[x], temporal); - line_prev[x] = ((pixel+0x1000007F)>>8); - dst[dline_offs+x]= ((pixel+0x10007FFF)>>16); - } - } } static void precalc_coefs(int *ct, double dist25) @@ -297,28 +253,16 @@ static int end_frame(AVFilterLink *inlink) AVFilterLink *outlink = inlink->dst->outputs[0]; AVFilterBufferRef *inpic = inlink ->cur_buf; AVFilterBufferRef *outpic = outlink->out_buf; - int cw = inpic->video->w >> hqdn3d->hsub; - int ch = inpic->video->h >> hqdn3d->vsub; - int ret; - - denoise(inpic->data[0], outpic->data[0], - hqdn3d->line, &hqdn3d->frame_prev[0], inpic->video->w, inpic->video->h, - inpic->linesize[0], outpic->linesize[0], - hqdn3d->coefs[0], - hqdn3d->coefs[0], - hqdn3d->coefs[1]); - denoise(inpic->data[1], outpic->data[1], - hqdn3d->line, &hqdn3d->frame_prev[1], cw, ch, - inpic->linesize[1], outpic->linesize[1], - hqdn3d->coefs[2], - hqdn3d->coefs[2], - hqdn3d->coefs[3]); - denoise(inpic->data[2], outpic->data[2], - hqdn3d->line, &hqdn3d->frame_prev[2], cw, ch, - inpic->linesize[2], outpic->linesize[2], - hqdn3d->coefs[2], - hqdn3d->coefs[2], - hqdn3d->coefs[3]); + int ret, c; + + for (c = 0; c < 3; c++) { + denoise(inpic->data[c], outpic->data[c], + hqdn3d->line, &hqdn3d->frame_prev[c], + inpic->video->w >> (!!c * hqdn3d->hsub), + inpic->video->h >> (!!c * hqdn3d->vsub), + inpic->linesize[c], outpic->linesize[c], + hqdn3d->coefs[c?2:0], hqdn3d->coefs[c?3:1]); + } if ((ret = ff_draw_slice(outlink, 0, inpic->video->h, 1)) < 0 || (ret = ff_end_frame(outlink)) < 0) @@ -337,6 +281,7 @@ AVFilter avfilter_vf_hqdn3d = { .inputs = (const AVFilterPad[]) {{ .name = "default", .type = AVMEDIA_TYPE_VIDEO, + .start_frame = ff_inplace_start_frame, .draw_slice = null_draw_slice, .config_props = config_input, .end_frame = end_frame }, -- cgit v1.2.3 From 0f583e6cc55dfea865b6b3c9b4ac653455f37ec2 Mon Sep 17 00:00:00 2001 From: Loren Merritt Date: Thu, 26 Jul 2012 22:51:15 +0000 Subject: vf_hqdn3d: reduce intermediate precision 11% faster on penryn, 7% on sandybridge, 5% on bulldozer Negligible change to output. Signed-off-by: Ronald S. Bultje --- libavfilter/vf_hqdn3d.c | 62 +++++++++++++++++++++++++------------------------ 1 file changed, 32 insertions(+), 30 deletions(-) (limited to 'libavfilter') diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index bb81b79f1d..46902fa290 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -33,32 +33,32 @@ #include "video.h" typedef struct { - int coefs[4][512*16]; - uint32_t *line; + int16_t coefs[4][512*16]; + uint16_t *line; uint16_t *frame_prev[3]; int hsub, vsub; } HQDN3DContext; -static inline uint32_t lowpass(unsigned int prev, unsigned int cur, int *coef) +static inline uint32_t lowpass(int prev, int cur, int16_t *coef) { - int dmul = prev-cur; - unsigned int d = (dmul+0x10007FF)>>12; // 0x1000 to convert to unsigned, 7FF for rounding + int d = (prev-cur)>>4; return cur + coef[d]; } static void denoise_temporal(uint8_t *src, uint8_t *dst, uint16_t *frame_ant, int w, int h, int sstride, int dstride, - int *temporal) + int16_t *temporal) { long x, y; uint32_t tmp; + temporal += 0x1000; + for (y = 0; y < h; y++) { for (x = 0; x < w; x++) { - tmp = lowpass(frame_ant[x]<<8, src[x]<<16, temporal); - frame_ant[x] = (tmp+0x7F)>>8; - dst[x] = (tmp+0x7FFF)>>16; + frame_ant[x] = tmp = lowpass(frame_ant[x], src[x]<<8, temporal); + dst[x] = (tmp+0x7F)>>8; } src += sstride; dst += dstride; @@ -67,47 +67,47 @@ static void denoise_temporal(uint8_t *src, uint8_t *dst, } static void denoise_spatial(uint8_t *src, uint8_t *dst, - uint32_t *line_ant, uint16_t *frame_ant, + uint16_t *line_ant, uint16_t *frame_ant, int w, int h, int sstride, int dstride, - int *spatial, int *temporal) + int16_t *spatial, int16_t *temporal) { long x, y; uint32_t pixel_ant; uint32_t tmp; + spatial += 0x1000; + temporal += 0x1000; + /* First line has no top neighbor. Only left one for each tmp and * last frame */ - pixel_ant = src[0]<<16; + pixel_ant = src[0]<<8; for (x = 0; x < w; x++) { - line_ant[x] = tmp = pixel_ant = lowpass(pixel_ant, src[x]<<16, spatial); - tmp = lowpass(frame_ant[x]<<8, tmp, temporal); - frame_ant[x] = (tmp+0x7F)>>8; - dst[x] = (tmp+0x7FFF)>>16; + line_ant[x] = tmp = pixel_ant = lowpass(pixel_ant, src[x]<<8, spatial); + frame_ant[x] = tmp = lowpass(frame_ant[x], tmp, temporal); + dst[x] = (tmp+0x7F)>>8; } for (y = 1; y < h; y++) { src += sstride; dst += dstride; frame_ant += w; - pixel_ant = src[0]<<16; + pixel_ant = src[0]<<8; for (x = 0; x < w-1; x++) { line_ant[x] = tmp = lowpass(line_ant[x], pixel_ant, spatial); - pixel_ant = lowpass(pixel_ant, src[x+1]<<16, spatial); - tmp = lowpass(frame_ant[x]<<8, tmp, temporal); - frame_ant[x] = (tmp+0x7F)>>8; - dst[x] = (tmp+0x7FFF)>>16; + pixel_ant = lowpass(pixel_ant, src[x+1]<<8, spatial); + frame_ant[x] = tmp = lowpass(frame_ant[x], tmp, temporal); + dst[x] = (tmp+0x7F)>>8; } line_ant[x] = tmp = lowpass(line_ant[x], pixel_ant, spatial); - tmp = lowpass(frame_ant[x]<<8, tmp, temporal); - frame_ant[x] = (tmp+0x7F)>>8; - dst[x] = (tmp+0x7FFF)>>16; + frame_ant[x] = tmp = lowpass(frame_ant[x], tmp, temporal); + dst[x] = (tmp+0x7F)>>8; } } static void denoise(uint8_t *src, uint8_t *dst, - uint32_t *line_ant, uint16_t **frame_ant_ptr, + uint16_t *line_ant, uint16_t **frame_ant_ptr, int w, int h, int sstride, int dstride, - int *spatial, int *temporal) + int16_t *spatial, int16_t *temporal) { long x, y; uint16_t *frame_ant = *frame_ant_ptr; @@ -129,16 +129,18 @@ static void denoise(uint8_t *src, uint8_t *dst, w, h, sstride, dstride, temporal); } -static void precalc_coefs(int *ct, double dist25) +static void precalc_coefs(int16_t *ct, double dist25) { int i; double gamma, simil, C; - gamma = log(0.25) / log(1.0 - dist25/255.0 - 0.00001); + gamma = log(0.25) / log(1.0 - FFMIN(dist25,252.0)/255.0 - 0.00001); for (i = -255*16; i <= 255*16; i++) { - simil = 1.0 - FFABS(i) / (16*255.0); - C = pow(simil, gamma) * 65536.0 * i / 16.0; + // lowpass() truncates (not rounds) the diff, so +15/32 for the midpoint of the bin. + double f = (i + 15.0/32.0) / 16.0; + simil = 1.0 - FFABS(f) / 255.0; + C = pow(simil, gamma) * 256.0 * f; ct[16*256+i] = lrint(C); } -- cgit v1.2.3 From 1ad715dbf3bf14505a7d713bb2ea42e5e7a63694 Mon Sep 17 00:00:00 2001 From: Loren Merritt Date: Fri, 27 Jul 2012 01:42:05 +0000 Subject: vf_hqdn3d: support 9 and 10bit colordepth Signed-off-by: Ronald S. Bultje --- libavfilter/vf_hqdn3d.c | 72 ++++++++++++++++++++++++++++++++++++------------- 1 file changed, 53 insertions(+), 19 deletions(-) (limited to 'libavfilter') diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 46902fa290..138b527a70 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -27,6 +27,7 @@ */ #include "libavutil/pixdesc.h" +#include "libavutil/intreadwrite.h" #include "avfilter.h" #include "formats.h" #include "internal.h" @@ -37,18 +38,25 @@ typedef struct { uint16_t *line; uint16_t *frame_prev[3]; int hsub, vsub; + int depth; } HQDN3DContext; +#define RIGHTSHIFT(a,b) (((a)+(((1<<(b))-1)>>1))>>(b)) +#define LOAD(x) ((depth==8 ? src[x] : AV_RN16A(src+(x)*2)) << (16-depth)) +#define STORE(x,val) (depth==8 ? dst[x] = RIGHTSHIFT(val, 16-depth)\ + : AV_WN16A(dst+(x)*2, RIGHTSHIFT(val, 16-depth))) + static inline uint32_t lowpass(int prev, int cur, int16_t *coef) { int d = (prev-cur)>>4; return cur + coef[d]; } +av_always_inline static void denoise_temporal(uint8_t *src, uint8_t *dst, uint16_t *frame_ant, int w, int h, int sstride, int dstride, - int16_t *temporal) + int16_t *temporal, int depth) { long x, y; uint32_t tmp; @@ -57,8 +65,8 @@ static void denoise_temporal(uint8_t *src, uint8_t *dst, for (y = 0; y < h; y++) { for (x = 0; x < w; x++) { - frame_ant[x] = tmp = lowpass(frame_ant[x], src[x]<<8, temporal); - dst[x] = (tmp+0x7F)>>8; + frame_ant[x] = tmp = lowpass(frame_ant[x], LOAD(x), temporal); + STORE(x, tmp); } src += sstride; dst += dstride; @@ -66,10 +74,11 @@ static void denoise_temporal(uint8_t *src, uint8_t *dst, } } +av_always_inline static void denoise_spatial(uint8_t *src, uint8_t *dst, uint16_t *line_ant, uint16_t *frame_ant, int w, int h, int sstride, int dstride, - int16_t *spatial, int16_t *temporal) + int16_t *spatial, int16_t *temporal, int depth) { long x, y; uint32_t pixel_ant; @@ -80,34 +89,35 @@ static void denoise_spatial(uint8_t *src, uint8_t *dst, /* First line has no top neighbor. Only left one for each tmp and * last frame */ - pixel_ant = src[0]<<8; + pixel_ant = LOAD(0); for (x = 0; x < w; x++) { - line_ant[x] = tmp = pixel_ant = lowpass(pixel_ant, src[x]<<8, spatial); + line_ant[x] = tmp = pixel_ant = lowpass(pixel_ant, LOAD(x), spatial); frame_ant[x] = tmp = lowpass(frame_ant[x], tmp, temporal); - dst[x] = (tmp+0x7F)>>8; + STORE(x, tmp); } for (y = 1; y < h; y++) { src += sstride; dst += dstride; frame_ant += w; - pixel_ant = src[0]<<8; + pixel_ant = LOAD(0); for (x = 0; x < w-1; x++) { line_ant[x] = tmp = lowpass(line_ant[x], pixel_ant, spatial); - pixel_ant = lowpass(pixel_ant, src[x+1]<<8, spatial); + pixel_ant = lowpass(pixel_ant, LOAD(x+1), spatial); frame_ant[x] = tmp = lowpass(frame_ant[x], tmp, temporal); - dst[x] = (tmp+0x7F)>>8; + STORE(x, tmp); } line_ant[x] = tmp = lowpass(line_ant[x], pixel_ant, spatial); frame_ant[x] = tmp = lowpass(frame_ant[x], tmp, temporal); - dst[x] = (tmp+0x7F)>>8; + STORE(x, tmp); } } -static void denoise(uint8_t *src, uint8_t *dst, - uint16_t *line_ant, uint16_t **frame_ant_ptr, - int w, int h, int sstride, int dstride, - int16_t *spatial, int16_t *temporal) +av_always_inline +static void denoise_depth(uint8_t *src, uint8_t *dst, + uint16_t *line_ant, uint16_t **frame_ant_ptr, + int w, int h, int sstride, int dstride, + int16_t *spatial, int16_t *temporal, int depth) { long x, y; uint16_t *frame_ant = *frame_ant_ptr; @@ -116,19 +126,26 @@ static void denoise(uint8_t *src, uint8_t *dst, *frame_ant_ptr = frame_ant = av_malloc(w*h*sizeof(uint16_t)); for (y = 0; y < h; y++, src += sstride, frame_ant += w) for (x = 0; x < w; x++) - frame_ant[x] = src[x]<<8; + frame_ant[x] = LOAD(x); src = frame_src; frame_ant = *frame_ant_ptr; } if (spatial[0]) denoise_spatial(src, dst, line_ant, frame_ant, - w, h, sstride, dstride, spatial, temporal); + w, h, sstride, dstride, spatial, temporal, depth); else denoise_temporal(src, dst, frame_ant, - w, h, sstride, dstride, temporal); + w, h, sstride, dstride, temporal, depth); } +#define denoise(...) \ + switch (hqdn3d->depth) {\ + case 8: denoise_depth(__VA_ARGS__, 8); break;\ + case 9: denoise_depth(__VA_ARGS__, 9); break;\ + case 10: denoise_depth(__VA_ARGS__, 10); break;\ + } + static void precalc_coefs(int16_t *ct, double dist25) { int i; @@ -222,7 +239,23 @@ static void uninit(AVFilterContext *ctx) static int query_formats(AVFilterContext *ctx) { static const enum PixelFormat pix_fmts[] = { - PIX_FMT_YUV420P, PIX_FMT_YUV422P, PIX_FMT_YUV411P, PIX_FMT_NONE + PIX_FMT_YUV420P, + PIX_FMT_YUV422P, + PIX_FMT_YUV444P, + PIX_FMT_YUV410P, + PIX_FMT_YUV411P, + PIX_FMT_YUV440P, + PIX_FMT_YUVJ420P, + PIX_FMT_YUVJ422P, + PIX_FMT_YUVJ444P, + PIX_FMT_YUVJ440P, + AV_NE( PIX_FMT_YUV420P9BE, PIX_FMT_YUV420P9LE ), + AV_NE( PIX_FMT_YUV422P9BE, PIX_FMT_YUV422P9LE ), + AV_NE( PIX_FMT_YUV444P9BE, PIX_FMT_YUV444P9LE ), + AV_NE( PIX_FMT_YUV420P10BE, PIX_FMT_YUV420P10LE ), + AV_NE( PIX_FMT_YUV422P10BE, PIX_FMT_YUV422P10LE ), + AV_NE( PIX_FMT_YUV444P10BE, PIX_FMT_YUV444P10LE ), + PIX_FMT_NONE }; ff_set_common_formats(ctx, ff_make_format_list(pix_fmts)); @@ -236,6 +269,7 @@ static int config_input(AVFilterLink *inlink) hqdn3d->hsub = av_pix_fmt_descriptors[inlink->format].log2_chroma_w; hqdn3d->vsub = av_pix_fmt_descriptors[inlink->format].log2_chroma_h; + hqdn3d->depth = av_pix_fmt_descriptors[inlink->format].comp[0].depth_minus1+1; hqdn3d->line = av_malloc(inlink->w * sizeof(*hqdn3d->line)); if (!hqdn3d->line) -- cgit v1.2.3