Bug 520408

Summary: 720p not detected over HDMI
Product: [Fedora] Fedora Reporter: Jaroslav Šmíd <jardasmid>
Component: kernelAssignee: Kernel Maintainer List <kernel-maint>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: itamar, kernel-maint, mschmidt
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-09-14 06:14:13 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Patch
none
Patch to parse detailed time info in CAE-EDID none

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.