From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753539Ab2ANI7D (ORCPT ); Sat, 14 Jan 2012 03:59:03 -0500 Received: from mail-bk0-f46.google.com ([209.85.214.46]:42010 "EHLO mail-bk0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752634Ab2ANI7A (ORCPT ); Sat, 14 Jan 2012 03:59:00 -0500 MIME-Version: 1.0 In-Reply-To: <20120113224211.GB29022@srcf.ucam.org> References: <20120113224211.GB29022@srcf.ucam.org> Date: Sat, 14 Jan 2012 09:58:58 +0100 Message-ID: Subject: Re: Handling special keys in platform drivers From: Corentin Chary To: Matthew Garrett Cc: platform-driver-x86@vger.kernel.org, LKML , linux-input@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 13, 2012 at 11:42 PM, Matthew Garrett wrote: > On Mon, Jan 09, 2012 at 07:24:14AM +0100, Corentin Chary wrote: > >> The real problem is that for keyboard backlight to work, it needs DE >> cooperation, and only gnome as implemented that right now, and other >> (except KDE) will probably neither have the resources to handle all >> the possible keys correctly. And of course, who should handle the keys >> when there is no DE running at all ? > > The problem with handling this in kernel is that there isn't an > obviously correct policy. ACPI will typically only expose 16 or so > backlight levels so the behaviour is easy enough, but (depending on how > the hardware is wired up) i915 may expose around 20,000. Further, one > machine may expose different backlight controls for different displays, > and in that situation you need to know which one the input event is > expected to correspond to. > > Doing backlight control properly involves a lot of policy work, and I'm > reluctant to argue that that ought to be in the kernel. But in the case of asus-wmi, that handle both the keys and the keyboard_backlight, could such handling be done in the kernel ? -- Corentin Chary http://xf.iksaif.net