diff options
author | Jeremy Cline <jcline@redhat.com> | 2018-10-23 14:36:56 -0400 |
---|---|---|
committer | Jeremy Cline <jcline@redhat.com> | 2018-10-23 14:36:56 -0400 |
commit | 764628643cfcb53d59edf15adf7f32508a220c0e (patch) | |
tree | 74ca9229375ba1562da2fee8ceb0f658f11f9bb8 /usb-dwc2-Turn-on-uframe-sched-everywhere.patch | |
parent | 6fcbd945bc574b93b64a087473420454f2dda0e5 (diff) | |
download | kernel-764628643cfcb53d59edf15adf7f32508a220c0e.tar.gz kernel-764628643cfcb53d59edf15adf7f32508a220c0e.tar.xz kernel-764628643cfcb53d59edf15adf7f32508a220c0e.zip |
Sync up stabilization with f29
Diffstat (limited to 'usb-dwc2-Turn-on-uframe-sched-everywhere.patch')
-rw-r--r-- | usb-dwc2-Turn-on-uframe-sched-everywhere.patch | 503 |
1 files changed, 503 insertions, 0 deletions
diff --git a/usb-dwc2-Turn-on-uframe-sched-everywhere.patch b/usb-dwc2-Turn-on-uframe-sched-everywhere.patch new file mode 100644 index 000000000..2b60a6913 --- /dev/null +++ b/usb-dwc2-Turn-on-uframe-sched-everywhere.patch @@ -0,0 +1,503 @@ +From patchwork Thu Jul 26 20:07:27 2018 +Content-Type: text/plain; charset="utf-8" +MIME-Version: 1.0 +Content-Transfer-Encoding: 7bit +X-Patchwork-Submitter: Doug Anderson <dianders@chromium.org> +X-Patchwork-Id: 10546405 +Return-Path: <linux-usb-owner@kernel.org> +Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org + [172.30.200.125]) + by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id AF0DC14E2 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:26 +0000 (UTC) +Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A60BA2BCA3 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:26 +0000 (UTC) +Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) + id 9A60C2BCAD; Thu, 26 Jul 2018 20:08:26 +0000 (UTC) +X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on + pdx-wl-mail.web.codeaurora.org +X-Spam-Level: +X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, + MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham + version=3.3.1 +Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 351412BCA3 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:26 +0000 (UTC) +Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand + id S1731286AbeGZV0Z (ORCPT + <rfc822;patchwork-linux-usb@patchwork.kernel.org>); + Thu, 26 Jul 2018 17:26:25 -0400 +Received: from mail-pg1-f195.google.com ([209.85.215.195]:40947 "EHLO + mail-pg1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org + with ESMTP id S1730402AbeGZV0Z (ORCPT + <rfc822;linux-usb@vger.kernel.org>); Thu, 26 Jul 2018 17:26:25 -0400 +Received: by mail-pg1-f195.google.com with SMTP id x5-v6so1799928pgp.7 + for <linux-usb@vger.kernel.org>; Thu, 26 Jul 2018 13:08:00 -0700 (PDT) +DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=chromium.org; s=google; + h=from:to:cc:subject:date:message-id:in-reply-to:references + :mime-version:content-transfer-encoding; + bh=l/myg6m+9/E2E5usmaA3mgBuQgW1vnAMb6sfkOe5tok=; + b=jYChSlM7RnMm2wPFHBUMCAw2dL4hsCluRiGF/p5szHUtz8pdiOYpM8xEXY0QvhCc2O + u6GuZehq+9VJJZmuLU2ukUtoqqq0FhAFla52X5fVdjHVrFYf9fWmlRwFBKIZKaj3idC3 + wybTMcBcZF7hqLY9wR/f01qaRV88AICE09YTM= +X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=1e100.net; s=20161025; + h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to + :references:mime-version:content-transfer-encoding; + bh=l/myg6m+9/E2E5usmaA3mgBuQgW1vnAMb6sfkOe5tok=; + b=AVDLgOeFhI7B5+hnwZe7vKY4xHtW+U7LrLuav5rFornHmNLPLsoukLpUcL2DmRBeeE + fkqgOwT5SpinUMiDSryB7L1kYh76A8VpvCxceX3svLYgomScJnarr5rU2xk+KDpXLPWt + N1lq3Vlc9/D1gRa1hE0MqivrYUCTQ9vemqm3w/O8K3BdPKwKFzebUIuTpyG6QLqAGiHP + 3/kgjUgO6/FFOi/cZHVQC7qLmzrtC/Hy4+Rzu2ZnuHqebyrQSksj1aeVCEFb+URq1ltN + aWshNBZf2p+WWtUatp8N4hCIcbMLfs9R2i2N7ebjvVO9E0nphvwLwLTsK1Md5Q72xj20 + 9L1A== +X-Gm-Message-State: AOUpUlHwtoz2An49xh7Lh3/1VCAzzNFm565pY8hT4D5tT1bmwWsfgD9t + KzCry/EPfoIMBC8bRBH/iIIzCw== +X-Google-Smtp-Source: + AAOMgpcD9dtJtKK85I4RVkl1vzkP+CHk3cdc63Iv+Dt0fV67dvsEw0HGXsF6uml0d/rpFrfcIJXFQw== +X-Received: by 2002:a62:930c:: with SMTP id + b12-v6mr3476478pfe.193.1532635680239; + Thu, 26 Jul 2018 13:08:00 -0700 (PDT) +Received: from tictac2.mtv.corp.google.com + ([2620:0:1000:1501:38e4:86fe:ec0c:4007]) + by smtp.gmail.com with ESMTPSA id + u11-v6sm5784169pfd.117.2018.07.26.13.07.58 + (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); + Thu, 26 Jul 2018 13:07:59 -0700 (PDT) +From: Douglas Anderson <dianders@chromium.org> +To: balbi@kernel.org, hminas@synopsys.com, gregkh@linuxfoundation.org +Cc: stefan.wahren@i2se.com, grigor.tovmasyan@synopsys.com, + Bruno Herrera <bruherrera@gmail.com>, + Zhangfei Gao <zhangfei.gao@linaro.org>, john.stultz@linaro.org, + Stephen Warren <swarren@wwwdotorg.org>, + Jerome Brunet <jbrunet@baylibre.com>, + Douglas Anderson <dianders@chromium.org>, + linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org +Subject: [REPOST PATCH 1/4] usb: dwc2: Turn on uframe_sched on "bcm" platforms +Date: Thu, 26 Jul 2018 13:07:27 -0700 +Message-Id: <20180726200730.18290-2-dianders@chromium.org> +X-Mailer: git-send-email 2.18.0.345.g5c9ce644c3-goog +In-Reply-To: <20180726200730.18290-1-dianders@chromium.org> +References: <20180726200730.18290-1-dianders@chromium.org> +MIME-Version: 1.0 +Sender: linux-usb-owner@vger.kernel.org +Precedence: bulk +List-ID: <linux-usb.vger.kernel.org> +X-Mailing-List: linux-usb@vger.kernel.org +X-Virus-Scanned: ClamAV using ClamSMTP + +There's no reason to have the uframe scheduler off on dwc2. Running +with uframe_sched = False is equivalent to saying "I don't want to run +the correct code, I want to run the old and incorrect code". + +The uframe scheduler has been off on Broadcom since commit +58b179dcf28c ("staging: dwc2: disable uframe_sched on the bcm2835"). +Since then there have been many many improvements, notably the commit +9f9f09b048f5 ("usb: dwc2: host: Totally redo the microframe +scheduler") + +Presumably if everyone is good w/ the uframe_sched turned back on we +can kill all the old and crufty non-uframe sched code. + +Signed-off-by: Douglas Anderson <dianders@chromium.org> +Tested-by: Stefan Wahren <stefan.wahren@i2se.com> +Acked-by: Minas Harutyunyan <hminas@synopsys.com> +--- + + drivers/usb/dwc2/params.c | 1 - + 1 file changed, 1 deletion(-) + +diff --git a/drivers/usb/dwc2/params.c b/drivers/usb/dwc2/params.c +index af075d4da895..14b8742e62fa 100644 +--- a/drivers/usb/dwc2/params.c ++++ b/drivers/usb/dwc2/params.c +@@ -47,7 +47,6 @@ static void dwc2_set_bcm_params(struct dwc2_hsotg *hsotg) + p->max_transfer_size = 65535; + p->max_packet_count = 511; + p->ahbcfg = 0x10; +- p->uframe_sched = false; + } + + static void dwc2_set_his_params(struct dwc2_hsotg *hsotg) + +From patchwork Thu Jul 26 20:07:28 2018 +Content-Type: text/plain; charset="utf-8" +MIME-Version: 1.0 +Content-Transfer-Encoding: 7bit +X-Patchwork-Submitter: Doug Anderson <dianders@chromium.org> +X-Patchwork-Id: 10546403 +Return-Path: <linux-usb-owner@kernel.org> +Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org + [172.30.200.125]) + by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id CCB37112E + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:23 +0000 (UTC) +Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id C35652BCA3 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:23 +0000 (UTC) +Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) + id B7B1D2BCAD; Thu, 26 Jul 2018 20:08:23 +0000 (UTC) +X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on + pdx-wl-mail.web.codeaurora.org +X-Spam-Level: +X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, + MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham + version=3.3.1 +Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 501742BCA3 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:23 +0000 (UTC) +Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand + id S1731764AbeGZV0q (ORCPT + <rfc822;patchwork-linux-usb@patchwork.kernel.org>); + Thu, 26 Jul 2018 17:26:46 -0400 +Received: from mail-pg1-f196.google.com ([209.85.215.196]:35268 "EHLO + mail-pg1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org + with ESMTP id S1731233AbeGZV00 (ORCPT + <rfc822;linux-usb@vger.kernel.org>); Thu, 26 Jul 2018 17:26:26 -0400 +Received: by mail-pg1-f196.google.com with SMTP id e6-v6so1810484pgv.2 + for <linux-usb@vger.kernel.org>; Thu, 26 Jul 2018 13:08:01 -0700 (PDT) +DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=chromium.org; s=google; + h=from:to:cc:subject:date:message-id:in-reply-to:references + :mime-version:content-transfer-encoding; + bh=B5ISkcd4OFF7lL/r7udB8vKFGOj44Y74EZmaS9ksBcc=; + b=OD5guXgfjtPnqOoPztqSAT2onCZGRpWRpcqpqOJ6ix58Hbx9461pM8x7tqO1q/JN3S + 3ohKncJQmwPuh5ypjtuO7AnTxlM2eMqddtWyNqis6Nd+xf2g5bfKZNPfsGoLqCpQTngn + EdEGXbZg1PqRcm6nYjs8yiMEI4Tm+I+/gdAUM= +X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=1e100.net; s=20161025; + h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to + :references:mime-version:content-transfer-encoding; + bh=B5ISkcd4OFF7lL/r7udB8vKFGOj44Y74EZmaS9ksBcc=; + b=K7NcdkmXdHp64bqY20aN08BR75pgF0anssT0ABkwZ5aGU/ikR8SuKWI8u9JB0P3Ue9 + Vjwpo/HZOxywaEKEcNFu1ak7f8eVm3bjGXrl7/bjbs4TF4H54e0IbGhSGle3ApkIjY7e + px9bvBmbIYXTcqUgkXDbPq7+VE8LjXrAKXiuiBXWDMiQ0eOTnpDAMOXmtUkjysTbmWAD + fvVwHuRbh8QqCIzvMBW7SEktIHXh/Rlz3CkDQGfMX+o29vblHv481I8TOZQ6OmxR2yIX + Q3m83HRiNqZC1I76nCKzi3Ffzyp5QHz8V8ZPU7BFzLY+AwNJuAEHMFWVq2LZUIjGUHAn + Bz4A== +X-Gm-Message-State: AOUpUlH1vxqnQ2Gtc5XVCmWY+jLtoWZPDvs77YzLPTOQasdUtp2DIUxc + U6QFTISg4mo0zqpIoiEEgWhbuBFFb6Y= +X-Google-Smtp-Source: + AAOMgpfZUUQvn0cT6OrLJVROHeARBUZI1ihZBf6CLB8hJ479EvKbBA9I8Z37qR0XhEdEhDL/CDSeLg== +X-Received: by 2002:a63:7d48:: with SMTP id m8-v6mr3321908pgn.0.1532635681614; + Thu, 26 Jul 2018 13:08:01 -0700 (PDT) +Received: from tictac2.mtv.corp.google.com + ([2620:0:1000:1501:38e4:86fe:ec0c:4007]) + by smtp.gmail.com with ESMTPSA id + u11-v6sm5784169pfd.117.2018.07.26.13.08.00 + (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); + Thu, 26 Jul 2018 13:08:00 -0700 (PDT) +From: Douglas Anderson <dianders@chromium.org> +To: balbi@kernel.org, hminas@synopsys.com, gregkh@linuxfoundation.org +Cc: stefan.wahren@i2se.com, grigor.tovmasyan@synopsys.com, + Bruno Herrera <bruherrera@gmail.com>, + Zhangfei Gao <zhangfei.gao@linaro.org>, john.stultz@linaro.org, + Stephen Warren <swarren@wwwdotorg.org>, + Jerome Brunet <jbrunet@baylibre.com>, + Douglas Anderson <dianders@chromium.org>, + linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org +Subject: [REPOST PATCH 2/4] usb: dwc2: Turn on uframe_sched on "his" platforms +Date: Thu, 26 Jul 2018 13:07:28 -0700 +Message-Id: <20180726200730.18290-3-dianders@chromium.org> +X-Mailer: git-send-email 2.18.0.345.g5c9ce644c3-goog +In-Reply-To: <20180726200730.18290-1-dianders@chromium.org> +References: <20180726200730.18290-1-dianders@chromium.org> +MIME-Version: 1.0 +Sender: linux-usb-owner@vger.kernel.org +Precedence: bulk +List-ID: <linux-usb.vger.kernel.org> +X-Mailing-List: linux-usb@vger.kernel.org +X-Virus-Scanned: ClamAV using ClamSMTP + +There's no reason to have the uframe scheduler off on dwc2. Running +with uframe_sched = False is equivalent to saying "I don't want to run +the correct code, I want to run the old and incorrect code". + +The uframe scheduler has been off on HiSilicon since commit +37dd9d65cc41 ("usb: dwc2: add support of hi6220"). Since then there +have been many many improvements, notably the commit 9f9f09b048f5 +("usb: dwc2: host: Totally redo the microframe scheduler") + +Presumably if everyone is good w/ the uframe_sched turned back on we +can kill all the old and crufty non-uframe sched code. + +Signed-off-by: Douglas Anderson <dianders@chromium.org> +Tested-by: John Stultz <john.stultz@linaro.org> +Acked-by: Minas Harutyunyan <hminas@synopsys.com> +--- + + drivers/usb/dwc2/params.c | 1 - + 1 file changed, 1 deletion(-) + +diff --git a/drivers/usb/dwc2/params.c b/drivers/usb/dwc2/params.c +index 14b8742e62fa..db97bb16ecc4 100644 +--- a/drivers/usb/dwc2/params.c ++++ b/drivers/usb/dwc2/params.c +@@ -67,7 +67,6 @@ static void dwc2_set_his_params(struct dwc2_hsotg *hsotg) + p->reload_ctl = false; + p->ahbcfg = GAHBCFG_HBSTLEN_INCR16 << + GAHBCFG_HBSTLEN_SHIFT; +- p->uframe_sched = false; + p->change_speed_quirk = true; + p->power_down = false; + } + +From patchwork Thu Jul 26 20:07:29 2018 +Content-Type: text/plain; charset="utf-8" +MIME-Version: 1.0 +Content-Transfer-Encoding: 7bit +X-Patchwork-Submitter: Doug Anderson <dianders@chromium.org> +X-Patchwork-Id: 10546401 +Return-Path: <linux-usb-owner@kernel.org> +Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org + [172.30.200.125]) + by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 00AA1112E + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:18 +0000 (UTC) +Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id EBA122BCA3 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:17 +0000 (UTC) +Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) + id DFBD62BCAD; Thu, 26 Jul 2018 20:08:17 +0000 (UTC) +X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on + pdx-wl-mail.web.codeaurora.org +X-Spam-Level: +X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, + MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham + version=3.3.1 +Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 80F1A2BCA3 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:17 +0000 (UTC) +Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand + id S1731760AbeGZV0k (ORCPT + <rfc822;patchwork-linux-usb@patchwork.kernel.org>); + Thu, 26 Jul 2018 17:26:40 -0400 +Received: from mail-pf1-f196.google.com ([209.85.210.196]:34992 "EHLO + mail-pf1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org + with ESMTP id S1731557AbeGZV02 (ORCPT + <rfc822;linux-usb@vger.kernel.org>); Thu, 26 Jul 2018 17:26:28 -0400 +Received: by mail-pf1-f196.google.com with SMTP id q7-v6so932151pff.2 + for <linux-usb@vger.kernel.org>; Thu, 26 Jul 2018 13:08:03 -0700 (PDT) +DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=chromium.org; s=google; + h=from:to:cc:subject:date:message-id:in-reply-to:references + :mime-version:content-transfer-encoding; + bh=poRASEvlVBUAcfLB4RhlwSj/p7/jDuj4PF7wn26CH6s=; + b=EuZ996MDr/awcYYi/XmOHQH60XNrvktL6KUpom+u4h04Llj4H+kWvXsF4G83MBGwJA + FLASRPQBPuyYwGxHe48PcaQys8VTNtnBDzUPyxGg44JBQgIt8deE5Q8DIG7fGZhZKsMR + vEbHJEa2w4NSqRv5TFhdj0LqYKIYEnZb5Yb8Y= +X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=1e100.net; s=20161025; + h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to + :references:mime-version:content-transfer-encoding; + bh=poRASEvlVBUAcfLB4RhlwSj/p7/jDuj4PF7wn26CH6s=; + b=mk5LbpJD8tXwgBRpWESITUpd8Cxl5frquJMsqVavNKf4i7ApHdAIzL1k3LdjbAJx0Y + V9RynHtNxn1V3OGefH0zJ8R5K7/DQBl0ro1ezx0alfwLZ2t1MWT0AV+mB7Yklo0797uS + HF4D2jy5XrdWJpf66VFL0aM9S3Pf8gAlBxiZzWeRL+rhy4DLFXwiz0p7KoHRDBeWe867 + SdnV1RiEcmzxBxeVJsAh6ZdGDJzsOR3VgMWGhPsFFww8Jt98Qw4I7AKdqsBwrSB5NidR + +ujVHor7C0JkzBV+pZxJCkjIC1GWrvhsmHEIBUTYH/CFMgGZVtQK1wIFPyFiwBEVjqgS + xdcA== +X-Gm-Message-State: AOUpUlH0HQCfvbelfBObzeFp05OY4YmlPJ8jzN9G7NckHDXOSfjcsbC7 + wabC2Da7cvQj9EFSzB972NjALA== +X-Google-Smtp-Source: + AAOMgpeTZUEvqBv0EEh4ci8/3umHRH8aF6yjX+RHRA2ViJhGz627ApLDNUterAuw1dXDa3gOx2rt8A== +X-Received: by 2002:a62:571b:: with SMTP id + l27-v6mr3539635pfb.29.1532635682969; + Thu, 26 Jul 2018 13:08:02 -0700 (PDT) +Received: from tictac2.mtv.corp.google.com + ([2620:0:1000:1501:38e4:86fe:ec0c:4007]) + by smtp.gmail.com with ESMTPSA id + u11-v6sm5784169pfd.117.2018.07.26.13.08.01 + (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); + Thu, 26 Jul 2018 13:08:02 -0700 (PDT) +From: Douglas Anderson <dianders@chromium.org> +To: balbi@kernel.org, hminas@synopsys.com, gregkh@linuxfoundation.org +Cc: stefan.wahren@i2se.com, grigor.tovmasyan@synopsys.com, + Bruno Herrera <bruherrera@gmail.com>, + Zhangfei Gao <zhangfei.gao@linaro.org>, john.stultz@linaro.org, + Stephen Warren <swarren@wwwdotorg.org>, + Jerome Brunet <jbrunet@baylibre.com>, + Douglas Anderson <dianders@chromium.org>, + linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org +Subject: [REPOST PATCH 3/4] usb: dwc2: Turn on uframe_sched on "amlogic" + platforms +Date: Thu, 26 Jul 2018 13:07:29 -0700 +Message-Id: <20180726200730.18290-4-dianders@chromium.org> +X-Mailer: git-send-email 2.18.0.345.g5c9ce644c3-goog +In-Reply-To: <20180726200730.18290-1-dianders@chromium.org> +References: <20180726200730.18290-1-dianders@chromium.org> +MIME-Version: 1.0 +Sender: linux-usb-owner@vger.kernel.org +Precedence: bulk +List-ID: <linux-usb.vger.kernel.org> +X-Mailing-List: linux-usb@vger.kernel.org +X-Virus-Scanned: ClamAV using ClamSMTP + +There's no reason to have the uframe scheduler off on dwc2. Running +with uframe_sched = False is equivalent to saying "I don't want to run +the correct code, I want to run the old and incorrect code". + +The uframe scheduler has been off on Amlogic since commit f94310ac076e +("usb: dwc2: add support for Meson8b and GXBB SoCs"). While this was +after most of the recent improvements, notably the commit 9f9f09b048f5 +("usb: dwc2: host: Totally redo the microframe scheduler"), presumably +the parameters were copied from another platform and the uframe +scheduler wasn't tried. + +Presumably if everyone is good w/ the uframe_sched turned back on we +can kill all the old and crufty non-uframe sched code. + +Signed-off-by: Douglas Anderson <dianders@chromium.org> +Reviewed-by: Minas Harutyunyan <hminas@synopsys.com> +--- + + drivers/usb/dwc2/params.c | 1 - + 1 file changed, 1 deletion(-) + +diff --git a/drivers/usb/dwc2/params.c b/drivers/usb/dwc2/params.c +index db97bb16ecc4..93380f7c32b4 100644 +--- a/drivers/usb/dwc2/params.c ++++ b/drivers/usb/dwc2/params.c +@@ -110,7 +110,6 @@ static void dwc2_set_amlogic_params(struct dwc2_hsotg *hsotg) + p->phy_type = DWC2_PHY_TYPE_PARAM_UTMI; + p->ahbcfg = GAHBCFG_HBSTLEN_INCR8 << + GAHBCFG_HBSTLEN_SHIFT; +- p->uframe_sched = false; + } + + static void dwc2_set_amcc_params(struct dwc2_hsotg *hsotg) + +From patchwork Thu Jul 26 20:07:30 2018 +Content-Type: text/plain; charset="utf-8" +MIME-Version: 1.0 +Content-Transfer-Encoding: 7bit +X-Patchwork-Submitter: Doug Anderson <dianders@chromium.org> +X-Patchwork-Id: 10546399 +Return-Path: <linux-usb-owner@kernel.org> +Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org + [172.30.200.125]) + by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id EE5DF14E2 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:08 +0000 (UTC) +Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id E29D12B8D1 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:08 +0000 (UTC) +Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) + id D5BC72BC7A; Thu, 26 Jul 2018 20:08:08 +0000 (UTC) +X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on + pdx-wl-mail.web.codeaurora.org +X-Spam-Level: +X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, + MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham + version=3.3.1 +Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) + by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 777DF2B8D1 + for <patchwork-linux-usb@patchwork.kernel.org>; + Thu, 26 Jul 2018 20:08:08 +0000 (UTC) +Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand + id S1731694AbeGZV0b (ORCPT + <rfc822;patchwork-linux-usb@patchwork.kernel.org>); + Thu, 26 Jul 2018 17:26:31 -0400 +Received: from mail-pl0-f65.google.com ([209.85.160.65]:45724 "EHLO + mail-pl0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org + with ESMTP id S1731687AbeGZV0a (ORCPT + <rfc822;linux-usb@vger.kernel.org>); Thu, 26 Jul 2018 17:26:30 -0400 +Received: by mail-pl0-f65.google.com with SMTP id j8-v6so1282332pll.12 + for <linux-usb@vger.kernel.org>; Thu, 26 Jul 2018 13:08:05 -0700 (PDT) +DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=chromium.org; s=google; + h=from:to:cc:subject:date:message-id:in-reply-to:references + :mime-version:content-transfer-encoding; + bh=Wc5hbhavH7L2NkclXtN89Ag9ST2Hc6CGNYH6SJz0Sjs=; + b=J/2EgAKpJggjM+FxQO4ZigL5pOsa2VS81Zd4JTat8gkRSe4+w+e67RkxPndSCVr9s7 + UWbWWIF9A8qeoHVZawq/yoqmkuv2MVhjaHQi8KGSCgDqBp3Eta0y5zQGH2w/N0Ms34Vq + M+Hhu0wAXOBFW+hBTFs57X8w1Wi+5PT/mZJNQ= +X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=1e100.net; s=20161025; + h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to + :references:mime-version:content-transfer-encoding; + bh=Wc5hbhavH7L2NkclXtN89Ag9ST2Hc6CGNYH6SJz0Sjs=; + b=hp0PTyRjmsnQoZEbfV4IIJw/wZH/hq0NhHKzTLITxRWFLI/Vk6nBTFwDRiTlPrwCSk + d4/9wiEKcw9WM/ZexzGrmHRD5CjTzf0fBvNICUdIq1R/MjiXxpmIQpFrEWbbub4ugk77 + VozHSlefe9yOf8SUyZXudxfl7uSV+PzNSYjsDA1A+9rsqU7f7eZ1AF7i7I4Po6RxfBiH + Dc/wiJNG8Tr+MyEk4L6sOLYDL7Dg0ZQYwdMaosqxvhgLzV1GhEm5SI29daxWF+ZBPNCv + FwN6ScdhHcsolzdiJa8L0DVvBU7tF5Tvc7ZAtjmkv26uclQDtHluAl7PIxfoAgx9RMqX + Gc3g== +X-Gm-Message-State: AOUpUlGRDVF5hQ9ZIzFRUWqXytjll0Gs1k1DVFlgnU4IWAA9SPBRRWsH + 4TTaLsxJbT53PWuFKR9SA6gNgw== +X-Google-Smtp-Source: + AAOMgpf+aFE9GQhs9ldJn5LrKjljoJ5SWqNFcLU7bAEVC6sXMVeMJhRia4DThQe+gBmI0K/dV4X1Ig== +X-Received: by 2002:a17:902:294a:: with SMTP id + g68-v6mr3361812plb.58.1532635684415; + Thu, 26 Jul 2018 13:08:04 -0700 (PDT) +Received: from tictac2.mtv.corp.google.com + ([2620:0:1000:1501:38e4:86fe:ec0c:4007]) + by smtp.gmail.com with ESMTPSA id + u11-v6sm5784169pfd.117.2018.07.26.13.08.03 + (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); + Thu, 26 Jul 2018 13:08:03 -0700 (PDT) +From: Douglas Anderson <dianders@chromium.org> +To: balbi@kernel.org, hminas@synopsys.com, gregkh@linuxfoundation.org +Cc: stefan.wahren@i2se.com, grigor.tovmasyan@synopsys.com, + Bruno Herrera <bruherrera@gmail.com>, + Zhangfei Gao <zhangfei.gao@linaro.org>, john.stultz@linaro.org, + Stephen Warren <swarren@wwwdotorg.org>, + Jerome Brunet <jbrunet@baylibre.com>, + Douglas Anderson <dianders@chromium.org>, + linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org +Subject: [REPOST PATCH 4/4] usb: dwc2: Turn on uframe_sched on + "stm32f4x9_fsotg" platforms +Date: Thu, 26 Jul 2018 13:07:30 -0700 +Message-Id: <20180726200730.18290-5-dianders@chromium.org> +X-Mailer: git-send-email 2.18.0.345.g5c9ce644c3-goog +In-Reply-To: <20180726200730.18290-1-dianders@chromium.org> +References: <20180726200730.18290-1-dianders@chromium.org> +MIME-Version: 1.0 +Sender: linux-usb-owner@vger.kernel.org +Precedence: bulk +List-ID: <linux-usb.vger.kernel.org> +X-Mailing-List: linux-usb@vger.kernel.org +X-Virus-Scanned: ClamAV using ClamSMTP + +There's no reason to have the uframe scheduler off on dwc2. Running +with uframe_sched = False is equivalent to saying "I don't want to run +the correct code, I want to run the old and incorrect code". + +The uframe scheduler has been off on stm32f4x9_fsotg since commit +e35b135055e2 ("usb: dwc2: Add support for STM32F429/439/469 USB OTG +HS/FS in FS mode (internal PHY)"). That commit is pretty recent, so +it's unclear to me why the uframe scheduler was left off. Hopefully +it's because someone copied it from other parameters and didn't think +to try it? + +Presumably if everyone is good w/ the uframe_sched turned back on we +can kill all the old and crufty non-uframe sched code. + +Signed-off-by: Douglas Anderson <dianders@chromium.org> +Reviewed-by: Minas Harutyunyan <hminas@synopsys.com> +--- + + drivers/usb/dwc2/params.c | 1 - + 1 file changed, 1 deletion(-) + +diff --git a/drivers/usb/dwc2/params.c b/drivers/usb/dwc2/params.c +index 93380f7c32b4..7be35bcde713 100644 +--- a/drivers/usb/dwc2/params.c ++++ b/drivers/usb/dwc2/params.c +@@ -131,7 +131,6 @@ static void dwc2_set_stm32f4x9_fsotg_params(struct dwc2_hsotg *hsotg) + p->max_packet_count = 256; + p->phy_type = DWC2_PHY_TYPE_PARAM_FS; + p->i2c_enable = false; +- p->uframe_sched = false; + p->activate_stm_fs_transceiver = true; + } + |