Re: [PATCH v3 6/8] usb: dwc2: host: Assume all devices are on one single_tt hub

From: Doug Anderson <dianders@chromium.org>
To: John Youn <John.Youn@synopsys.com>
Cc: "Felipe Balbi" <balbi@ti.com>, "Yunzhi Li" <lyz@rock-chips.com>,
	"Heiko Stübner" <heiko@sntech.de>,
	"open list:ARM/Rockchip SoC..."
	<linux-rockchip@lists.infradead.org>,
	"Julius Werner" <jwerner@chromium.org>,
	"Herrero, Gregory" <gregory.herrero@intel.com>,
	"Kaukab, Yousaf" <yousaf.kaukab@intel.com>,
	"Dinh Nguyen" <dinguyen@opensource.altera.com>,
	"Alan Stern" <stern@rowland.harvard.edu>,
	"Ming Lei" <ming.lei@canonical.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 6/8] usb: dwc2: host: Assume all devices are on one single_tt hub
Date: Wed, 25 Nov 2015 16:44:22 -0800	[thread overview]
Message-ID: <CAD=FV=XtrsN60+VK-Y9ouyswXum4dY0v85x-yAyV3narSvUCGw@mail.gmail.com> (raw)
In-Reply-To: <CAD=FV=UcqEbFuyAq=RpbjmazFGdOyqNgQWQC7UDeL2O1crKn=A@mail.gmail.com>

Hi,

On Mon, Nov 23, 2015 at 4:28 PM, Doug Anderson <dianders@chromium.org> wrote:
> John,
>
> On Thu, Nov 19, 2015 at 8:33 PM, John Youn <John.Youn@synopsys.com> wrote:
>> Patches 1-3:
>> Acked-by: John Youn <johnyoun@synopsys.com>
>>
>> Patch 2:
>> Tested-by: John Youn <johnyoun@synopsys.com>
>>
>> Tested on core version 3.20 using internal TE for un-aligned
>> buffers.
>>
>> I haven't had time to look into the scheduling patches yet. But I
>> agree with you that there are fundamental problems. I'll await
>> your rewrite.
>
> Thanks!  I'm going to attempt to at least get things reposted with
> your tags before Thanksgiving.  I'm hopeful that I'll actually be able
> to post a RFC patch for the microframe rewrite by then, too!  :)
>
> I have something that seems to be sane at
> <https://chromium-review.googlesource.com/#/c/313808/>, but it
> obviously still needs a lot of cleanup.  I also need to pick all of
> the things that have landed on Felipe's tree recently and rebase atop
> those.

I didn't succeed at getting something postable.  :(  I could easily
repost some of the earlier patches (like fixing aligned memory, etc)
if there's a desire for those, but I figured I'd wait until I had
something postable for the microframe scheduler.

I've definitely made progress in the last few days and things are a
bit cleaner at <https://chromium-review.googlesource.com/#/c/313808/3>.
It's also working reasonably well, but I'd still like to understand
some of the behavior I'm seeing before I actually post it to the
mailing list.

I'm going to be away from my computer for about a week, so I'll plan
to pick this up again when I'm back...

-Doug