From: Mario Limonciello <mario_limonciello@dell.com>
To: "matthew.garrett@nebula.com" <matthew.garrett@nebula.com>
Cc: LKML <linux-kernel@vger.kernel.org>,
"platform-driver-x86@vger.kernel.org"
<platform-driver-x86@vger.kernel.org>
Subject: Re: [PATCH v5 0/1] Add WMI driver to control AlienFX features
Date: Tue, 01 Apr 2014 17:18:00 -0500 [thread overview]
Message-ID: <533B3B18.1030000@dell.com> (raw)
In-Reply-To: <1395842248-27868-1-git-send-email-mario_limonciello@dell.com>
On 03/26/2014 08:57 AM, Limonciello, Mario wrote:
> This is a driver for the Alienware X51, X51 R2 and some future HW.
> - For the X51 / X51 R2 it allows control of the color and brightness
> of each zone via a sysfs interface for the color and a LED interface
> for the brightness
> - For the future HW it allows control of color and brightness as well
> as an HDMI mux that's included in the HW.
>
> Changes from v4 to v5:
> - Make zone control more like the thingm RGB LED in the kernel. Brightness
> control is independent of the RGB adjustment. RGB adjustment takes a 24-bit
> hexadecimal triplet.
> - Introduced a new WMAX method for platforms that use HDMI as well as a new
> lighting control interface. This removes the present necessity of having
> to do a DMI match on HDMI for now.
> - Dynamically create number of sysfs zone nodes based on DMI data. Present
> platforms have 3 nodes, but some future ones will have anywhere from 2
> to 32 nodes.
> - Make lighting control state and HDMI more readable and usable for
> interacting. Rather than having to know numbers, cat'ing the nodes will
> output the available options and the presently selected option (in a
> single line)
>
> Mario Limonciello (1):
> Add WMI driver for controlling AlienFX features on some Alienware
> products
>
> drivers/platform/x86/Kconfig | 12 +
> drivers/platform/x86/Makefile | 2 +
> drivers/platform/x86/alienware-wmi.c | 557 +++++++++++++++++++++++++++++++++++
> 3 files changed, 571 insertions(+)
> create mode 100644 drivers/platform/x86/alienware-wmi.c
>
Matthew,
Any feedback to apply on this?
Thanks,
next prev parent reply other threads:[~2014-04-01 22:18 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-26 13:57 [PATCH v5 0/1] Add WMI driver to control AlienFX features Mario Limonciello
2014-03-26 13:57 ` [PATCH v5 1/1] Add WMI driver for controlling AlienFX features on some Alienware products Mario Limonciello
2014-04-01 22:18 ` Mario Limonciello [this message]
2014-04-01 22:24 ` [PATCH v5 0/1] Add WMI driver to control AlienFX features Matthew Garrett
2014-04-01 22:24 ` Matthew Garrett
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=533B3B18.1030000@dell.com \
--to=mario_limonciello@dell.com \
--cc=linux-kernel@vger.kernel.org \
--cc=matthew.garrett@nebula.com \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.