[Ffmpeg-devel] [PATCH] fix for PSP encoding

Nick Stewart nick.stewart
Tue Dec 6 16:55:58 CET 2005


>Hi,
>
>On 12/6/05, Nick Stewart <nick.stewart at cambridgeimaging.co.uk> wrote:
>> >> As the author don't update it, and some people asks me about this
>> patch,
>> >> I clean it.
>> >> Note that it is untested.
>> >>
>> >>
>> >> Matthieu
>> >>
>> >>
>> >
>> >This patch is working for i586 and x86_64 and a PSP with 2.0
firmware.
>> >
>> >One small issue it has, with this command line:
>> >
>> >ffmpeg -y -i "small.avi" -title "SMALL" -f psp -r 29.97 -s 368x208
-b
>> >400 -acodec aac -ar 24000 -ab 32 "M4V10034.MP4"
>> >
>> >The output fps is nan ?
>> >
>> >
>> >ffmpeg version CVS, build 3276800, Copyright (c) 2000-2004 Fabrice
>> Bellard
>> >   configuration:  --enable-mp3lame --enable-vorbis --enable-faad
>> >--enable-faadbin --enable-faac --enable-xvid --enable-a52
--enable-dts
>> >--enable-pp --enable-dc1394 --enable-gpl --enable-libogg
>> >--enable-pthreads --cc=gcc -V 3.4
>> >   built on Oct  6 2005 23:19:10, gcc: 3.4.5 20050821 (prerelease)
>> > (Debian 3.4.4-8)
>> >Input #0, avi, from 'small.avi':
>> >   Duration: 02:08:43.7, start: 0.000000, bitrate: 10 kb/s
>> >   Stream #0.0, 29.97 fps: Video: mpeg2video, yuv420p, 720x480, 9800
>> kb/s
>> >   Stream #0.1: Audio: mp3, 48000 Hz, stereo, 32 kb/s
>> >Output #0, psp, to 'M4V10034.MP4':
>> >   Stream #0.0,   nan fps: Video: mpeg4, yuv420p, 368x208, q=2-31,
400
>> kb/s
>> >   Stream #0.1: Audio: aac, 24000 Hz, stereo, 32 kb/s
>> >Stream mapping:
>> >   Stream #0.0 -> #0.0
>> >   Stream #0.1 -> #0.1
>> > [mpeg4 @ 0x7f9930]removing common factors from framerate
>> >Press [q] to stop encoding
>> >frame=  828 q=0.0 Lsize=    1506kB time=27.6 bitrate= 446.6kbits/s
>> >video:1016kB audio:207kB global headers:0kB muxing overhead
23.182991%
>> >
>> >
>> >Angus
>> >
>>
>> What's the current status of this patch? Was there something wrong
with
>> it that has stopped it being submitted to CVS?
>>
>> It appears to work well (tested on PSP 2.0, used mingw to compile)
and
>> without it users can't use CVS ffmpeg for PSP 2.0+ encoding.
>
>Well, the fact that output fps is set to "nan" doesn't seem like a
>good thing. I guess it would be good to fix that problem before
>considering this patch for inclusion.
>
>Guillaume

Well on my mingw build of ffmpeg no matter what
commandline/format/sourcefile I use to encode with the FPS reads "nan". 

So this patch isn't the cause of that for me. Is CVS ffmpeg "output fps"
working fine for everyone else (even on mingw32)? And if so does this
patch break it for "-f psp" files?

Either way, I'll look into resolving it. Any other issues with the
patch?

-Nick





More information about the ffmpeg-devel mailing list