[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3910636.aGHp6YakQl@vostro.rjw.lan>
Date: Fri, 27 Sep 2013 02:26:35 +0200
From: "Rafael J. Wysocki" <rjw@...k.pl>
To: Jörg Otte <jrg.otte@...il.com>
Cc: Jani Nikula <jani.nikula@...ux.intel.com>,
Aaron Lu <aaron.lu@...el.com>,
Igor Gnatenko <i.gnatenko.brain@...il.com>,
ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
intel-gfx <intel-gfx@...ts.freedesktop.org>,
dri-devel@...ts.freedesktop.org,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Daniel Vetter <daniel@...ll.ch>,
Matthew Garrett <matthew.garrett@...ula.com>,
Seth Forshee <seth.forshee@...onical.com>,
Lee Chun-Yi <joeyli.kernel@...il.com>,
Richard Purdie <rpurdie@...ys.net>,
Yves-Alexis Perez <corsac@...ian.org>,
Felipe Contreras <felipe.contreras@...il.com>,
Henrique de Moraes Holschuh <ibm-acpi@....eng.br>,
Ben Jencks <ben@...ncks.net>,
Steven Newbury <steve@...wbury.org.uk>,
James Hogan <james@...anarts.com>,
Kamal Mostafa <kamal@...onical.com>,
Joerg Platte <jplatte@...sa.net>,
Kalle Valo <kvalo@...rom.com>,
Martin Steigerwald <Martin@...htvoll.de>,
Mike Galbraith <bitbucket@...ine.de>
Subject: Re: [PATCH v3 0/4] Fix Win8 backlight issue
On Thursday, September 26, 2013 09:49:03 AM Jörg Otte wrote:
> 2013/9/25 Jani Nikula <jani.nikula@...ux.intel.com>:
> > On Wed, 25 Sep 2013, Jörg Otte <jrg.otte@...il.com> wrote:
> >> 2013/9/25 Jani Nikula <jani.nikula@...ux.intel.com>:
> >>> On Wed, 25 Sep 2013, Aaron Lu <aaron.lu@...el.com> wrote:
> >>>> On Wed, Sep 25, 2013 at 10:29:37AM +0200, Jörg Otte wrote:
> >>>>> Backlight can't be modified with this patch set - neither with
> >>>>> function keys nor with the gui. It is a step backward to v3.11-rc1
> >>>
> >>> So both hotkeys and GUI work in v3.11-rc1? And v3.12-rc2?
> >>
> >> In v3.11-rc1 it didn't work.
> >> Since a later rc (I don't remember exactly which) it did work.
> >> In v3.12-rc1/2 it does work.
> >> In v3.12-rc2 + Aaron's patch set it again doesn't work.
> >>
> >>>
> >>>> Thanks for the test.
> >>>>
> >>>> Please check /sys/class/backlight, is there only one link file
> >>>> intel_backlight?
> >>>
> >>> Indeed, are there others? fujitsu-laptop perhaps? If yes, try
> >>> CONFIG_FUJITSU_LAPTOP=n or an appropriate module blacklist.
> >>>
> >>> Just checking, you do have CONFIG_BACKLIGHT_CLASS_DEVICE=y?
> >>
> >> There is only one intel_backlight link.
> >> Yes, I have CONFIG_BACKLIGHT_CLASS_DEVICE=y
> >>
> >>> Embarrassingly there was a bug in i915 fixed just recently where the
> >>> backlight device wasn't registered for
> >>> CONFIG_BACKLIGHT_CLASS_DEVICE=m...
> >>>
> >>>> If so, please cd inside and try modify the brightness file using echo
> >>>> with some values in the range of 0 - max_brightness, does the
> >>>> backlight level change when you echo a new value? I guess it doesn't,
> >>>> or you wouldn't notice problem. If indeed so, perhaps file a bug at
> >>>> http://bugzilla.kernel.org, Drivers/Video(DRI-Intel)? I suppose Jani
> >>>> and Daniel can help fix your problem.
> >>>
> >>> Yup, please check the above, and report back.
> >>
> >> - echo 0..max_brightness > brightness does not work.
> >
> > Thanks for the info.
> >
> > How about v3.12-rc2 without Aaron's patches? Does intel_backlight also
> > not work there? How about the acpi_video0 (which I presume is present)
> > sysfs interface?
> >
>
> Without Aaron's patches intel_backlight also does not work.
> But acpi_video0 does (and takes precedence I think).
So can you please open a bug entry at bugzilla.kernel.org against i915?
The fact that ACPI video works for you doesn't mean that intel_backlight
shouldn't, I suppose?
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists