From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751468AbdBXIxD (ORCPT <rfc822;w@1wt.eu>);
        Fri, 24 Feb 2017 03:53:03 -0500
Received: from mx1.redhat.com ([209.132.183.28]:57268 "EHLO mx1.redhat.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1751296AbdBXIwx (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 24 Feb 2017 03:52:53 -0500
Subject: Re: LED devices & poll() for brightness attribute
To: =?UTF-8?Q?Pali_Roh=c3=a1r?= <pali.rohar@gmail.com>,
        Pavel Machek <pavel@ucw.cz>
References: <20170222090630.GB21558@pali> <20170223144852.GD24201@pali>
 <20170223202333.GA19376@amd> <201702232144.54558@pali>
 <20170223210321.GB19376@amd> <20170224085055.GA2157@pali>
Cc: Richard Purdie <rpurdie@rpsys.net>,
        Jacek Anaszewski <jacek.anaszewski@gmail.com>,
        linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org,
        Darren Hart <dvhart@infradead.org>,
        platform-driver-x86@vger.kernel.org
From: Hans de Goede <hdegoede@redhat.com>
Message-ID: <e87ee6f8-8c7e-7370-33ae-6f420f91a582@redhat.com>
Date: Fri, 24 Feb 2017 09:52:11 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.7.0
MIME-Version: 1.0
In-Reply-To: <20170224085055.GA2157@pali>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Fri, 24 Feb 2017 08:52:15 +0000 (UTC)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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.

Regards,

Hans