[Xorg-driver-geode] AMD GEODE FreeBSD Driver Update, using avoid-idle.patch (BEFORE MSR is applied)

[Larry Maloney]

Jordon,

I used the avoid-idle.patch and had a result.....

I'm getting ready to install MSR, and wanted to share this before I 
added MSR.

After applying the patch, then exiting X, my graphics still appeared to 
be hung.

Also, (I neglected to mention before) my monitor (CRT) says "Frequency 
too high"

I continued on my way by ssh back into the board.  A few minutes later, 
the VESA
screen saver (I have that enabled on my OS build) kicked in, and VESA 
reset the display.

My console came back after the VESA screen saver displayed the image.

Should I add a VESA reset into the driver when exiting?

Don't know if that helps.

I'm including my latest Xorg.0.log and my dmesg.

I will now add the MSR.

Larry

PS:  I'm mailing this using a Windows e-mail client.  I'm not sure if 
the file formating is looking OK for you guys.  Please tell me the files 
are wordwrapped, and unreadable.
PSS:  DOS CTRL/LF bit me on the avoid-idle.patch.  I had to use unix2dos 
to make the patch work, just an FYI for anybody else who uses a windows 
e-mail client to receive their mail ! :)

Jordan Crouse wrote:
> On 20/12/07 10:17 -0800, Larry Maloney wrote:
>   
>> Jordan,
>>
>> So with the NoAccel True option placed inside the card section, my graphics 
>> work. :)
>>     
>
> Our worst fears realized.
>
>   
>> Everything isn't perfect though.  (As I'm sure you suspect)  If I try to 
>> terminate the X server
>> the graphics are stalled (hung).  I can still access the system via remote 
>> shell.
>>     
>
> Ouch - I see why.   Mia culpa.  Try the attached patch.
>
>   
>> Anyway, I now have a high res screen with many colors (don't know what 
>> color depth I have)
>>     
>
> Looks like 8 bit according to the log.
>
>   
>> I'm including my Xorg.0.log file. 
>> It appears that the VESA drivers are loading.  Is this correct?  I guess 
>> turning off the acceleration
>> enables the probing to work.
>>     
>
> You're not really loading a vesa driver here - you are loading VBE routines
> for probing DDC, which obviously was successful.  Your log shows the same
> sort of VGA interaction that we would expect.
>
> Avoiding acceleration doesn't really affect this one way or the other -
> if VGA is detected,  then we probe - graphics be damned.  This is why
> its different then what Martin-Eric is chasing.
>
>   
>> I will go ahead and add in the debug code you have asked for. 
>>     
>
> That would be good - I'm very worried that this is where the problem is -
> but I'm not sure how to go about fixing it.
>
> Jordan
>   
> ------------------------------------------------------------------------
>
> On shutdown, we loop waiting for the GPU, which isn't needed if
>
> From: Jordan Crouse <jordan.crouse at amd.com>
>
> we don't have accel enabled.
> ---
>
>  src/amd_lx_driver.c |    9 ++++++---
>  1 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/src/amd_lx_driver.c b/src/amd_lx_driver.c
> index 7efda1f..1687752 100644
> --- a/src/amd_lx_driver.c
> +++ b/src/amd_lx_driver.c
> @@ -917,7 +917,8 @@ LXSwitchMode(int index, DisplayModePtr pMode, int flags)
>      int rotate;
>  
>      /* Syn the engine and shutdown the DAC momentarily */
> -    gp_wait_until_idle();
> +    if (!pGeode->NoAccel)
> +	gp_wait_until_idle();
>  
>      /* Set up the memory for the new mode */
>      rotate = LXGetRotation(pScrni->pScreen);
> @@ -949,7 +950,8 @@ LXLeaveGraphics(ScrnInfoPtr pScrni)
>    GeodeRec *pGeode = GEODEPTR(pScrni);
>    VG_PANNING_COORDINATES panning;
>  
> -  gp_wait_until_idle();
> +  if (!pGeode->NoAccel)
> +	gp_wait_until_idle();
>   
>    lx_disable_dac_power(pScrni, DF_CRT_DISABLE);
>  
> @@ -1025,7 +1027,8 @@ LXEnterGraphics(ScreenPtr pScrn, ScrnInfoPtr pScrni)
>  
>    pGeode->VGAActive = gu3_get_vga_active();
>  
> -  gp_wait_until_idle();
> +  if (!pGeode->NoAccel)
> +	gp_wait_until_idle();
>    
>    //lx_disable_dac_power(pScrni, DF_CRT_DISABLE);
>    
>   


-- 
==============================
Larry P. Maloney
CEO/CTO SoftNet Solutions

Phone: 650-669-1942
  Web: www.softnetsolution.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.x.org/archives/xorg-driver-geode/attachments/20071220/dd0f12f7/attachment-0001.html 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: Xorg.0.log
Url: http://lists.x.org/archives/xorg-driver-geode/attachments/20071220/dd0f12f7/attachment-0002.txt 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: messages
Url: http://lists.x.org/archives/xorg-driver-geode/attachments/20071220/dd0f12f7/attachment-0003.txt 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: larrypatrickmaloney.vcf
Type: text/x-vcard
Size: 318 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-driver-geode/attachments/20071220/dd0f12f7/attachment-0001.vcf