On Fri, 2009-04-03 at 10:43 +0200, Maarten Maathuis wrote: > You need these ports for LVDS? > > Because i know these ports are correct for dvi and friends. > > So if you changed them for lvds, we need to seperate port 4 and 5 into > lvds and non-lvds. Why doesn't xf86-video-nv do this? There's a RH bug (492289) where the ddx doesn't get any modes (on DVI), but kms (which has the old values) does. I didn't actually hear from the reporter yet, but seeing as nv uses the old values I figured that could be a likely culprit...
Did your change actually fix anything? Your commit message was a vague "some minor modesetting changes" seemingly without reason. Ben. > > Maarten. > > On Fri, Apr 3, 2009 at 6:26 AM, Ben Skeggs > <[email protected]> wrote: > > src/nv50reg.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > New commits: > > commit 11be9a982073d66a68cd3db2bfc611eb58d3ea81 > > Author: Ben Skeggs <[email protected]> > > Date: Fri Apr 3 12:22:04 2009 +1000 > > > > nv50: fix i2c port addresses > > > > diff --git a/src/nv50reg.h b/src/nv50reg.h > > index b3125ce..4e21220 100644 > > --- a/src/nv50reg.h > > +++ b/src/nv50reg.h > > @@ -50,8 +50,8 @@ > > #define NV50_PCONNECTOR_I2C_PORT_1 0x0000e150 > > #define NV50_PCONNECTOR_I2C_PORT_2 0x0000e168 > > #define NV50_PCONNECTOR_I2C_PORT_3 0x0000e180 > > -#define NV50_PCONNECTOR_I2C_PORT_4 0x0000e254 > > -#define NV50_PCONNECTOR_I2C_PORT_5 0x0000e274 > > +#define NV50_PCONNECTOR_I2C_PORT_4 0x0000e240 > > +#define NV50_PCONNECTOR_I2C_PORT_5 0x0000e258 > > > > /* 0x00610024 is the state register to read, all it's bits also exist in > > 0x0061002C in the form of interrupt switches. */ > > #define NV50_DISPLAY_SUPERVISOR 0x00610024 > > > > ------------------------------------------------------------------------------ > > _______________________________________________ > > Nouveau-cvs mailing list > > [email protected] > > https://lists.sourceforge.net/lists/listinfo/nouveau-cvs > > _______________________________________________ Nouveau mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/nouveau
