linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: joeyli <jlee@suse.com>
To: Fabio Coatti <fabio.coatti@gmail.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>,
	Bob Moore <robert.moore@intel.com>,
	Feng Tang <feng.tang@intel.com>, Len Brown <len.brown@intel.com>,
	Tom St Denis <tstdenis@elliptictech.com>,
	linux-kernel@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: Re: 3.7.X vs 3.6.11 HP Folio 9470m brightness regression?
Date: Mon, 14 Jan 2013 17:50:59 +0800	[thread overview]
Message-ID: <1358157059.14569.16.camel@linux-s257.site> (raw)
In-Reply-To: <CADpTngVJqvXfgnd2cMRFL49nt9so+dcwKos8cSLPRj6L0eEPMg@mail.gmail.com>

於 一,2013-01-14 於 09:13 +0100,Fabio Coatti 提到:
> While looking at a different issue (rfkill), related to this commit:
> ====
> > a57f7f9175b8ccbc9df83ac13860488913115de4 is the first bad commit
> > commit a57f7f9175b8ccbc9df83ac13860488913115de4
> > Author: Bob Moore <robert.moore@intel.com>
> > Date: Fri Aug 17 10:55:02 2012 +0800
> >
> > ACPICA: Add Windows8/Server2012 string for _OSI method.
> > This change adds a new _OSI string, "Windows 2012" for both Windows 8
> > and Windows Server 2012.
> > >From Microsoft document "How to Identify the Windows Version in ACPI
> > by Using _OSI", July 13, 2012.
> >
> > Signed-off-by: Bob Moore <robert.moore@intel.com>
> > Signed-off-by: Feng Tang <feng.tang@intel.com>
> > Signed-off-by: Len Brown <len.brown@intel.com>
> ====
> 
> I discovered that reverting this commit (or, as suggested by joeyli
> <jlee@suse.com>, booting with acpi_osi="!Windows 2012" ) also fixes
> the brightness issue. So it seems related to the same ACPI issue, now
> I wonder how it's happened that I got the wrong commit while
> bisecting. Sorry for the noise, now the issue is still there but makes
> more sense ):)

Use "!Windows 2012" can fallback to Windows 2009 mode for workaround
rkfill button and brightness issue on your machine, but we still have
problem with Windows 2012 mode. 

That means we need spend more time to trace the dsdt stuff for guess
what's behavior on HP machine with Windows 8.


Thanks a lot!
Joey Lee

