From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759549AbdCVNF1 (ORCPT ); Wed, 22 Mar 2017 09:05:27 -0400 Received: from mga07.intel.com ([134.134.136.100]:40483 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759451AbdCVNE7 (ORCPT ); Wed, 22 Mar 2017 09:04:59 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,205,1486454400"; d="scan'208";a="1125762750" Subject: Re: [PATCH v3] usb: xhci: bInterval quirk for TI TUSB73x0 To: Roger Quadros , mathias.nyman@intel.com References: <1489049268-30607-1-git-send-email-rogerq@ti.com> <235e20da-ad1c-d417-02e8-962eaf657d1d@cogentembedded.com> <1d4406de-c428-a2b2-55b2-5e68d21aab3a@ti.com> <9de96f48-1570-974a-2ceb-acf051a80e6b@ti.com> <787d6f0f-d69c-ba4a-8892-b3558c005184@ti.com> Cc: Sergei Shtylyov , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org From: Mathias Nyman Message-ID: <58D276E4.80607@linux.intel.com> Date: Wed, 22 Mar 2017 15:06:44 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: <787d6f0f-d69c-ba4a-8892-b3558c005184@ti.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 22.03.2017 13:43, Roger Quadros wrote: > As per [1] issue #4, > "The periodic EP scheduler always tries to schedule the EPs > that have large intervals (interval equal to or greater than > 128 microframes) into different microframes. So it maintains > an internal counter and increments for each large interval > EP added. When the counter is greater than 128, the scheduler > rejects the new EP. So when the hub re-enumerated 128 times, > it triggers this condition." > > This results in Bandwidth error when devices with periodic > endpoints (ISO/INT) having bInterval > 7 are plugged and > unplugged several times on a TUSB73x0 XHCI host. > > Workaround this issue by limiting the bInterval to 7 > (i.e. interval to 6) for High-speed or faster periodic endpoints. > > [1] - http://www.ti.com/lit/er/sllz076/sllz076.pdf > > Signed-off-by: Roger Quadros > --- Added to my for-usb-next queue. Thanks -Mathias