[FFmpeg-devel] [PATCH 06/10] swscale/arm/yuv2rgb: only process one line at a time for the yuv420p and nv{12, 21} formats

[Michael Niedermayer]

On Sat, Mar 26, 2016 at 01:05:56PM +0100, Matthieu Bouron wrote:
> On Sat, Mar 26, 2016 at 2:09 AM, Michael Niedermayer <michael at niedermayer.cc
> > wrote:
> 
> > On Fri, Mar 25, 2016 at 11:46:01PM +0100, Matthieu Bouron wrote:
> > > From: Matthieu Bouron <matthieu.bouron at stupeflix.com>
> > >
> > > ---
> > >  libswscale/arm/yuv2rgb_neon.S | 89
> > ++++++++++++-------------------------------
> > >  1 file changed, 24 insertions(+), 65 deletions(-)
> >
> > breaks build
> >
> >  make distclean ; ../configure --cross-prefix=/usr/arm-linux-gnueabi/bin/
> > --cc='ccache arm-linux-gnueabi-gcc-4.5' --extra-cflags='-mfpu=neon
> > -mfloat-abi=softfp' --cpu=cortex-a8 --arch=armv7 --target-os=linux
> > --enable-cross-compile && make -j12
> >
> > CC      libavutil/arm/float_dsp_init_arm.o
> > src/libswscale/arm/yuv2rgb_neon.S: Assembler messages:
> > src/libswscale/arm/yuv2rgb_neon.S:269: Error: thumb conditional
> > instruction should be in IT block -- `subeq r6,r6,r0'
> > src/libswscale/arm/yuv2rgb_neon.S:269: Error: thumb conditional
> > instruction should be in IT block -- `addne r6,r7'
> >
> 
> [...]
> 
> Patch updated with the relevant it instructions added. It still does build
> on my rpi2 setup but is not tested on the same setup as yours.
> Can you confirm it builds/works on your setup ?

confirmed up to and includng this patch

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160326/b67be2a8/attachment.sig>