linux-kernel - Re: [PATCH] hid, 2.6.23 - add support for new macbook and apple aluminum keyboard "fn" key

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <Pine.LNX.4.64.0802122143280.7699@twin.jikos.cz>
Date:	Tue, 12 Feb 2008 21:48:32 +0100 (CET)
From:	Jiri Kosina <jkosina@...e.cz>
To:	Chris Nolan <chrisonthelists@...olan.com>
cc:	linux-kernel@...r.kernel.org,
	Tobias Müller <Tobias_Mueller@...m.info>
Subject: Re: [PATCH] hid, 2.6.23 - add support for new macbook and apple
 aluminum keyboard "fn" key

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/