From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754881AbZHTPVY (ORCPT ); Thu, 20 Aug 2009 11:21:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754842AbZHTPVV (ORCPT ); Thu, 20 Aug 2009 11:21:21 -0400 Received: from cantor2.suse.de ([195.135.220.15]:33315 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754835AbZHTPVT (ORCPT ); Thu, 20 Aug 2009 11:21:19 -0400 Date: Thu, 20 Aug 2009 17:21:20 +0200 Message-ID: From: Takashi Iwai To: Andres Salomon Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, Takashi Iwai , cjb@laptop.org, deepak@laptop.org, linux-geode@lists.infradead.org, Jordan Crouse , Tobias_Mueller@twam.info Subject: Re: [PATCH 1/3] cs5535-gpio: add AMD CS5535/CS5536 GPIO driver support In-Reply-To: <20090818175314.16c81ca4@mycelium.queued.net> References: <20090818175314.16c81ca4@mycelium.queued.net> User-Agent: Wanderlust/2.15.6 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.7 Emacs/23.1 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Tue, 18 Aug 2009 17:53:14 -0400, Andres Salomon wrote: > > > This creates a CS5535/CS5536 GPIO driver which uses a gpio_chip backend > (allowing GPIO users to use the generic GPIO API if desired) while also > allowing architecture-specific users directly (via the cs5535_gpio_* > functions). Will be any user of cs5535_gpio_*() expected? If not, it'd be better not to export stuff, IMO. > Tested on an OLPC machine. Some Leemotes also use CS5536 (with a mips > cpu), which is why this is in drivers/gpio rather than arch/x86. > Currently, it conflicts with older geode GPIO support; once MFGPT support > is reworked to also be more generic, the older geode code will be removed. ... or you can rewrite the old driver to use the functions above? Then the ugly Kconfig check can be dropped, too. (yeah that's an answer to my own question above.) BTW CONFIG_GPIO_CS5536 and CONFIG_CS5536_GPIO are so confusing... :) > Signed-off-by: Andres Salomon I can happily merge this patch series to sound GIT tree if no one objects, since it's basically for OLPC sound stuff. thanks, Takashi