From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olof Johansson Subject: Re: [PATCH 07/11] gpio: tegra: Hide tegra_gpio_enable/disable() Date: Wed, 4 Apr 2012 11:01:06 -0700 Message-ID: <20120404180106.GG29919@quad.lixom.net> References: <1333148404-17691-1-git-send-email-swarren@wwwdotorg.org> <1333148404-17691-8-git-send-email-swarren@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1333148404-17691-8-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: Colin Cross , Linus Walleij , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Stephen Warren , Chris Ball , linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org On Fri, Mar 30, 2012 at 05:00:00PM -0600, Stephen Warren wrote: > From: Stephen Warren > > Recent pinctrl discussions concluded that gpiolib APIs should in fact do > whatever is required to mux a GPIO onto pins, by calling pinctrl APIs if > required. This change implements this for the Tegra GPIO driver, and removes > calls to the Tegra-specific APIs from drivers and board files. > > Cc: Chris Ball > Cc: linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Signed-off-by: Stephen Warren > --- > v2: Adjusted per changes to latest version of Documentation/gpio.txt. > Fixed tegra_board_pinmux_bus_notify() to still check the device name, > so it waits for the pinmux device to be registered, not just /any/ device. Acked-by: Olof Johansson I'm quite glad to see this happen. Erik had the intention to keep pin setup separate from gpio drivers, but in the end it just results in duplicate setup of the same stuff in different places. In hindsight this does seem quite a bit cleaner. -Olof