Attachment 209521 Details for Bug 310331 – add xf86PruneDriverModes after xf86ValidateModes.

[patch] add xf86PruneDriverModes after xf86ValidateModes.

fix_hang.patch (text/plain), 1.29 KB, created by LisaWu on 2007-09-28 05:35:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: LisaWu

Created: 2007-09-28 05:35:16 UTC

Size: 1.29 KB

patch

obsolete

>--- src/radeon_driver.c	2007-09-29 05:31:41.000000000 +0800
>+++ src/radeon_driver.c-mod	2007-09-29 05:32:00.000000000 +0800
>@@ -3941,6 +3941,14 @@
> 					  info->FbMapSize,
> 					  LOOKUP_BEST_REFRESH);
> 
>+			if (modesFound == -1) return FALSE;
>+			/*prune invalid mode after xf86ValidateModes*/
>+			xf86PruneDriverModes(pScrn);
>+			if (!modesFound || !pScrn->modes) {
>+				xf86DrvMsg(pScrn->scrnIndex, X_ERROR, "No valid modes found\n");
>+				return FALSE;
>+			}  
>+
> 	    } 
>         }
> 
>@@ -4203,7 +4211,7 @@
> 		 * mode using xf86ValidateModes routine
> 		 * This can happen when DDC is disabled.
> 		 */
>-		if (info->PanelXRes < 320 || info->PanelYRes < 200)
>+		if (info->PanelXRes < 320 || info->PanelYRes < 200){
> 		    modesFound =
> 			xf86ValidateModes(pScrn,
> 					  pScrn->monitor->Modes,
>@@ -4220,6 +4228,16 @@
> 					  pScrn->display->virtualY,
> 					  info->FbMapSize,
> 					  LOOKUP_BEST_REFRESH);
>+             
>+			if (modesFound == -1) return FALSE;
>+             /*prune invalid mode after xf86ValidateModes*/
>+			 xf86PruneDriverModes(pScrn);
>+			 if (!modesFound || !pScrn->modes) {
>+				 xf86DrvMsg(pScrn->scrnIndex, X_ERROR, "No valid modes found\n");
>+				 return FALSE;
>+			 }
>+
>+		}
> 		else if (!info->IsSecondary)
> 		    modesFound = RADEONValidateFPModes(pScrn, pScrn->display->modes);
> 	    }

Actions: View | Diff

Attachments on bug 310331: 209521