On Wed, Sep 21, 2011 at 10:12 AM, Adam Jackson <ajax at redhat.com> 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 at redhat.com>
> ---
> ?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 at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>

Reply via email to