From owner-freebsd-current@FreeBSD.ORG Fri Jan 30 22:45:54 2015 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id ECFC2BBC for ; Fri, 30 Jan 2015 22:45:54 +0000 (UTC) Received: from mail.wilcox-tech.com (mail.foxkit.us [192.99.209.192]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mail.wilcox-tech.com", Issuer "mail.wilcox-tech.com" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 897FD1E4 for ; Fri, 30 Jan 2015 22:45:53 +0000 (UTC) Received: (qmail 6068 invoked from network); 30 Jan 2015 22:47:03 -0000 Received: from ip68-13-243-137.ok.ok.cox.net (HELO ?192.168.1.253?) (emyers@wilcox-tech.com@68.13.243.137) by mail.foxkit.us with ESMTPA; 30 Jan 2015 22:47:03 -0000 Message-ID: <54CC0999.90500@interlinked.me> Date: Fri, 30 Jan 2015 16:45:45 -0600 From: Elizabeth Myers User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: John Baldwin , freebsd-current@freebsd.org Subject: Re: Questions on adding backlight support for the i915 driver References: <54C883E7.4000300@interlinked.me> <54CBA0A4.30708@FreeBSD.org> In-Reply-To: <54CBA0A4.30708@FreeBSD.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 30 Jan 2015 22:45:55 -0000 On 01/30/15 09:17, John Baldwin wrote: > Humm. If the code is going to live in the drm driver, then I would > start with hanging a sysctl off of the drm device itself. (Each new-bus > device_t has a sysctl ctx you can get to hang new nodes off of the > device's node.) I'm wondering if that's the correct place for it to live. So far it's the only real place it *can* live that makes logical sense imho, unless anyone has better ideas. This is awesome. As for upower, it should probably be patched to know about other types of brightness sysctls if/when this is implemented. -- Cheers, Elizabeth