[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg
Subject:    Re: [PATCH] xfree86: Edid quirk for Philips LCD LP154W01
From:       ykzhao <yakui.zhao () intel ! com>
Date:       2009-11-20 5:55:15
Message-ID: 1258696515.3823.20.camel () localhost ! localdomain
[Download RAW message or body]

On Wed, 2009-11-18 at 18:04 +0800, Zhao, Yakui wrote:
> From: Zhao Yakui <yakui.zhao@intel.com>
> 
> Move the EDID quirk for Philips LCD LP154W01 as the panel reports the vertical
> size in cm.
Hi, Keith/Ajax
   How about this patch?
   The incorrect quirk is applied to Philips LCD(154W01), which causes
that the incorrect DPI is obtained. In such case the user will get the
incorrect display.

thanks.
   Yakui
> 
> https://bugs.freedesktop.org/show_bug.cgi?id=24482
> 
> Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
> ---
>  hw/xfree86/modes/xf86EdidModes.c |   10 ++++------
>  1 files changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/hw/xfree86/modes/xf86EdidModes.c b/hw/xfree86/modes/xf86EdidModes.c
> index 449078e..356e51e 100644
> --- a/hw/xfree86/modes/xf86EdidModes.c
> +++ b/hw/xfree86/modes/xf86EdidModes.c
> @@ -131,8 +131,11 @@ static Bool quirk_detailed_v_in_cm (int scrnIndex, xf86MonPtr DDC)
>  	return TRUE;
>  
>      /* Bug #21000: LGPhilipsLCD LP154W01-TLAJ */
> +    /* Bug #10304: LGPhilipsLCD LP154W01-A5 */
>      if (memcmp (DDC->vendor.name, "LPL", 4) == 0 &&
> -	DDC->vendor.prod_id == 47360)
> +	(DDC->vendor.prod_id == 47360 ||
> +		DDC->vendor.prod_id == 0 ||
> +		DDC->vendor.prod_id == 0x2a00))
>  	return TRUE;
>  
>      /* Bug #21750: Samsung Syncmaster 2333HD */
> @@ -145,11 +148,6 @@ static Bool quirk_detailed_v_in_cm (int scrnIndex, xf86MonPtr DDC)
>  
>  static Bool quirk_detailed_use_maximum_size (int scrnIndex, xf86MonPtr DDC)
>  {
> -    /* Bug #10304: LGPhilipsLCD LP154W01-A5 */
> -    if (memcmp (DDC->vendor.name, "LPL", 4) == 0 &&
> -	(DDC->vendor.prod_id == 0 || DDC->vendor.prod_id == 0x2a00))
> -	return TRUE;
> -
>      /* Bug #21324: Iiyama Vision Master 450 */
>      if (memcmp (DDC->vendor.name, "IVM", 4) == 0 &&
>  	DDC->vendor.prod_id == 6400)

_______________________________________________
xorg mailing list
xorg@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/xorg
[prev in list] [next in list] [prev in thread] [next in thread]