Re: [PATCH] hid, 2.6.23 - add support for new macbook and applealuminum keyboard "fn" key

From: Jiri Kosina
Date: Tue Feb 12 2008 - 15:48:51 EST


On Tue, 12 Feb 2008, Chris Nolan wrote:

> Seeking feedback for this patch which adds support for the "fn" key on
> the new macbook (3,1) and apple aluminum keyboard. I have tested this
> with kernel 2.6.23 (both 2.6.23.9 and 2.6.23.15) under Fedora 8 and it
> seems to work for me.

Hi Chris,

Some part of the patch is already in 2.6.25-rc1 -- please see commit
a45d82d19a6c2a717bcc33cff243199b77fa0082, which adds support for Apple
Aluminium keyboards. So the patch would need some changes that it would
apply on top of current tree.

Also, if you could get Signed-off-by for the new patch from the author(s)
of the original patch, that would be great.

> The original patch was created by Martin Szulecki (of mactel-linux) and
> Chris Irwin. I can't take any credit - I've just found that it works and
> would like to see it tested and committed upstream if possible.
> I am linking to the patch because a) its quite long, and b) I am not sure how
> to prevent thunderbird from wrapping the lines and I'd hate to annoy the list
> with some malformed garbage.
> Patch is here:
> https://bugzilla.redhat.com/attachment.cgi?id=294650

Also, Tobias (added to CC) seems to be currently working on adding support
for a subset of keyboards that this patch adds. Tobias -- it seems like
this patch is handling a superset of what your patch does, right?

It would be great if you could put this all together into one condensated
patch which I could merge into my tree.

Thanks guys!

--
Jiri Kosina
SUSE Labs

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/