On Thu, 23 Sep 2010 20:36:47 +0200, Hette Visser wrote:
> This patch fixes the black screen bug on Dell e6510:
> https://bugs.freedesktop.org/show_bug.cgi?id=29278
>
> It adds two delays to give the eDP panel time to turn on before we
> continue with the next write.
Can you please base your patc
On Thu, 23 Sep 2010 12:20:02 -0700, Simon Que wrote:
> I've subscribed to the intel-gfx list and am re-sending my email.
> Please see below, thanks.
It's probably a safe assumption that the vast majority of panels will want
to have dithering enabled, so just rearrange the code to set the defaults
I've subscribed to the intel-gfx list and am re-sending my email.
Please see below, thanks.
Simon
-- Forwarded message --
From:
Date: Thu, Sep 23, 2010 at 12:10 PM
Subject: [PATCH] CHROMIUM: i915: Added default LVDS options for the no-VBT case
To: s...@chromium.org
Your messa
On Thu, 23 Sep 2010 20:36:47 +0200
Hette Visser wrote:
> This patch fixes the black screen bug on Dell e6510:
> https://bugs.freedesktop.org/show_bug.cgi?id=29278
>
> It adds two delays to give the eDP panel time to turn on before we
> continue with the next write.
>
>
> --- drivers/gpu/drm/i9
This patch fixes the black screen bug on Dell e6510:
https://bugs.freedesktop.org/show_bug.cgi?id=29278
It adds two delays to give the eDP panel time to turn on before we
continue with the next write.
--- drivers/gpu/drm/i915/intel_dp.c.orig2010-09-23 20:13:36.046856913
+0200
+++ driver
2010/9/23 Jeremy Huddleston :
> That seems off to me. This is doing more than changing the c->next
> dereference. You're now freeing it where you weren't before.
>
> Previously, you freed it inside:
> if (c->isCurrent && (c->drawPriv == glxPriv || c->readPriv == glxPriv))
> if(!c->idExists)
>
On Thu, 23 Sep 2010 09:04:11 -0400, Kristian Høgsberg
wrote:
> Signed-off-by: Kristian Høgsberg
Now that is starting to look familiar ;-)
Reported-by: Julien Cristau
Tested-by: Chris Wilson
--
Chris Wilson, Intel Open Source Technology Centre
_
Signed-off-by: Kristian Høgsberg
---
Chris Wilson points out that we were still accessing c->next after free.
Here's an updated version that fixes that.
Kristian
glx/glxext.c | 11 +--
1 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/glx/glxext.c b/glx/glxext.c
index e2
Signed-off-by: Kristian Høgsberg
---
> ==2989== Invalid write of size 4
> ==2989==at 0x48CE6E5: DrawableGone (glxext.c:169)
> ==2989==by 0x809F401: FreeResource (resource.c:601)
> ==2989==by 0x80845CE: ProcDestroyWindow (dispatch.c:733)
> ==2989==by 0x8087D76: Dispatch (dispatch.c
- "Carl Worth" wrote:
> This is the first release candidate in preparation for the upcoming
> 2.13.0 release. We will appreciate any feedback we can get from
> testing of this snapshot.
>
> As can be seen below, Chris Wilson in responsible for a huge majority
> of
> the bug fixes in 2.12.90
10 matches
Mail list logo
