Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753324AbZI2H76 (ORCPT ); Tue, 29 Sep 2009 03:59:58 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752891AbZI2H75 (ORCPT ); Tue, 29 Sep 2009 03:59:57 -0400 Received: from mail.atmel.fr ([81.80.104.162]:44874 "EHLO atmel-es2.atmel.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752720AbZI2H74 (ORCPT ); Tue, 29 Sep 2009 03:59:56 -0400 Message-ID: <4AC1BE68.2080504@atmel.com> Date: Tue, 29 Sep 2009 09:59:36 +0200 From: Nicolas Ferre Organization: atmel User-Agent: Thunderbird 2.0.0.23 (Windows/20090812) MIME-Version: 1.0 To: David Brownell , Greg Kroah-Hartman CC: linux-usb@vger.kernel.org, avictor.za@gmail.com, linux-arm-kernel@lists.infradead.org, patrice.vilchez@atmel.com, linux-kernel@vger.kernel.org, Hong Xu Subject: Re: [PATCH 4/5] at91/USB: USB drivers modifications for at91sam9g10 References: <1245341772-1689-1-git-send-email-nicolas.ferre@atmel.com> <4AB0D325.9080407@atmel.com> <4ABC9A4C.8000702@atmel.com> <200909251051.34916.david-b@pacbell.net> In-Reply-To: <200909251051.34916.david-b@pacbell.net> 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: 4712 Lines: 137 David Brownell : > On Friday 25 September 2009, Nicolas Ferre wrote: >> Greg, >> >> Please, can you consider the inclusion of this patch. I have no news >> about it for a long time... > > ISTR either acking this or suggesting it go with the rest > of the G10 patches ... regardless, seems OK to me now that > its precursors have merged. Good. Greg, can you handle this for inclusion in mainline soon ? Thanks, Bye. >> Here is a link to its patchwork entry: >> http://patchwork.kernel.org/patch/31173/ >> >> Thanks for your help. >> >> Nicolas Ferre : >>> Hi, >>> >>> Here is a little "ping" about integration of this patch. >>> >>> Nicolas Ferre : >>>> From: Hong Xu >>>> >>>> Modify both host and gadget USB drivers for at91sam9g10. >>>> This add a clock management equivalent to at91sam9261 on usb drivers. >>>> It also add the way of handling gadget pull-ups (like the at91sam9261). >>> I wonder if we can consider its inclusion as at91sam9g10 is now >>> integrated in mainline. >>> Do you want me to split this in several pieces for host/gadget split. >>> >>> Thanks. >>> >>>> Signed-off-by: Hong Xu >>>> Signed-off-by: Nicolas Ferre >>>> --- >>>> drivers/usb/gadget/at91_udc.c | 6 +++--- >>>> drivers/usb/host/ohci-at91.c | 10 +++++----- >>>> 2 files changed, 8 insertions(+), 8 deletions(-) >>>> >>>> diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c >>>> index 0b2bb8f..bfed3d9 100644 >>>> --- a/drivers/usb/gadget/at91_udc.c >>>> +++ b/drivers/usb/gadget/at91_udc.c >>>> @@ -892,7 +892,7 @@ static void pullup(struct at91_udc *udc, int is_on) >>>> >>>> txvc |= AT91_UDP_TXVC_PUON; >>>> at91_udp_write(udc, AT91_UDP_TXVC, txvc); >>>> - } else if (cpu_is_at91sam9261()) { >>>> + } else if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) { >>>> u32 usbpucr; >>>> >>>> usbpucr = at91_sys_read(AT91_MATRIX_USBPUCR); >>>> @@ -910,7 +910,7 @@ static void pullup(struct at91_udc *udc, int is_on) >>>> >>>> txvc &= ~AT91_UDP_TXVC_PUON; >>>> at91_udp_write(udc, AT91_UDP_TXVC, txvc); >>>> - } else if (cpu_is_at91sam9261()) { >>>> + } else if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) { >>>> u32 usbpucr; >>>> >>>> usbpucr = at91_sys_read(AT91_MATRIX_USBPUCR); >>>> @@ -1692,7 +1692,7 @@ static int __init at91udc_probe(struct platform_device *pdev) >>>> udc->ep[3].maxpacket = 64; >>>> udc->ep[4].maxpacket = 512; >>>> udc->ep[5].maxpacket = 512; >>>> - } else if (cpu_is_at91sam9261()) { >>>> + } else if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) { >>>> udc->ep[3].maxpacket = 64; >>>> } else if (cpu_is_at91sam9263()) { >>>> udc->ep[0].maxpacket = 64; >>>> diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci-at91.c >>>> index b29b0fe..fb64def 100644 >>>> --- a/drivers/usb/host/ohci-at91.c >>>> +++ b/drivers/usb/host/ohci-at91.c >>>> @@ -35,7 +35,7 @@ extern int usb_disabled(void); >>>> >>>> static void at91_start_clock(void) >>>> { >>>> - if (cpu_is_at91sam9261()) >>>> + if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) >>>> clk_enable(hclk); >>>> clk_enable(iclk); >>>> clk_enable(fclk); >>>> @@ -46,7 +46,7 @@ static void at91_stop_clock(void) >>>> { >>>> clk_disable(fclk); >>>> clk_disable(iclk); >>>> - if (cpu_is_at91sam9261()) >>>> + if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) >>>> clk_disable(hclk); >>>> clocked = 0; >>>> } >>>> @@ -142,7 +142,7 @@ static int usb_hcd_at91_probe(const struct hc_driver *driver, >>>> >>>> iclk = clk_get(&pdev->dev, "ohci_clk"); >>>> fclk = clk_get(&pdev->dev, "uhpck"); >>>> - if (cpu_is_at91sam9261()) >>>> + if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) >>>> hclk = clk_get(&pdev->dev, "hck0"); >>>> >>>> at91_start_hc(pdev); >>>> @@ -156,7 +156,7 @@ static int usb_hcd_at91_probe(const struct hc_driver *driver, >>>> /* Error handling */ >>>> at91_stop_hc(pdev); >>>> >>>> - if (cpu_is_at91sam9261()) >>>> + if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) >>>> clk_put(hclk); >>>> clk_put(fclk); >>>> clk_put(iclk); >>>> @@ -193,7 +193,7 @@ static void usb_hcd_at91_remove(struct usb_hcd *hcd, >>>> release_mem_region(hcd->rsrc_start, hcd->rsrc_len); >>>> usb_put_hcd(hcd); >>>> >>>> - if (cpu_is_at91sam9261()) >>>> + if (cpu_is_at91sam9261() || cpu_is_at91sam9g10()) >>>> clk_put(hclk); >>>> clk_put(fclk); >>>> clk_put(iclk); -- 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/