From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751738AbaBMPAm (ORCPT ); Thu, 13 Feb 2014 10:00:42 -0500 Received: from mga09.intel.com ([134.134.136.24]:1953 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751246AbaBMPAl (ORCPT ); Thu, 13 Feb 2014 10:00:41 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.95,838,1384329600"; d="scan'208";a="454916961" From: Jani Nikula To: Luis Ortega Cc: daniel.vetter@ffwll.ch, airlied@linux.ie, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [Regression 3.14-rc2] drm/i915: Brightness adjustment is broken in 945GM In-Reply-To: <20140213153458.GA724@Len.lan_dn> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20140213153458.GA724@Len.lan_dn> User-Agent: Notmuch/0.17+61~gab0de040c7c1 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) Date: Thu, 13 Feb 2014 17:04:23 +0200 Message-ID: <87lhxfvytk.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 13 Feb 2014, Luis Ortega wrote: > Hi, testing 3.14-rc2 I noticed I could not adjust the brightness of the > screen any longer. This problem is already present in 3.14-rc1. 3.13 works fine. > > My hardware is a netbook with intel atom and a 945GM graphics card. > > I bisected the problem down to the next commit: I dare say either your bisect went sour or you don't have 945GM. Please verify your steps. Please provide dmesg with drm.debug=0xe with 3.14-rc2. BR, Jani. > > bc0bb9fd1c7810407ab810d204bbaecb255fddde is the first bad commit > commit bc0bb9fd1c7810407ab810d204bbaecb255fddde > Author: Jani Nikula > Date: Thu Nov 14 12:14:29 2013 +0200 > > drm/i915: remove QUIRK_NO_PCH_PWM_ENABLE > > The quirk was added as what I'd say was a stopgap measure in > > commit e85843bec6c2ea7c10ec61238396891cc2b753a9 > Author: Kamal Mostafa > Date: Fri Jul 19 15:02:01 2013 -0700 > > drm/i915: quirk no PCH_PWM_ENABLE for Dell XPS13 backlight > > without really digging into what was going on. > > Also, as mentioned in the related bug [1], having the quirk regressed > some of the machines it was supposed to fix to begin with, and there > were patches posted to disable the quirk on such machines [2]! > > The fact is, we do need the BLM_PCH_PWM_ENABLE bit set to have > backlight. With the quirk, we've relied on BIOS to have set it, and our > save/restore code to retain it. With the full backlight setup at enable, > we have no place for things that rely on previous state. > > With the per platform hooks, we've also made a change in the PCH > platform enable order: setting the backlight duty cycle between CPU and > PCH PWM enable. Some experimenting and > > commit 770c12312ad617172b1a65b911d3e6564fc5aca8 > Author: Takashi Iwai > Date: Sat Aug 11 08:56:42 2012 +0200 > > drm/i915: Fix blank panel at reopening lid > > indicate that we can't set the backlight before enabling CPU PWM; the > value just won't stick. But AFAICT we should do it before enabling the > PCH PWM. > > Finally, any fallout we should fix properly, preferrably without quirks, > and absolutely without quirks that rely on existing state. With the per > platform hooks have much more flexibility to adjust the sequence as > required by platforms. > > [1] https://bugzilla.kernel.org/show_bug.cgi?id=47941 > [2] http://lkml.kernel.org/r/1378229848-29113-1-git-send-email-kamal@canonical.com > > Signed-off-by: Jani Nikula > Reviewed-by: Imre Deak > Signed-off-by: Daniel Vetter > > > Additionally, when I had narrowed the problem to around 11 commits all the > resulting kernel I compiled made the screen blink and shake from left to right > non-stop. In the -rc kernels this doesn't happen. Thought it was worth > mentioning. -- Jani Nikula, Intel Open Source Technology Center