From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965480Ab2CBJHR (ORCPT ); Fri, 2 Mar 2012 04:07:17 -0500 Received: from na3sys009aog108.obsmtp.com ([74.125.149.199]:36277 "EHLO na3sys009aog108.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965455Ab2CBJHD (ORCPT ); Fri, 2 Mar 2012 04:07:03 -0500 Authentication-Results: mr.google.com; spf=pass (google.com: domain of rnayak@ti.com designates 10.236.153.230 as permitted sender) smtp.mail=rnayak@ti.com Message-ID: <4F508DAE.9000005@ti.com> Date: Fri, 02 Mar 2012 14:36:54 +0530 From: Rajendra Nayak User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.20) Gecko/20110805 Thunderbird/3.1.12 MIME-Version: 1.0 To: Tony Lindgren CC: linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, linux-mmc@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 0/4] Start getting rid of pdata callbacks with gpio_find_by_chip_name() References: <20120301185044.29210.44521.stgit@kaulin.local> In-Reply-To: <20120301185044.29210.44521.stgit@kaulin.local> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Tony, On Friday 02 March 2012 12:25 AM, Tony Lindgren wrote: > Hi all, > > This series adds gpio_find_by_name() that allows finding > GPIOs on specific gpio_chips. As the GPIO numbers can be > dynamic, it's hard to find the GPIO numbers from drivers > using them directly. > > So far we've dealt with this using platform specific callbacks, > but that is messy. This series removes the needs for these > callbacks for omap hsmmc driver. Further callbacks can be > removed people are OK with adding gpio_find_by_name(). > > This series is based on the omap fixes-non-critical that's > needed for the arch/arm/mach-omap2 parts of this series. I tested these on my beagle/panda/omap4sdp and they seem to work fine, also fixing the broken panda card detect (due to missing card_detect_irq in the board file). There are still issues however when I build twl4030-gpio as a module, which I already commented on, and the fact that the init sequence now works by luck :) The other issue also is that the multiple insmod/rmmod test suggested by Russell still fails, since the second time around the gpio_requests in the board callback fail because they are not freed when you do a module unload/unbind. That would need this patch from me to add the .teardown hooks http://marc.info/?l=linux-omap&m=133007767831297&w=2 regards, Rajendra > > Regards, > > Tony > > --- > > Tony Lindgren (4): > gpiolib: Add gpiochip_find_by_name() and gpio_find_by_chip_name() > mmc: omap_hsmmc: Use gpio_find_by_chip_name() for omap_hsmmc_gpio_init() > mmc: omap_hsmmc: Use GPIO offset for external GPIO chips > mmc: omap_hsmmc: Simplify init for twl6030 MMC card detect > > > arch/arm/mach-omap2/board-3430sdp.c | 13 +- > arch/arm/mach-omap2/board-4430sdp.c | 45 -------- > arch/arm/mach-omap2/board-cm-t35.c | 8 - > arch/arm/mach-omap2/board-devkit8000.c | 7 - > arch/arm/mach-omap2/board-igep0020.c | 8 - > arch/arm/mach-omap2/board-omap3beagle.c | 9 +- > arch/arm/mach-omap2/board-omap3evm.c | 8 - > arch/arm/mach-omap2/board-omap3pandora.c | 13 +- > arch/arm/mach-omap2/board-omap3stalker.c | 8 - > arch/arm/mach-omap2/board-omap3touchbook.c | 7 - > arch/arm/mach-omap2/board-omap4panda.c | 52 ---------- > arch/arm/mach-omap2/board-zoom-peripherals.c | 7 - > arch/arm/mach-omap2/hsmmc.c | 3 + > arch/arm/mach-omap2/hsmmc.h | 5 + > arch/arm/plat-omap/include/plat/mmc.h | 3 + > drivers/gpio/gpio-twl4030.c | 2 > drivers/gpio/gpiolib.c | 47 +++++++++ > drivers/mfd/twl6030-irq.c | 33 +++--- > drivers/mmc/host/omap_hsmmc.c | 140 +++++++++++++++++++------- > include/asm-generic/gpio.h | 3 - > 20 files changed, 204 insertions(+), 217 deletions(-) >