On Sat, Jun 15, 2013 at 10:27:06PM +0200, Rafael J. Wysocki wrote:
> On Saturday, June 15, 2013 08:29:42 PM Daniel Vetter wrote:
> > Hi all,
> >
> > So to me it looks like the discussion is going in circles a bit, hence let
> > me drop my maintainer-opinion here:
> >
> > 1. Matthew's patch series
On Saturday, June 15, 2013 08:29:42 PM Daniel Vetter wrote:
> Hi all,
>
> So to me it looks like the discussion is going in circles a bit, hence let
> me drop my maintainer-opinion here:
>
> 1. Matthew's patch series here looks reasonable, and if it fixes a bunch
> of systems (which it seems to)
On Sat, Jun 15, 2013 at 08:29:42PM +0200, Daniel Vetter wrote:
> Aside at the end: If the gnome tool indeed has its own backlight code and
> doesn't just use that as a fallback if the xrandr backligh property isn't
> available, then that's just a serious bug in gnome and should be fixed
> asap. Bu
Hi all,
So to me it looks like the discussion is going in circles a bit, hence let
me drop my maintainer-opinion here:
1. Matthew's patch series here looks reasonable, and if it fixes a bunch
of systems (which it seems to) it has my Ack and imo should go in. If acpi
maintainers can smash their Ac
On Sat, Jun 15, 2013 at 08:29:15PM +0800, Aaron Lu wrote:
> On 06/15/2013 12:19 PM, Matthew Garrett wrote:
> > The vendor will presumably have tested that backlight control works - if
> > the GPU driver uses the ACPI interface and backlight control is broken,
> > then the vendor would fix it.
>
On 06/15/2013 12:19 PM, Matthew Garrett wrote:
> On Sat, Jun 15, 2013 at 12:14:42PM +0800, Aaron Lu wrote:
>> On 06/15/2013 09:38 AM, Matthew Garrett wrote:
>>> Well, Windows 8 will only use the ACPI backlight interface if the GPU
>>> driver decides to, right? So the logic for deciding whether to r
On Wed, 2013-06-05 at 13:34 +0200, Daniel Vetter wrote:
> i9xx doesn't use pre_enable at all, so we can fold this in now.
>
> Signed-off-by: Daniel Vetter
I managed to review 23-28 for now, so on those apart from the two
nitpicks:
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/i915/intel_dis
