All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Menzel <paulepanter@users.sourceforge.net>
To: dri-devel@lists.freedesktop.org
Cc: Ian Pilcher <arequipeno@gmail.com>, stable@vger.kernel.org
Subject: [PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for Philips 32PFL5404H
Date: Wed, 15 Aug 2012 16:31:07 +0200	[thread overview]
Message-ID: <1345041067.4264.16.camel@mattotaupa> (raw)


[-- Attachment #1.1: Type: text/plain, Size: 2653 bytes --]

Date: Wed, 8 Aug 2012 23:12:19 +0200

< ajax> i would preface this whole discussion with the observation that all tvs are garbage

Connecting the Philips 32PFL5404H [1] a garbled screen is shown with
vertical stripes in the top half.

As written in the referenced Bugzilla #26294 report I am pretty sure
this worked sometime before 2010. My guess is that EDID beforehand was
interpreted incorrectly – as probably MS Windows does – which made it
work.

In commit bc42aabc [2]

        commit bc42aabc6a01b92b0f961d65671564e0e1cd7592
        Author: Adam Jackson <ajax@redhat.com>
        Date:   Wed May 23 16:26:54 2012 -0400

            drm/edid/quirks: ViewSonic VA2026w

Adam Jackson added the quirk `EDID_QUIRK_FORCE_REDUCED_BLANKING` which
is also needed for this Philips TV.

The problem is that the Model number is set to zero. I hope this will
not break other Philips TVs out there.

All log files and output from `xrandr` is included in the referenced
Bugzilla report #26294.

[1] http://www.p4c.philips.com/cgi-bin/dcbint/cpindex.pl?ctn=32PFL5404H/12&scy=DE&slg=de
[2] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commit;h=bc42aabc6a01b92b0f961d65671564e0e1cd7592

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=26294
Tested-by: Paul Menzel <paulepanter@users.sourceforge.net>
    (ASUS Eee PC 701 4G with Debian Sid/unstable connected over VGA)
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Cc: <dri-devel@lists.freedesktop.org>
Cc: Adam Jackson <ajax@redhat.com>
Cc: Ian Pilcher <arequipeno@gmail.com>
Cc: <stable@vger.kernel.org>
---
Ian, I did not base this patch on your series, to make it easier to get
back ported. I can easily rebase it though, so hopefully some maintainer
can tell me what to do.

I also do not know if URLs in the quirk comments are considered useful
or not.
---
 drivers/gpu/drm/drm_edid.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index a8743c3..eb452e6 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -111,6 +111,11 @@ static struct edid_quirk {
 	{ "LPL", 0, EDID_QUIRK_DETAILED_USE_MAXIMUM_SIZE },
 	{ "LPL", 0x2a00, EDID_QUIRK_DETAILED_USE_MAXIMUM_SIZE },
 
+	/* Philips 32PFL5404H TV */
+	/* http://www.p4c.philips.com/cgi-bin/dcbint/cpindex.pl?ctn=32PFL5404H/12&scy=DE&slg=de */
+	/* https://bugs.freedesktop.org/show_bug.cgi?id=26294 */
+	{ "PHL", 0, EDID_QUIRK_FORCE_REDUCED_BLANKING },
+
 	/* Philips 107p5 CRT */
 	{ "PHL", 57364, EDID_QUIRK_FIRST_DETAILED_PREFERRED },
 
-- 
1.7.10.4

[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

             reply	other threads:[~2012-08-15 14:31 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-15 14:31 Paul Menzel [this message]
2012-09-23 16:16 ` [PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for Philips 32PFL5404H Paul Menzel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1345041067.4264.16.camel@mattotaupa \
    --to=paulepanter@users.sourceforge.net \
    --cc=arequipeno@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.