Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753997AbZIQIOS (ORCPT ); Thu, 17 Sep 2009 04:14:18 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753848AbZIQIOQ (ORCPT ); Thu, 17 Sep 2009 04:14:16 -0400 Received: from mail.atmel.fr ([81.80.104.162]:50344 "EHLO atmel-es2.atmel.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753611AbZIQION (ORCPT ); Thu, 17 Sep 2009 04:14:13 -0400 Message-ID: <4AB1EFC5.2060707@atmel.com> Date: Thu, 17 Sep 2009 10:13:57 +0200 From: Nicolas Ferre Organization: atmel User-Agent: Thunderbird 2.0.0.23 (Windows/20090812) MIME-Version: 1.0 To: Jean-Christophe PLAGNIOL-VILLARD CC: linux-usb@vger.kernel.org, avictor.za@gmail.com, patrice.vilchez@atmel.com, linux-kernel@vger.kernel.org, david-b@pacbell.net, haavard.skinnemoen@atmel.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] at91: use gpiolib calls for USB vbus pin on at91sam9g45 References: <4AB10F7D.9060706@atmel.com> <1253122169-14806-1-git-send-email-nicolas.ferre@atmel.com> <20090916215328.GG4172@game.jcrosoft.org> In-Reply-To: <20090916215328.GG4172@game.jcrosoft.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2292 Lines: 59 Jean-Christophe PLAGNIOL-VILLARD : > On 19:29 Wed 16 Sep , Nicolas Ferre wrote: >> Change pin configuration for USB vbus on at91sam9g45: use the generic gpiolib >> call instead of the at91 specific one. >> Use gpio_request() function with same identifier for OHCI and EHCI hosts as >> they are sharing the same pin. >> >> Signed-off-by: Nicolas Ferre >> --- >> This patch is on top of at91sam9g45 USB integration one: >> "[PATCH 2/2] at91/USB: at91sam9g45 series USB host integration" >> http://lkml.org/lkml/2009/6/9/221 >> >> arch/arm/mach-at91/at91sam9g45_devices.c | 12 ++++++++---- >> 1 files changed, 8 insertions(+), 4 deletions(-) >> >> diff --git a/arch/arm/mach-at91/at91sam9g45_devices.c b/arch/arm/mach-at91/at91sam9g45_devices.c >> index 5be8cf2..7d939c0 100644 >> --- a/arch/arm/mach-at91/at91sam9g45_devices.c >> +++ b/arch/arm/mach-at91/at91sam9g45_devices.c >> @@ -118,8 +118,10 @@ void __init at91_add_device_usbh_ohci(struct at91_usbh_data *data) >> >> /* Enable VBus control for UHP ports */ >> for (i = 0; i < data->ports; i++) { >> - if (data->vbus_pin[i]) >> - at91_set_gpio_output(data->vbus_pin[i], 0); >> + if (data->vbus_pin[i]) { >> + gpio_request(data->vbus_pin[i], "usb host vbus"); >> + gpio_direction_output(data->vbus_pin[i], 0); >> + } >> } >> >> usbh_ohci_data = *data; >> @@ -173,8 +175,10 @@ void __init at91_add_device_usbh_ehci(struct at91_usbh_data *data) >> >> /* Enable VBus control for UHP ports */ >> for (i = 0; i < data->ports; i++) { >> - if (data->vbus_pin[i]) >> - at91_set_gpio_output(data->vbus_pin[i], 0); >> + if (data->vbus_pin[i]) { >> + gpio_request(data->vbus_pin[i], "usb host vbus"); >> + gpio_direction_output(data->vbus_pin[i], 0); >> + } >> } > as you do the same think for ehci & ohci why not factorize it? Just because you may choose only one or the other controller. For instance, if you only want ohci, you can disable ehci selection and always have its configuration done. Best regards, -- Nicolas Ferre -- 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/