Thread (6 messages) 6 messages, 5 authors, 2011-03-16

Re: [PATCH v2] matroxfb: remove incorrect Matrox G200eV support

From: Yannick Heneault <hidden>
Date: 2011-03-16 20:24:25
Also in: lkml

It impossible that this patch should have work on a system. The patch 
only declare the G200eV as a regular G200 which is not case. Many 
registers are different, including at least the PLL programming 
sequence. If the G200eV is programmed like a regular G200, it will not 
display anything.

Yannick

On 03/11/2011 05:23 PM, Gary Hade wrote:
On Mon, Mar 07, 2011 at 07:59:16PM +1100, Benjamin Herrenschmidt wrote:
   
quoted
On Fri, 2011-03-04 at 12:29 -0800, Darrick J. Wong wrote:
     
quoted
This is all quite strange -- 2.5 years ago when I wrote the patch it
seemed to
work ok.  On newer revisions of the x3650M2 it seems broken.  The
original
machine I wrote it for was cut up ages ago.

I suppose we could simply blacklist any G200eV with a subsystem vendor
ID of
0x1014 (IBM) until we figure out how to correct the driver.  Our
customers will
be deprived, but as it seems to be broken across most of our product
lines I
doubt any of them are making serious use of it anyway. :)

Something like this?
       
Does X work with the open source drivers ?
     
Yes, X works fine on at least the IBM System x boxes that have
the Matrox G200eV.

   
quoted
In that case a better
approach would be to try to figure out what's different between the way
the 2 drivers setup the card registers and fix matroxfb..
     
I suppose someone could attempt this but with vesafb available as
an alternate fb provider and no known demand for repaired G200eV
support in matroxfb, I am not sure what benefit it would provide.

Gary


   
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help