From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751430AbdBXI6Z (ORCPT ); Fri, 24 Feb 2017 03:58:25 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:35014 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751005AbdBXI6S (ORCPT ); Fri, 24 Feb 2017 03:58:18 -0500 Date: Fri, 24 Feb 2017 09:58:14 +0100 From: Pali =?utf-8?B?Um9ow6Fy?= To: Hans de Goede Cc: Pavel Machek , Richard Purdie , Jacek Anaszewski , linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org, Darren Hart , platform-driver-x86@vger.kernel.org Subject: Re: LED devices & poll() for brightness attribute Message-ID: <20170224085814.GB2157@pali> References: <20170222090630.GB21558@pali> <20170223144852.GD24201@pali> <20170223202333.GA19376@amd> <201702232144.54558@pali> <20170223210321.GB19376@amd> <20170224085055.GA2157@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23.1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 24 February 2017 09:52:11 Hans de Goede wrote: > Hi, > > On 24-02-17 09:50, Pali Rohár wrote: > >On Thursday 23 February 2017 22:03:21 Pavel Machek wrote: > >>So, if you seriously want to do this, can you > >> > >>1) write specification explaining how desktops should poll() the > >>brightness files > >> > >>2) get someone to volunteer to implement that specification for at > >>least GNOME and KDE? > > > >Hm... Probably I should try to implement it myself :-) and look deeply > >what I'm proposing... > > > >Hans, you probably already did some changes for hw brightness changed > >attribute... Do you have some links or application parts which handle > >this for GNOME, KDE or other DE? So I could look at correct application? > > > >IIRC KDE4 uses upowerd for controlling keyboard backlight. > > Gnome uses upower too, and current upower master has patches to > support brightness_hw_changed which you could use as a start. Ok, thanks! -- Pali Rohár pali.rohar@gmail.com