> 
> 
> 
> 2013/1/12 Fabio Coatti <fabio.coatti@gmail.com>:
> > well, I tried to do that, bisecting between 3.6.10 and 3.7. I'm not
> > sure to have done that properly, as the result seems quite harmless,
> > however here it is:
> >
> > d978348b300595b067b80980f6a78db09fcd584c is the first bad commit
> > commit d978348b300595b067b80980f6a78db09fcd584c
> > Author: Bob Moore <robert.moore@intel.com>
> > Date:   Fri Aug 17 10:51:47 2012 +0800
> >
> >     ACPICA: Split ACPICA initialization functions to new file, utxfinit.c
> >
> >     Split utxface.c to improve modularity and reduce file size.
> >
> >     Signed-off-by: Bob Moore <robert.moore@intel.com>
> >     Signed-off-by: Feng Tang <feng.tang@intel.com>
> >     Signed-off-by: Len Brown <len.brown@intel.com>
> >
> > :040000 040000 57bc8451026afe02fdbc2970b152164ff4e5412a
> > 55311ce2f9fed66501b06f93f67fa99b54d46e86 M      drivers
> >
> > Please let me know If I've done something wrong or if I can help in other ways.
> >
> > Many thanks for the hint.
> >
> >
> > 2013/1/12 Daniel Vetter <daniel.vetter@ffwll.ch>:
> >> On Sat, Jan 12, 2013 at 12:22 PM, Fabio Coatti <fabio.coatti@gmail.com> wrote:
> >>> I'm seeing the same behaviour on 3.7.2; even
> >>> echo "NN" > brightness has no effect.
> >>> I've tried to look at changes between 3.6.11 and 3.7.2 but probably
> >>> this is beyond my skills ;)
> >>
> >> If the regression is with the acpi_video backlight, then it's very
> >> likely not a bug in drm/i915. Quickest way to figure that out is to do
> >> a git bisect between 3.6 and 3.7, my favourite howto is:
> >> http://www.reactivated.net/weblog/archives/2006/01/using-git-bisect-to-find-buggy-kernel-patches/
> >>
> >> Cheers, Daniel
> >>
> >>> 2012/12/20 Fabio Coatti <fabio.coatti@gmail.com>:
> >>>> In data mercoledì 19 dicembre 2012 17:10:29, hai scritto:
> >>>>> ----- Original Message -----
> >>>>>
> >>>>> > From: "Fabio Coatti" <fabio.coatti@gmail.com>
> >>>>> > To: linux-kernel@vger.kernel.org
> >>>>> > Sent: Wednesday, 19 December, 2012 5:01:30 PM
> >>>>> > Subject: 3.7.X vs 3.6.11 HP Folio 9470m brightness regression?
> >>>>> >
> >>>>> > Hi,
> >>>>> > I'm testing kernels 3.7.X vs 3.6.11 on an HP Folio 9470m machine,
> >>>>> > i915
> >>>>> > driver for a VGA compatible controller: Intel Corporation 3rd Gen
> >>>>> > Core
> >>>>> > processor Graphics Controller (rev 09) 8086:0166
> >>>>> >
> >>>>> > With 3.6.11, the brightness can be correctly set using fn+f9/10: kde
> >>>>> > shows the bar and brightness changes.
> >>>>> >
> >>>>> > With 3.7.1 kde shows the percentage bar and
> >>>>> > /sys/class/backlight/acpi_video0/brightness is updated, but the
> >>>>> > screen
> >>>>> > brightness does not changes.
> >>>>>
> >>>>> I had a similar problem with my Dell laptop and i915/acpi...
> >>>>>
> >>>>> Is there another acpi_video? device?  In my case two showed up in 3.7
> >>>>> whereas in 3.6 there was one.  In 3.7 it was controlling the wrong one
> >>>>> apparently.
> >>>>
> >>>> Well, this seems not to be the case, I have only one acpi_video:
> >>>>
> >>>> calvin ~ # ls -l /sys/class/backlight/
> >>>> totale 0
> >>>> lrwxrwxrwx 1 root root 0 20 dic 09.20 acpi_video0 ->
> >>>> ../../devices/pci0000:00/0000:00:02.0/backlight/acpi_video0
> >>>>
> >>>> calvin ~ # ls -l /sys/devices/pci0000:00/0000:00:02.0/backlight
> >>>> totale 0
> >>>> drwxr-xr-x 3 root root 0 20 dic 09.20 acpi_video0
> >>>>
> >>>>
> >>>>
> >>>> --
> >>>> Fabio
> >>>
> >>>
> >>>
> >>> --
> >>> Fabio
> >>
> >>
> >>
> >> --
> >> Daniel Vetter
> >> Software Engineer, Intel Corporation
> >> +41 (0) 79 365 57 48 - http://blog.ffwll.ch
> >
> >
> >
> > --
> > Fabio
> 
> 
> 



  reply	other threads:[~2013-01-14  9:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-19 22:01 3.7.X vs 3.6.11 HP Folio 9470m brightness regression? Fabio Coatti
2012-12-19 22:10 ` Tom St Denis
2012-12-20  9:27   ` Fabio Coatti
2013-01-12 11:22     ` Fabio Coatti
2013-01-12 12:42       ` Daniel Vetter
2013-01-12 20:35         ` Fabio Coatti
2013-01-14  8:13           ` Fabio Coatti
2013-01-14  9:50             ` joeyli [this message]
2013-01-14 11:26               ` Fabio Coatti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1358157059.14569.16.camel@linux-s257.site \
    --to=jlee@suse.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=fabio.coatti@gmail.com \
    --cc=feng.tang@intel.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=len.brown@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robert.moore@intel.com \
    --cc=tstdenis@elliptictech.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).