Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755293Ab0KCN4Y (ORCPT ); Wed, 3 Nov 2010 09:56:24 -0400 Received: from cantor.suse.de ([195.135.220.2]:55809 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752243Ab0KCN4W (ORCPT ); Wed, 3 Nov 2010 09:56:22 -0400 Date: Wed, 3 Nov 2010 09:56:21 -0400 (EDT) From: Jiri Kosina To: gimli Cc: linux-kernel@vger.kernel.org Subject: Re: Resend [PATCH 3/4] MacbookAir3,1 and 3,2 hid support In-Reply-To: <7b3b313ce7138c19191295a645dfb38f@mognix.dark-green.com> Message-ID: References: <66c1cd33ff5b2d908b2b31defb8e0cd1@mognix.dark-green.com> <3e6b335b371a07cd60221a25c5b6b47d@mognix.dark-green.com> <7b3b313ce7138c19191295a645dfb38f@mognix.dark-green.com> User-Agent: Alpine 2.00 (LNX 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 615 Lines: 28 On Wed, 3 Nov 2010, gimli wrote: > Does this look better ? What I would actually prefer would be something like if ( ... ) table = macbookair_fn_keys; else if ( ... ) table = powerbook_fn_keys; else table = apple_fn_keys; apple_find_translation(table, usage->code); > I also use the USB id's for the if. Thanks, -- Jiri Kosina SUSE Labs, Novell Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/