Bug 19301 - No more 1280x1024 with xorg-x11-drv-geode-2.11.0-1.fc10.i386
Summary: No more 1280x1024 with xorg-x11-drv-geode-2.11.0-1.fc10.i386
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/geode (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-27 03:33 UTC by billiboy
Modified: 2010-10-27 00:31 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Used xorg.conf (1.24 KB, text/plain)
2008-12-27 03:35 UTC, billiboy
no flags Details
Xorg.0.log.geode-2.10.1 of working driver (28.81 KB, text/plain)
2008-12-27 03:35 UTC, billiboy
no flags Details
Xorg.0.log.geode-2.11.0 of nonworking driver (26.62 KB, text/plain)
2008-12-27 03:36 UTC, billiboy
no flags Details
Desktop snapshot with autodetected mode (159.43 KB, image/png)
2010-08-28 03:45 UTC, billiboy
no flags Details

Description billiboy 2008-12-27 03:33:47 UTC
No more 1280x1024 with xorg-x11-drv-geode-2.11.0-1.fc10.i386                                                                                                                         
                                                                                                                                                                                     
Boot up F10-i686-Live-KDE                                                                                                                                                            
replace /etc/X11/xorg.conf with attached xorg.conf                                                                                                                                   
(because hardwired xorg.conf for geode)                                                                                                                                              
Leave -> Logout to restart X                                                                                                                                                         
=> Xorg.0.log.geode-2.10.1                                                                                                                                                           
Mode 1280x1024 is working as expected                                                                                                                                                
yum update xorg-drv-geode                                                                                                                                                            
restart X                                                                                                                                                                            
=> Xorg.0.log.geode-2.11.0                                                                                                                                                           
Mode 1280x1024 no longer working                                                                                                                                                     
                                                                                                                                                                                     
In  Xorg.0.log.geode-2.10.1 is this:                                                                                                                                                 
                                                                                                                                                                                     
(II) GEODE(0): Printing DDC gathered Modelines:                                                                                                                                      
(II) GEODE(0): Modeline "1280x1024"x0.0  108.00  1280 1328 1440 1688  1024 1025 1028 1066 +hsync +vsync (64.0 kHz)                                                                   
                                                                                                                                                                                     
In Xorg.0.log.geode-2.11.0 is this:                                                                                                                                                  
                                                                                                                                                                                     
(II) GEODE(0): Printing probed modes for output default                                                                                                                              
(II) GEODE(0): Modeline "1028x1024"x60.0  108.00  1028 1328 1440 1688  1024 1025 1028 1066 -hsync -vsync (64.0 kHz)                                                                  
                                                                                                                                                                                     
Can 1028 be a typo of 1280?
Comment 1 billiboy 2008-12-27 03:35:12 UTC
Created attachment 21502 [details]
Used xorg.conf
Comment 2 billiboy 2008-12-27 03:35:50 UTC
Created attachment 21503 [details]
Xorg.0.log.geode-2.10.1 of working driver
Comment 3 billiboy 2008-12-27 03:36:28 UTC
Created attachment 21504 [details]
Xorg.0.log.geode-2.11.0 of nonworking driver
Comment 4 billiboy 2009-01-27 06:54:24 UTC
because off no response till now I downloaded xf86-video-geode-2.11.0.tar.gz
and did a grep for 1028.
in lx_panel.c I found this code-snippet

DisplayModeRec lx_panel_modes[] = {
    {MODEPREFIX, 31200, 320, 354, 384, 400, 0, 240, 249, 253, 260, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 320x200@75 */
    {MODEPREFIX, 25175, 640, 656, 744, 800, 0, 480, 490, 492, 525, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 640x480@60 */
    {MODEPREFIX, 40000, 800, 840, 968, 1056, 0, 600, 601, 605, 628, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 880x600@60 */
    {MODEPREFIX, 65000, 1024, 1048, 1184, 1344, 0, 768, 771, 777, 806, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1024x768@60 */
    {MODEPREFIX, 81600, 1152, 1216, 1336, 1520, 0, 864, 865, 868, 895, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1152x864@60 */
    {MODEPREFIX, 108000, 1028, 1328, 1440, 1688, 0, 1024, 1025, 1028, 1066, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1280x1024@60 */
    {MODEPREFIX, 162000, 1600, 1664, 1856, 2160, 0, 1200, 1201, 1204, 1250, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1600x100@60 */
};

by logical comparing the entries there are possible typos

DisplayModeRec lx_panel_modes[] = {
    {MODEPREFIX, 31200, 320, 354, 384, 400, 0, 240, 249, 253, 260, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 320x200@75 */

--------------------------------------------------^^
possible typo, should be mayby 60


    {MODEPREFIX, 25175, 640, 656, 744, 800, 0, 480, 490, 492, 525, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 640x480@60 */
    {MODEPREFIX, 40000, 800, 840, 968, 1056, 0, 600, 601, 605, 628, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 880x600@60 */
    {MODEPREFIX, 65000, 1024, 1048, 1184, 1344, 0, 768, 771, 777, 806, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1024x768@60 */
    {MODEPREFIX, 81600, 1152, 1216, 1336, 1520, 0, 864, 865, 868, 895, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1152x864@60 */
    {MODEPREFIX, 108000, 1028, 1328, 1440, 1688, 0, 1024, 1025, 1028, 1066, 0,

-------------------------^^^^
possible critical typo, should be mayby 1280

        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1280x1024@60 */
    {MODEPREFIX, 162000, 1600, 1664, 1856, 2160, 0, 1200, 1201, 1204, 1250, 0,
        V_NHSYNC | V_NVSYNC, MODESUFFIX}
    ,                                  /* 1600x100@60 */

-----------------------------------------------^^^
possible typo, should be mayby 1200
};

I am a stupid user and not knowing what I am doing, so can please
a developer confirm my observations?
Comment 5 Karl Vogel 2009-08-07 05:56:45 UTC
Had the same issue when going from 2.10.1 to 2.11.3 - It seems the way to configure modelines for panels was changed.

You now need to add the modeline for the panel in the DEVICE section as:

--
Option "PanelMode" "108000 1280 1328 1440 1688 1024 1025 1028 1066"
--

instead of a Modeline in the MONITOR section.
Comment 6 Martin-Éric Racine 2010-08-23 09:23:23 UTC
Does this bug still apply to Geode 2.11.9 or not?
Comment 7 billiboy 2010-08-28 03:45:40 UTC
Created attachment 38236 [details]
Desktop snapshot with autodetected mode

This bug does still apply to Geode 2.11.9
See attachment: "Desktop snapshot with autodetected mode"
for reference.
Comment 8 frank huang 2010-10-27 00:25:13 UTC
A patch for this typo error has been commmitted. Can we close this issue?
http://cgit.freedesktop.org/xorg/driver/xf86-video-geode/commit/?id=becaa0ae375e996c2f83192bb84a5c89f94933dd

Thanks,
Frank
Comment 9 Martin-Éric Racine 2010-10-27 00:31:10 UTC
Yes, we can close it, since the fix has been committed to GIT.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.