Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1263755 > unrolled thread

[PATCH 0/2] Configure AHB to post data transfers

Started byAndy Gross <agross@codeaurora.org>
First post2015-11-06 07:10 +0100
Last post2015-11-06 16:50 +0100
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 0/2] Configure AHB to post data transfers Andy Gross <agross@codeaurora.org> - 2015-11-06 07:10 +0100
    [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB Andy Gross <agross@codeaurora.org> - 2015-11-06 07:10 +0100
      Re: [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB Georgi Djakov <georgi.djakov@linaro.org> - 2015-11-06 11:20 +0100
        Re: [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB Alan Stern <stern@rowland.harvard.edu> - 2015-11-06 16:50 +0100

#1263755 — [PATCH 0/2] Configure AHB to post data transfers

FromAndy Gross <agross@codeaurora.org>
Date2015-11-06 07:10 +0100
Subject[PATCH 0/2] Configure AHB to post data transfers
Message-ID<qrInL-JI-5@gated-at.bofh.it>
This patch configures the ChipIdea USB 2.0 controller found on
Qualcomm platforms to post data transfers on the AHB bus.  This
yields approximately a 50% increase in performance.

Andy Gross (2):
  usb: chipidea: msm: Use posted data writes on AHB
  usb: host: ehci-msm: Use posted data writes on AHB

 drivers/usb/chipidea/ci_hdrc_msm.c |    3 ++-
 drivers/usb/host/ehci-msm.c        |    4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation

--
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/

[toc] | [next] | [standalone]


#1263757 — [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB

FromAndy Gross <agross@codeaurora.org>
Date2015-11-06 07:10 +0100
Subject[PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB
Message-ID<qrInM-JI-11@gated-at.bofh.it>
In reply to#1263755
This patch sets the AHBMODE to allow for posted data writes.  This
results in higher performance.

Signed-off-by: Andy Gross <agross@codeaurora.org>
---
 drivers/usb/host/ehci-msm.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/host/ehci-msm.c b/drivers/usb/host/ehci-msm.c
index c4f84c8..c23e285 100644
--- a/drivers/usb/host/ehci-msm.c
+++ b/drivers/usb/host/ehci-msm.c
@@ -57,8 +57,8 @@ static int ehci_msm_reset(struct usb_hcd *hcd)
 
 	/* bursts of unspecified length. */
 	writel(0, USB_AHBBURST);
-	/* Use the AHB transactor */
-	writel(0, USB_AHBMODE);
+	/* Use the AHB transactor, allow posted data writes */
+	writel(0x8, USB_AHBMODE);
 	/* Disable streaming mode and select host mode */
 	writel(0x13, USB_USBMODE);
 
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation

--
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/

[toc] | [prev] | [next] | [standalone]


#1263895 — Re: [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB

FromGeorgi Djakov <georgi.djakov@linaro.org>
Date2015-11-06 11:20 +0100
SubjectRe: [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB
Message-ID<qrMhI-3aK-7@gated-at.bofh.it>
In reply to#1263757
On 11/06/2015 08:04 AM, Andy Gross wrote:
> This patch sets the AHBMODE to allow for posted data writes.  This
> results in higher performance.
> 
> Signed-off-by: Andy Gross <agross@codeaurora.org>

With these patches I see significant improvement in throughput
on my db410c board.

Tested-by: Georgi Djakov <georgi.djakov@linaro.org>

> ---
>  drivers/usb/host/ehci-msm.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/usb/host/ehci-msm.c b/drivers/usb/host/ehci-msm.c
> index c4f84c8..c23e285 100644
> --- a/drivers/usb/host/ehci-msm.c
> +++ b/drivers/usb/host/ehci-msm.c
> @@ -57,8 +57,8 @@ static int ehci_msm_reset(struct usb_hcd *hcd)
>  
>  	/* bursts of unspecified length. */
>  	writel(0, USB_AHBBURST);
> -	/* Use the AHB transactor */
> -	writel(0, USB_AHBMODE);
> +	/* Use the AHB transactor, allow posted data writes */
> +	writel(0x8, USB_AHBMODE);
>  	/* Disable streaming mode and select host mode */
>  	writel(0x13, USB_USBMODE);
>  
> 

--
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/

[toc] | [prev] | [next] | [standalone]


#1264099 — Re: [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB

FromAlan Stern <stern@rowland.harvard.edu>
Date2015-11-06 16:50 +0100
SubjectRe: [PATCH 2/2] usb: host: ehci-msm: Use posted data writes on AHB
Message-ID<qrRr3-6mq-9@gated-at.bofh.it>
In reply to#1263895
On Fri, 6 Nov 2015, Georgi Djakov wrote:

> On 11/06/2015 08:04 AM, Andy Gross wrote:
> > This patch sets the AHBMODE to allow for posted data writes.  This
> > results in higher performance.
> > 
> > Signed-off-by: Andy Gross <agross@codeaurora.org>
> 
> With these patches I see significant improvement in throughput
> on my db410c board.
> 
> Tested-by: Georgi Djakov <georgi.djakov@linaro.org>
> 
> > ---
> >  drivers/usb/host/ehci-msm.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/usb/host/ehci-msm.c b/drivers/usb/host/ehci-msm.c
> > index c4f84c8..c23e285 100644
> > --- a/drivers/usb/host/ehci-msm.c
> > +++ b/drivers/usb/host/ehci-msm.c
> > @@ -57,8 +57,8 @@ static int ehci_msm_reset(struct usb_hcd *hcd)
> >  
> >  	/* bursts of unspecified length. */
> >  	writel(0, USB_AHBBURST);
> > -	/* Use the AHB transactor */
> > -	writel(0, USB_AHBMODE);
> > +	/* Use the AHB transactor, allow posted data writes */
> > +	writel(0x8, USB_AHBMODE);
> >  	/* Disable streaming mode and select host mode */
> >  	writel(0x13, USB_USBMODE);

Acked-by: Alan Stern <stern@rowland.harvard.edu>

--
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/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web