Re: [PATCH] drm/edid: Don't pretend (badly) to support composite sync

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 21, 2011 at 10:12 AM, Adam Jackson <ajax@xxxxxxxxxx> wrote:
> If hardware does require composite sync for a mode, it's out of luck
> with current drivers, since none support that.  Just skip the mode
> instead, there'll be others in the EDID.  (Unless there's not, in which
> case, failing is correct anyway.)

In theory radeon supports it, but I've never tried it.

Alex

>
> While we're at it, hush the other mode support messages behind
> DRM_DEBUG_KMS, they're too chatty for normal dmesg.
>
> Signed-off-by: Adam Jackson <ajax@xxxxxxxxxx>
> ---
>  drivers/gpu/drm/drm_edid.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 7425e5c..a15f10b 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -849,11 +849,12 @@ static struct drm_display_mode *drm_mode_detailed(struct drm_device *dev,
>                return NULL;
>
>        if (pt->misc & DRM_EDID_PT_STEREO) {
> -               printk(KERN_WARNING "stereo mode not supported\n");
> +               DRM_DEBUG_KMS("stereo mode not supported\n");
>                return NULL;
>        }
>        if (!(pt->misc & DRM_EDID_PT_SEPARATE_SYNC)) {
> -               printk(KERN_WARNING "composite sync not supported\n");
> +               DRM_DEBUG_KMS("composite sync not supported\n");
> +               return NULL;
>        }
>
>        /* it is incorrect if hsync/vsync width is zero */
> --
> 1.7.6
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux