Re: [Intel-gfx] [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-12-09 Thread Aaron Lu
On 12/08/2014 07:04 PM, Jani Nikula wrote: > On Mon, 08 Dec 2014, Jani Nikula wrote: >> On Mon, 08 Dec 2014, Aaron Lu wrote: >>> We have a new bug report that has the same problem: >>> https://bugzilla.kernel.org/show_bug.cgi?id=88941 >>> >>> The posted patch solves the problem. I know it's not p

Re: [Intel-gfx] [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-12-08 Thread Jani Nikula
On Mon, 08 Dec 2014, Jani Nikula wrote: > On Mon, 08 Dec 2014, Aaron Lu wrote: >> We have a new bug report that has the same problem: >> https://bugzilla.kernel.org/show_bug.cgi?id=88941 >> >> The posted patch solves the problem. I know it's not perfect, but it >> doesn't seem it would do any har

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-12-08 Thread Jani Nikula
On Mon, 08 Dec 2014, Aaron Lu wrote: > We have a new bug report that has the same problem: > https://bugzilla.kernel.org/show_bug.cgi?id=88941 > > The posted patch solves the problem. I know it's not perfect, but it > doesn't seem it would do any harm to existing systems so should be safe. > > Bet

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-12-07 Thread Aaron Lu
We have a new bug report that has the same problem: https://bugzilla.kernel.org/show_bug.cgi?id=88941 The posted patch solves the problem. I know it's not perfect, but it doesn't seem it would do any harm to existing systems so should be safe. Better, if someone can shed some light on how this sh

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-03-04 Thread Daniel Vetter
On Wed, Feb 19, 2014 at 04:59:06PM +0800, Aaron Lu wrote: > On 02/19/2014 03:33 PM, Matthew Garrett wrote: > > On Wed, Feb 19, 2014 at 03:31:29PM +0800, Aaron Lu wrote: > > > >> DID2 is in system memory region and has some assigned value like 0x400 > >> when we read it. For this case it is easy si

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-19 Thread Aaron Lu
On 02/19/2014 03:33 PM, Matthew Garrett wrote: > On Wed, Feb 19, 2014 at 03:31:29PM +0800, Aaron Lu wrote: > >> DID2 is in system memory region and has some assigned value like 0x400 >> when we read it. For this case it is easy since there is only one output >> device that is of type LVDS so we ca

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-18 Thread Matthew Garrett
On Wed, Feb 19, 2014 at 03:31:29PM +0800, Aaron Lu wrote: > DID2 is in system memory region and has some assigned value like 0x400 > when we read it. For this case it is easy since there is only one output > device that is of type LVDS so we can match it to connector of type eDP > or LVDS, suppose

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-18 Thread Aaron Lu
On 02/13/2014 08:03 PM, Daniel Vetter wrote: > On Thu, Feb 13, 2014 at 11:08 AM, Chris Wilson > wrote: >> On Thu, Feb 13, 2014 at 05:10:25PM +0800, Aaron Lu wrote: >>> On 02/12/2014 06:31 PM, Chris Wilson wrote: On Wed, Feb 12, 2014 at 11:05:40AM +0800, Aaron Lu wrote: > The ACPI table o

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-13 Thread Daniel Vetter
On Thu, Feb 13, 2014 at 11:08 AM, Chris Wilson wrote: > On Thu, Feb 13, 2014 at 05:10:25PM +0800, Aaron Lu wrote: >> On 02/12/2014 06:31 PM, Chris Wilson wrote: >> > On Wed, Feb 12, 2014 at 11:05:40AM +0800, Aaron Lu wrote: >> >> The ACPI table on ASUS UX302LA has more than 8 output devices under

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-13 Thread Chris Wilson
On Thu, Feb 13, 2014 at 05:10:25PM +0800, Aaron Lu wrote: > On 02/12/2014 06:31 PM, Chris Wilson wrote: > > On Wed, Feb 12, 2014 at 11:05:40AM +0800, Aaron Lu wrote: > >> The ACPI table on ASUS UX302LA has more than 8 output devices under the > >> graphics controller device node. The problem is, th

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-13 Thread Aaron Lu
On 02/12/2014 06:31 PM, Chris Wilson wrote: > On Wed, Feb 12, 2014 at 11:05:40AM +0800, Aaron Lu wrote: >> The ACPI table on ASUS UX302LA has more than 8 output devices under the >> graphics controller device node. The problem is, the real active output >> device, the LCD panel, is listed the last.

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-12 Thread Chris Wilson
On Wed, Feb 12, 2014 at 11:05:40AM +0800, Aaron Lu wrote: > The ACPI table on ASUS UX302LA has more than 8 output devices under the > graphics controller device node. The problem is, the real active output > device, the LCD panel, is listed the last. The result is, the LCD's > device id doesn't get

Re: [PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-12 Thread Jani Nikula
On Wed, 12 Feb 2014, Chris Wilson wrote: > On Wed, Feb 12, 2014 at 11:05:40AM +0800, Aaron Lu wrote: >> The ACPI table on ASUS UX302LA has more than 8 output devices under the >> graphics controller device node. The problem is, the real active output >> device, the LCD panel, is listed the last. T

[PATCH] drm/i915/opregion: work around buggy firmware that provides 8+ output devices

2014-02-11 Thread Aaron Lu
The ACPI table on ASUS UX302LA has more than 8 output devices under the graphics controller device node. The problem is, the real active output device, the LCD panel, is listed the last. The result is, the LCD's device id doesn't get recorded in the active device list CADL array and when the _DCS c