From: Matthew Garrett <mjg@redhat.com>
To: "Miguel Gómez" <magomez@igalia.com>
Cc: Thadeu Lima de Souza Cascardo <cascardo@holoscopio.com>,
don@syst.com.br, platform-driver-x86@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/1 v2] classmate-laptop: Add support for Classmate V4 accelerometer.
Date: Mon, 23 Jul 2012 14:28:41 +0100 [thread overview]
Message-ID: <20120723132841.GD21495@srcf.ucam.org> (raw)
In-Reply-To: <5006BFEC.1010407@igalia.com>
On Wed, Jul 18, 2012 at 03:53:48PM +0200, Miguel Gómez wrote:
> >1) reverting the patch in acpica that puts all names in uppercase
> >2) using a case-insensitive match in acpi bus driver
> >3) using uppercase name in classmate-laptop driver
>
> I think the easiest and less risky way to fix this is by adding a
> new hardware id (as proposed in the thread's patch), as it doesn't
> add any side effects (that the other solutions might add), and it
> doesn't break the behaviour of the old code.
I've lost track of the implementation state here - if all names are
upper-cased in acpica, why is this not already matching?
--
Matthew Garrett | mjg59@srcf.ucam.org
next prev parent reply other threads:[~2012-07-23 13:28 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-29 12:46 [PATCH 0/1] Add support for Classmate V3 accelerometer to classmate-laptop Miguel Gómez
2012-06-29 12:46 ` [PATCH] classmate-laptop: Add support for Classmate V3 accelerometer Miguel Gómez
2012-06-29 13:39 ` [PATCH 0/1 v2] Add support for Classmate V4 accelerometer to classmate-laptop Miguel Gómez
2012-06-29 13:39 ` [PATCH 1/1 v2] classmate-laptop: Add support for Classmate V4 accelerometer Miguel Gómez
2012-07-17 22:47 ` Thadeu Lima de Souza Cascardo
2012-07-18 13:53 ` Miguel Gómez
2012-07-23 13:28 ` Matthew Garrett [this message]
2012-07-23 13:33 ` Miguel Gómez
2012-07-23 13:36 ` Matthew Garrett
2012-07-23 13:44 ` Miguel Gómez
2012-07-24 3:09 ` Thadeu Lima de Souza Cascardo
2012-07-24 13:05 ` [PATCH] classmate-laptop: Fix extra keys hardware id Miguel Gómez
2012-07-24 13:17 ` Miguel Gómez
2012-07-24 13:21 ` Matthew Garrett
2012-07-17 14:34 ` [PATCH 0/1 v2] Add support for Classmate V4 accelerometer to classmate-laptop Miguel Gómez
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=20120723132841.GD21495@srcf.ucam.org \
--to=mjg@redhat.com \
--cc=cascardo@holoscopio.com \
--cc=don@syst.com.br \
--cc=linux-kernel@vger.kernel.org \
--cc=magomez@igalia.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.