[PATCH] [Maverick] drm/radeon: Update LVDS connector edid_property only once

[Chase Douglas]

On Thu, 2010-06-24 at 16:58 +0200, Alberto Milone wrote:
> Hi all,
> 
> After looking at Arjan's patch to cache the EDID for the intel driver,
> I decided to write a similar patch which should help us reduce boot
> time with radeon.
> 
> Please consider adopting my patch in Maverick's kernel.
> 
> Note: my patch is not the upstream code, read here for further details:
> http://lists.freedesktop.org/archives/dri-devel/2010-May/000948.html

Do you know why it's not upstream? I didn't see any replies to your
post. If it's just a matter of not receiving any replies, it sometimes
takes a resend or two for people to notice.

The patch seems to make sense, but I would be worried about adding a
commit that diverges us from upstream. As we've seen in places like the
dell x86 platform driver, as soon as we start making ubuntu-only changes
that don't get upstream, ongoing maintenance becomes a larger burden.
There's still time to get upstream to pick up the patch before we need
to lock down the kernel for feature freeze.

I'm curious how much time this saves. Do you have any statistics?

Thanks,

-- Chase