Re: [Gnewsense-dev] Re: gNewSense-MIPS and SiliconMotion video driver

[Brett Smith]

On Thu, 2010-03-18 at 11:18 +1030, Karl Goetz wrote:
> > On Mon, 2010-03-08 at 23:40 -0500, Quiliro Ordóñez wrote:
> > > Will this fix come to the repos soon?
> > 
> > I have been working on this a little bit.  The patch applies pretty
> > easily to the SiliconMotion driver that's currently in the metad
> > repository (you just have to change the indentation in one of the
> > patch chunks).  I have a modified Debian source tree with the patch
> > added that gets to the build process cleanly.
> 
> I assume that's the xserver-xorg-video-siliconmotion package.

Yes.

> Are you seeing similar errors to those pasted at the bottom? This is a
> build I attempted on metad and lenny.
>
> ../../src/smi_video.c:1877: warning: unused variable 'pScreen'
> /tmp/ccBlnV2F.s: Assembler messages:
> /tmp/ccBlnV2F.s:4616: Error: illegal operands `xor $f0,$f0,$f0'
> /tmp/ccBlnV2F.s:4617: Error: opcode not supported on this processor: mips1 
> (mips1) `ldc1 $f4,($4)'
> /tmp/ccBlnV2F.s:4618: Error: unrecognized opcode `punpcklbh $f2,$f4,$f0'
> /tmp/ccBlnV2F.s:4619: Error: unrecognized opcode `punpckhbh $f4,$f4,$f0'
> /tmp/ccBlnV2F.s:4620: Error: opcode not supported on this processor: mips1 
> (mips1) `ldc1 $f16,8($4)'

[snip]

Yep, those are the ones.  I even tried this with binutils 2.20 with the
loongson2f patch and still got this problem.

Those opcodes are coming from some assembly code that's included
directly in the patch.  This patch was written specifically for the
Loongson, so I wonder what's going on...

-- 
Brett Smith
Licensing Compliance Engineer, Free Software Foundation