From: Matthew Garrett <matthew.garrett@nebula.com>
To: "mario_limonciello@dell.com" <mario_limonciello@dell.com>
Cc: "platform-driver-x86@vger.kernel.org"
<platform-driver-x86@vger.kernel.org>
Subject: Re: [PATCH v4 0/1] Drop individual LED nodes for colors
Date: Thu, 27 Feb 2014 20:45:49 +0000 [thread overview]
Message-ID: <1393533948.14900.57.camel@x230> (raw)
In-Reply-To: <1393533182-2725-1-git-send-email-mario_limonciello@dell.com>
On Thu, 2014-02-27 at 14:33 -0600, Mario Limonciello wrote:
> It was raised that it's not possible to make atomic changes that require
> multiple colors such as from magenta to cyan. So I decided to drop the
> 36 nodes for controlling 3 zones and instead reduce it to 3 per zone.
> The color needs to be packed into that node when setting it, but how it's
> done is well commented.
Hm. I'm not a huge fan of this approach - do any other drivers do it the
same way? It seems like this forces userspace code to special-case this
system.
--
Matthew Garrett <matthew.garrett@nebula.com>
next prev parent reply other threads:[~2014-02-27 20:45 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-27 20:33 [PATCH v4 0/1] Drop individual LED nodes for colors Mario Limonciello
2014-02-27 20:33 ` [PATCH v4 1/1] Add WMI driver for controlling AlienFX and HDMI on Alienware Mario Limonciello
2014-02-27 20:45 ` Matthew Garrett [this message]
2014-02-27 20:59 ` [PATCH v4 0/1] Drop individual LED nodes for colors Mario Limonciello
2014-02-28 21:42 ` Mario Limonciello
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=1393533948.14900.57.camel@x230 \
--to=matthew.garrett@nebula.com \
--cc=mario_limonciello@dell.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.