Bug 520408 - 720p not detected over HDMI
Summary: 720p not detected over HDMI
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ---
Assignee: Kernel Maintainer List
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-08-31 11:27 UTC by Jaroslav Šmíd
Modified: 2009-09-14 11:13 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-09-14 06:14:13 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Patch (26.35 KB, patch)
2009-08-31 11:29 UTC, Jaroslav Šmíd
no flags Details | Diff
Patch to parse detailed time info in CAE-EDID (5.51 KB, patch)
2009-09-01 11:37 UTC, Jaroslav Šmíd
no flags Details | Diff

Description Jaroslav Šmíd 2009-08-31 11:27:34 UTC
Description of problem:
720p resolution is not detected for TV using HDMI output. More info in upstream bugreport (https://bugs.freedesktop.org/show_bug.cgi?id=23098)

There is already a solution for that problem, but the patch they provided me with seem to be for 2.6.32 version of linux kernel, although I managed to get it applied for 2.6.31rc8, but I had to apply it on top of other patches (also mentioned in upstream bugreport).

What I want is to ask you if those patches could be included in next Fedora release (12). I created patch composed all of those in original bugreport or you could use those patches one by one. They doesn't seem to break things, they just "fix" modes detection based on EDID. I think they could be safely included.

Thanks.

Comment 1 Jaroslav Šmíd 2009-08-31 11:29:41 UTC
Created attachment 359251 [details]
Patch

Comment 2 Chuck Ebbert 2009-08-31 16:14:35 UTC
The fedora kernel already has a large part of the upstream code for 2.6.32 applied. Here are the results of trying to apply your patch:

patching file drivers/gpu/drm/drm_edid.c
Hunk #1 FAILED at 60.
Hunk #2 FAILED at 246.
Hunk #3 FAILED at 279.
Hunk #4 succeeded at 524 (offset 38 lines).
Hunk #5 FAILED at 549.
Hunk #6 FAILED at 561.
Hunk #7 FAILED at 586.
Hunk #8 FAILED at 618.
Hunk #9 succeeded at 615 (offset 8 lines).
Hunk #10 succeeded at 820 (offset 38 lines).
Hunk #11 succeeded at 942 (offset 8 lines).
7 out of 11 hunks FAILED -- saving rejects to file drivers/gpu/drm/drm_edid.c.rej
patching file drivers/gpu/drm/drm_modes.c
Hunk #1 FAILED at 62.
1 out of 1 hunk FAILED -- saving rejects to file drivers/gpu/drm/drm_modes.c.rej
patching file include/drm/drm_crtc.h
Hunk #1 FAILED at 736.
1 out of 1 hunk FAILED -- saving rejects to file include/drm/drm_crtc.h.rej

Comment 3 Jaroslav Šmíd 2009-09-01 02:21:18 UTC
My patch is for vanilla 2.6.31rc8, I suppose you applied it on patched kernel (?). I'll download fedora's src.rpm for 2.6.31 and make new patch.

Comment 4 Jaroslav Šmíd 2009-09-01 11:35:11 UTC
I can see that most of the patch is really already included in fedora's rpm (in drm-next.patch). Only this (http://www.mail-archive.com/dri-devel@lists.sourceforge.net/msg42703.html) part is not there.

Comment 5 Jaroslav Šmíd 2009-09-01 11:37:52 UTC
Created attachment 359374 [details]
Patch to parse detailed time info in CAE-EDID

Comment 6 Jaroslav Šmíd 2009-09-12 19:23:33 UTC
ping

Comment 7 Michal Schmidt 2009-09-14 06:14:13 UTC
Jaroslav,
Your proposed patch seems to be already included in the Fedora kernel as a part of drm-next.patch since:
* Tue Sep 08 2009 Dave Airlie <airlied> 2.6.31-0.206.rc9.git1
- drm: rebase to drm-next - r600 accel + kms should start working now

I am closing this bug. Please reopen if you can still reproduce the problem.

Comment 8 Jaroslav Šmíd 2009-09-14 11:13:09 UTC
Ok, thanks.


Note You need to log in before you can comment on or make changes to this bug.