[FFmpeg-devel] [PATCH] Blackfin MPEGVIDEO DCT Quantization, Last Non Zero optimization

Michael Niedermayer michaelni
Wed Jun 13 02:42:16 CEST 2007


Hi

On Tue, Jun 12, 2007 at 03:04:09PM -0400, mmh wrote:
Content-Description: message body text
> 
> I would like to commit this as soon as possible.  The patch touches a
> couple of other common places most of which you have all reviewed this
> morning.  Sorry about backing out of making the change to quant loop.
> The code is much easier for me to read this way.  The read_time patch
> needs to happen before this patch is applied.  And Reimar, Mans and
> myself have been working to converge on the union vs just take lsw for
> now issue in a separate thread.  I guess I'm just looking for an ok
> for touching the Makefile and mpegvideo.c.
[...]
> Index: libavcodec/bfin/dsputil_bfin.c
> ===================================================================
> --- libavcodec/bfin/dsputil_bfin.c	(revision 9231)
> +++ libavcodec/bfin/dsputil_bfin.c	(working copy)
> @@ -285,6 +285,7 @@
>      c->put_no_rnd_pixels_tab[0][2] = bfin_put_pixels16_y2_nornd;
>      c->put_no_rnd_pixels_tab[0][3] = ff_bfin_put_pixels16_xy2_nornd;
>  
> +    c->idct_permutation_type = FF_NO_IDCT_PERM;
>      c->fdct               = ff_bfin_fdct;
>      c->idct               = ff_bfin_idct;
>      c->idct_add           = bfin_idct_add;

seperate commit


[...]
> +#ifdef BFIN_PROFILE
> +
> +static double Telem[16];
> +static char  *TelemNames[16];
> +static int    TelemCnt;

the profile belongs to a seperate patch/commit


> +
> +#define PROF(lab,e) { int __e = e; char*__lab = lab; uint64_t _t0 = read_time();
> +#define EPROF()       _t0 = read_time()-_t0; Telem[__e] = Telem[__e] + _t0; TelemNames[__e] = __lab; }

_* stuff is reserved


[...]
> +    av_log (NULL,AV_LOG_DEBUG,"%-20s: %12.4f\t%12.4f\n%20.4f\t%d\n",
> +	    "total",s/TelemCnt,s/TelemCnt/64,s,TelemCnt);
> +}

tab

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070613/bf48d22d/attachment.pgp>



More information about the ffmpeg-devel mailing list