[{"id":1770244,"web_url":"http://patchwork.ozlabs.org/comment/1770244/","msgid":"<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>","list_archive_url":null,"date":"2017-09-18T15:13:56","subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","submitter":{"id":12009,"url":"http://patchwork.ozlabs.org/api/people/12009/","name":"Marek Vasut","email":"marex@denx.de"},"content":"On 09/18/2017 03:40 PM, Bin Meng wrote:\n> \n> This adds the missing interrupt transfer support to xHCI driver, so\n> that devices like USB keyboard that uses interrupt transfer when\n> CONFIG_SYS_USB_EVENT_POLL is defined can work.\n> \n> This also adds full speed device support. Unlike low/high/super speed\n> devices, full speed device may report its endpoint 0 max packet size\n> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the\n> initial 8 bytes device descriptor, and later adjusts the context for\n> endpoint 0.\n> \n> This series also made several updates to xHCI driver to conform with\n> the spec.\n> \n> This series is available at u-boot-x86/xhci-working for testing.\nLooks good to me. Stefan, can you please give A-B/R-B on the series ?\nI'd like to pick it for 2017.09 :)","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwqP21krvz9s4s\n\tfor ;\n\tTue, 19 Sep 2017 01:17:09 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid B7A86C21F44; Mon, 18 Sep 2017 15:17:04 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id B5F6AC21D90;\n\tMon, 18 Sep 2017 15:17:01 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 19CA5C21DD7; Mon, 18 Sep 2017 15:17:00 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.10])\n\tby lists.denx.de (Postfix) with ESMTPS id CE027C21D90\n\tfor ; Mon, 18 Sep 2017 15:16:59 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xwqNq4LPBz1rdPL;\n\tMon, 18 Sep 2017 17:16:59 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xwqNq3w2Yz1qqkR;\n\tMon, 18 Sep 2017 17:16:59 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id l1a0CPSCk7bP; Mon, 18 Sep 2017 17:16:58 +0200 (CEST)","from [IPv6:::1] (p578adb1c.dip0.t-ipconnect.de [87.138.219.28])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tMon, 18 Sep 2017 17:16:58 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"sQdpLTMc4dMN12iTOG8zrHVAEiCLhqZlTvOO6IWKTho=","To":"Bin Meng , U-Boot Mailing List ","References":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>","From":"Marek Vasut ","Message-ID":"<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>","Date":"Mon, 18 Sep 2017 17:13:56 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>","Content-Language":"en-US","Cc":"Stefan Roese ","Subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1770252,"web_url":"http://patchwork.ozlabs.org/comment/1770252/","msgid":"","list_archive_url":null,"date":"2017-09-18T15:26:38","subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","submitter":{"id":13,"url":"http://patchwork.ozlabs.org/api/people/13/","name":"Stefan Roese","email":"sr@denx.de"},"content":"On 18.09.2017 17:13, Marek Vasut wrote:\n> On 09/18/2017 03:40 PM, Bin Meng wrote:\n>>\n>> This adds the missing interrupt transfer support to xHCI driver, so\n>> that devices like USB keyboard that uses interrupt transfer when\n>> CONFIG_SYS_USB_EVENT_POLL is defined can work.\n>>\n>> This also adds full speed device support. Unlike low/high/super speed\n>> devices, full speed device may report its endpoint 0 max packet size\n>> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the\n>> initial 8 bytes device descriptor, and later adjusts the context for\n>> endpoint 0.\n>>\n>> This series also made several updates to xHCI driver to conform with\n>> the spec.\n>>\n>> This series is available at u-boot-x86/xhci-working for testing.\n> Looks good to me. Stefan, can you please give A-B/R-B on the series ?\n> I'd like to pick it for 2017.09 :)\n\nThat would be 2017.11, right?\n\nSure, I'll review and test this series most likely tomorrow.\n\nThanks,\nStefan","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwqc8286kz9s78\n\tfor ;\n\tTue, 19 Sep 2017 01:26:48 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid E8E77C21F14; Mon, 18 Sep 2017 15:26:43 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id A0530C21DB9;\n\tMon, 18 Sep 2017 15:26:41 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 892FFC21DB9; Mon, 18 Sep 2017 15:26:40 +0000 (UTC)","from mx2.mailbox.org (mx2.mailbox.org [80.241.60.215])\n\tby lists.denx.de (Postfix) with ESMTPS id 3762BC21C4C\n\tfor ; Mon, 18 Sep 2017 15:26:40 +0000 (UTC)","from smtp1.mailbox.org (smtp1.mailbox.org [80.241.60.240])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mx2.mailbox.org (Postfix) with ESMTPS id F1F4A4AF99;\n\tMon, 18 Sep 2017 17:26:39 +0200 (CEST)","from smtp1.mailbox.org ([80.241.60.240])\n\tby spamfilter03.heinlein-hosting.de (spamfilter03.heinlein-hosting.de\n\t[80.241.56.117]) (amavisd-new, port 10030)\n\twith ESMTP id iQI_pYNUVF8J; Mon, 18 Sep 2017 17:26:38 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW\n\tautolearn=unavailable autolearn_force=no version=3.4.0","X-Virus-Scanned":"amavisd-new at heinlein-support.de","To":"Marek Vasut , Bin Meng ,\n\tU-Boot Mailing List ","References":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>\n\t<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>","From":"Stefan Roese ","Message-ID":"","Date":"Mon, 18 Sep 2017 17:26:38 +0200","MIME-Version":"1.0","In-Reply-To":"<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>","Content-Language":"en-US","Subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1770259,"web_url":"http://patchwork.ozlabs.org/comment/1770259/","msgid":"","list_archive_url":null,"date":"2017-09-18T15:38:43","subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","submitter":{"id":13,"url":"http://patchwork.ozlabs.org/api/people/13/","name":"Stefan Roese","email":"sr@denx.de"},"content":"Hi Bin,\n\nOn 18.09.2017 17:26, Stefan Roese wrote:\n> On 18.09.2017 17:13, Marek Vasut wrote:\n>> On 09/18/2017 03:40 PM, Bin Meng wrote:\n>>>\n>>> This adds the missing interrupt transfer support to xHCI driver, so\n>>> that devices like USB keyboard that uses interrupt transfer when\n>>> CONFIG_SYS_USB_EVENT_POLL is defined can work.\n>>>\n>>> This also adds full speed device support. Unlike low/high/super speed\n>>> devices, full speed device may report its endpoint 0 max packet size\n>>> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the\n>>> initial 8 bytes device descriptor, and later adjusts the context for\n>>> endpoint 0.\n>>>\n>>> This series also made several updates to xHCI driver to conform with\n>>> the spec.\n>>>\n>>> This series is available at u-boot-x86/xhci-working for testing.\n>> Looks good to me. Stefan, can you please give A-B/R-B on the series ?\n>> I'd like to pick it for 2017.09 :)\n> \n> That would be 2017.11, right?\n> \n> Sure, I'll review and test this series most likely tomorrow.\n\nI did a quick test and most really looks very good. Thanks for all\nthis great work on this xHCI stuff.\n\nBUT, issuing a few commands, I was able to hang my board. This\nsequence is reproducible on my board:\n\n\nU-Boot 2017.09-00246-g28758a2992 (Sep 18 2017 - 17:29:52 +0200)\n\nCPU: x86_64, vendor Intel, device 30679h\nDRAM: 4 GiB\nMMC: pci_mmc: 0, pci_mmc: 1, pci_mmc: 2\nSF: Detected w25q64cv with page size 256 Bytes, erase size 4 KiB, total 8 MiB\nModel: theadorable-x86-DFI-BT700\nNet: No ethernet found.\nscanning bus for devices...\nTarget spinup took 0 ms.\nSATA link 1 timeout.\nAHCI 0001.0300 32 slots 2 ports 3 Gbps 0x3 impl SATA mode\nflags: 64bit ncq stag pm led clo pio slum part sxs \n Device 0: (0:0) Vendor: ATA Prod.: SanDisk Ultra II Rev: X411\n Type: Hard Disk\n Capacity: 457862.8 MB = 447.1 GB (937703088 x 512)\n960138 bytes read in 72 ms (12.7 MiB/s)\nVideo: 800x600x16\nHit any key to stop autoboot: 0 \n=> \n=> \n=> usb reset\nresetting USB...\nUSB0: Register 7000820 NbrPorts 7\nStarting the controller\nUSB XHCI 1.00\nscanning bus 0 for devices... cannot reset port 1!?\n7 USB Device(s) found\n scanning usb for storage devices... 2 Storage Device(s) found\n=> usb tree\nUSB device tree:\n 1 Hub (5 Gb/s, 0mA)\n | U-Boot XHCI Host Controller \n |\n +-2 Hub (480 Mb/s, 100mA)\n | |\n | +-5 Hub (12 Mb/s, 100mA)\n | \n +-3 Hub (480 Mb/s, 2mA)\n | |\n | +-6 Mass Storage (480 Mb/s, 98mA)\n | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n | | \n | +-7 Mass Storage (480 Mb/s, 500mA)\n | General USB Flash Disk 000000000000034B\n | \n +-4 Vendor specific (5 Gb/s, 64mA)\n Realtek USB 10/100/1000 LAN 000002000000\n \n=> ls usb 1\n System Volume Information/\n 16779264 PRIME-B350-PLUS-ASUS-0613.CAP\n 16779264 PRIME-B350-PLUS-ASUS-0805.CAP\n\n3 file(s), 1 dir(s)\n\n=> usb tree\nUSB device tree:\n 1 Hub (5 Gb/s, 0mA)\n | U-Boot XHCI Host Controller \n |\n +-2 Hub (480 Mb/s, 100mA)\n | |\n | +-5 Hub (12 Mb/s, 100mA)\n | \n +-3 Hub (480 Mb/s, 2mA)\n | |\n | +-6 Mass Storage (480 Mb/s, 98mA)\n | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n | | \n | +-7 Mass Storage (480 Mb/s, 500mA)\n | | General USB Flash Disk 000000000000034B\n | |\n | |General Protection\nEIP: 0010:[<7b56f724>] EFLAGS: 00010246\nOriginal EIP :[]\nEAX: 7b33ac18 EBX: 00000000 ECX: ffffffbf EDX: 7b33ac18\nESI: 7b33ad30 EDI: 00000006 EBP: 7b37b188 ESP: 7b33ac08\n DS: 0018 ES: 0018 FS: 0020 GS: 0018 SS: 0018\nCR0: 00000033 CR2: 00000000 CR3: 00000000 CR4: 00000600\nDR0: 00000000 DR1: 00000000 DR2: 00000000 DR3: 00000000\nDR6: ffff0ff0 DR7: 00000400\nStack:\n 0x7b33ac48 : 0x7b37b160\n 0x7b33ac44 : 0x00000000\n 0x7b33ac40 : 0x00000000\n 0x7b33ac3c : 0x00000000\n 0x7b33ac38 : 0x00000000\n 0x7b33ac34 : 0x20202020\n 0x7b33ac30 : 0x52474d54\n 0x7b33ac2c : 0x7b37baa8\n 0x7b33ac28 : 0x7b37baa8\n 0x7b33ac24 : 0x7b33ad30\n 0x7b33ac20 : 0x7b5aa84e\n 0x7b33ac1c : 0x7b55f008\n 0x7b33ac18 : 0xffffffbf\n 0x7b33ac14 : 0xcc0a06e4\n 0x7b33ac10 : 0xc0e88a40\n 0x7b33ac0c : 0x568ad686\n--->0x7b33ac08 : 0x7b56f776\n 0x7b33ac04 : 0x00010246\n 0x7b33ac00 : 0x00000010\n 0x7b33abfc : 0x7b56f724\n### ERROR ### Please RESET the board ###\n\n\nAre you able to reproduce this on your board(s) as well? Any ideas\non this? Please let me know if I can do any further tests.\n\nThanks,\nStefan","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwqtQ6JxMz9s7G\n\tfor ;\n\tTue, 19 Sep 2017 01:39:09 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 66FD3C21F4D; Mon, 18 Sep 2017 15:38:51 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 85AF8C21D90;\n\tMon, 18 Sep 2017 15:38:47 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 7CB46C21D90; Mon, 18 Sep 2017 15:38:46 +0000 (UTC)","from mx2.mailbox.org (mx2.mailbox.org [80.241.60.215])\n\tby lists.denx.de (Postfix) with ESMTPS id DF116C21C4C\n\tfor ; Mon, 18 Sep 2017 15:38:45 +0000 (UTC)","from smtp1.mailbox.org (smtp1.mailbox.org [80.241.60.240])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mx2.mailbox.org (Postfix) with ESMTPS id B1AFA49A71;\n\tMon, 18 Sep 2017 17:38:45 +0200 (CEST)","from smtp1.mailbox.org ([80.241.60.240])\n\tby spamfilter02.heinlein-hosting.de (spamfilter02.heinlein-hosting.de\n\t[80.241.56.116]) (amavisd-new, port 10030)\n\twith ESMTP id EMbaI6IIaKlv; Mon, 18 Sep 2017 17:38:43 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"*","X-Spam-Status":"No, score=1.8 required=5.0 tests=DRUGS_MUSCLE,\n\tRCVD_IN_DNSWL_LOW\n\tautolearn=no autolearn_force=no version=3.4.0","X-Virus-Scanned":"amavisd-new at heinlein-support.de","From":"Stefan Roese ","To":"Bin Meng , U-Boot Mailing List ","References":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>\n\t<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>\n\t","Message-ID":"","Date":"Mon, 18 Sep 2017 17:38:43 +0200","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","Cc":"Marek Vasut ","Subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1770264,"web_url":"http://patchwork.ozlabs.org/comment/1770264/","msgid":"<94a13757-9415-e3b0-ae14-e4f06921c0e7@denx.de>","list_archive_url":null,"date":"2017-09-18T15:33:06","subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","submitter":{"id":12009,"url":"http://patchwork.ozlabs.org/api/people/12009/","name":"Marek Vasut","email":"marex@denx.de"},"content":"On 09/18/2017 05:26 PM, Stefan Roese wrote:\n> On 18.09.2017 17:13, Marek Vasut wrote:\n>> On 09/18/2017 03:40 PM, Bin Meng wrote:\n>>>\n>>> This adds the missing interrupt transfer support to xHCI driver, so\n>>> that devices like USB keyboard that uses interrupt transfer when\n>>> CONFIG_SYS_USB_EVENT_POLL is defined can work.\n>>>\n>>> This also adds full speed device support. Unlike low/high/super speed\n>>> devices, full speed device may report its endpoint 0 max packet size\n>>> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the\n>>> initial 8 bytes device descriptor, and later adjusts the context for\n>>> endpoint 0.\n>>>\n>>> This series also made several updates to xHCI driver to conform with\n>>> the spec.\n>>>\n>>> This series is available at u-boot-x86/xhci-working for testing.\n>> Looks good to me. Stefan, can you please give A-B/R-B on the series ?\n>> I'd like to pick it for 2017.09 :)\n> \n> That would be 2017.11, right?\n\nYes :)\n\n> Sure, I'll review and test this series most likely tomorrow.\n\nThanks\n\n> Thanks,\n> Stefan","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwr2j0yX1z9s06\n\tfor ;\n\tTue, 19 Sep 2017 01:46:21 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid CAF5CC21E68; Mon, 18 Sep 2017 15:46:18 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5B827C21D90;\n\tMon, 18 Sep 2017 15:46:16 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid D48AAC21D90; Mon, 18 Sep 2017 15:46:14 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id 8FC3FC21C4C\n\tfor ; Mon, 18 Sep 2017 15:46:14 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xwr2Z2gZ7z1rMrT;\n\tMon, 18 Sep 2017 17:46:14 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xwr2Z2G1Dz1qqkG;\n\tMon, 18 Sep 2017 17:46:14 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id mlRi1D-c6E-D; Mon, 18 Sep 2017 17:46:13 +0200 (CEST)","from [IPv6:::1] (p578adb1c.dip0.t-ipconnect.de [87.138.219.28])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tMon, 18 Sep 2017 17:46:13 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"U2yubBuz2zQyDK+9uSNTbIa9ruTXyT62YqL5EaahtlY=","To":"Stefan Roese , Bin Meng ,\n\tU-Boot Mailing List ","References":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>\n\t<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>\n\t","From":"Marek Vasut ","Message-ID":"<94a13757-9415-e3b0-ae14-e4f06921c0e7@denx.de>","Date":"Mon, 18 Sep 2017 17:33:06 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","Subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1770603,"web_url":"http://patchwork.ozlabs.org/comment/1770603/","msgid":"","list_archive_url":null,"date":"2017-09-19T01:38:30","subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","submitter":{"id":64981,"url":"http://patchwork.ozlabs.org/api/people/64981/","name":"Bin Meng","email":"bmeng.cn@gmail.com"},"content":"Hi Stefan,\n\nOn Mon, Sep 18, 2017 at 11:38 PM, Stefan Roese wrote:\n> Hi Bin,\n>\n> On 18.09.2017 17:26, Stefan Roese wrote:\n>> On 18.09.2017 17:13, Marek Vasut wrote:\n>>> On 09/18/2017 03:40 PM, Bin Meng wrote:\n>>>>\n>>>> This adds the missing interrupt transfer support to xHCI driver, so\n>>>> that devices like USB keyboard that uses interrupt transfer when\n>>>> CONFIG_SYS_USB_EVENT_POLL is defined can work.\n>>>>\n>>>> This also adds full speed device support. Unlike low/high/super speed\n>>>> devices, full speed device may report its endpoint 0 max packet size\n>>>> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the\n>>>> initial 8 bytes device descriptor, and later adjusts the context for\n>>>> endpoint 0.\n>>>>\n>>>> This series also made several updates to xHCI driver to conform with\n>>>> the spec.\n>>>>\n>>>> This series is available at u-boot-x86/xhci-working for testing.\n>>> Looks good to me. Stefan, can you please give A-B/R-B on the series ?\n>>> I'd like to pick it for 2017.09 :)\n>>\n>> That would be 2017.11, right?\n>>\n>> Sure, I'll review and test this series most likely tomorrow.\n>\n> I did a quick test and most really looks very good. Thanks for all\n> this great work on this xHCI stuff.\n>\n> BUT, issuing a few commands, I was able to hang my board. This\n> sequence is reproducible on my board:\n>\n>\n> U-Boot 2017.09-00246-g28758a2992 (Sep 18 2017 - 17:29:52 +0200)\n>\n> CPU: x86_64, vendor Intel, device 30679h\n> DRAM: 4 GiB\n> MMC: pci_mmc: 0, pci_mmc: 1, pci_mmc: 2\n> SF: Detected w25q64cv with page size 256 Bytes, erase size 4 KiB, total 8 MiB\n> Model: theadorable-x86-DFI-BT700\n> Net: No ethernet found.\n> scanning bus for devices...\n> Target spinup took 0 ms.\n> SATA link 1 timeout.\n> AHCI 0001.0300 32 slots 2 ports 3 Gbps 0x3 impl SATA mode\n> flags: 64bit ncq stag pm led clo pio slum part sxs\n> Device 0: (0:0) Vendor: ATA Prod.: SanDisk Ultra II Rev: X411\n> Type: Hard Disk\n> Capacity: 457862.8 MB = 447.1 GB (937703088 x 512)\n> 960138 bytes read in 72 ms (12.7 MiB/s)\n> Video: 800x600x16\n> Hit any key to stop autoboot: 0\n> =>\n> =>\n> => usb reset\n> resetting USB...\n> USB0: Register 7000820 NbrPorts 7\n> Starting the controller\n> USB XHCI 1.00\n> scanning bus 0 for devices... cannot reset port 1!?\n> 7 USB Device(s) found\n> scanning usb for storage devices... 2 Storage Device(s) found\n> => usb tree\n> USB device tree:\n> 1 Hub (5 Gb/s, 0mA)\n> | U-Boot XHCI Host Controller\n> |\n> +-2 Hub (480 Mb/s, 100mA)\n> | |\n> | +-5 Hub (12 Mb/s, 100mA)\n> |\n> +-3 Hub (480 Mb/s, 2mA)\n> | |\n> | +-6 Mass Storage (480 Mb/s, 98mA)\n> | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n> | |\n> | +-7 Mass Storage (480 Mb/s, 500mA)\n> | General USB Flash Disk 000000000000034B\n> |\n> +-4 Vendor specific (5 Gb/s, 64mA)\n> Realtek USB 10/100/1000 LAN 000002000000\n>\n> => ls usb 1\n> System Volume Information/\n> 16779264 PRIME-B350-PLUS-ASUS-0613.CAP\n> 16779264 PRIME-B350-PLUS-ASUS-0805.CAP\n>\n> 3 file(s), 1 dir(s)\n>\n> => usb tree\n> USB device tree:\n> 1 Hub (5 Gb/s, 0mA)\n> | U-Boot XHCI Host Controller\n> |\n> +-2 Hub (480 Mb/s, 100mA)\n> | |\n> | +-5 Hub (12 Mb/s, 100mA)\n> |\n> +-3 Hub (480 Mb/s, 2mA)\n> | |\n> | +-6 Mass Storage (480 Mb/s, 98mA)\n> | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n> | |\n> | +-7 Mass Storage (480 Mb/s, 500mA)\n> | | General USB Flash Disk 000000000000034B\n> | |\n> | |General Protection\n> EIP: 0010:[<7b56f724>] EFLAGS: 00010246\n> Original EIP :[]\n> EAX: 7b33ac18 EBX: 00000000 ECX: ffffffbf EDX: 7b33ac18\n> ESI: 7b33ad30 EDI: 00000006 EBP: 7b37b188 ESP: 7b33ac08\n> DS: 0018 ES: 0018 FS: 0020 GS: 0018 SS: 0018\n> CR0: 00000033 CR2: 00000000 CR3: 00000000 CR4: 00000600\n> DR0: 00000000 DR1: 00000000 DR2: 00000000 DR3: 00000000\n> DR6: ffff0ff0 DR7: 00000400\n> Stack:\n> 0x7b33ac48 : 0x7b37b160\n> 0x7b33ac44 : 0x00000000\n> 0x7b33ac40 : 0x00000000\n> 0x7b33ac3c : 0x00000000\n> 0x7b33ac38 : 0x00000000\n> 0x7b33ac34 : 0x20202020\n> 0x7b33ac30 : 0x52474d54\n> 0x7b33ac2c : 0x7b37baa8\n> 0x7b33ac28 : 0x7b37baa8\n> 0x7b33ac24 : 0x7b33ad30\n> 0x7b33ac20 : 0x7b5aa84e\n> 0x7b33ac1c : 0x7b55f008\n> 0x7b33ac18 : 0xffffffbf\n> 0x7b33ac14 : 0xcc0a06e4\n> 0x7b33ac10 : 0xc0e88a40\n> 0x7b33ac0c : 0x568ad686\n> --->0x7b33ac08 : 0x7b56f776\n> 0x7b33ac04 : 0x00010246\n> 0x7b33ac00 : 0x00000010\n> 0x7b33abfc : 0x7b56f724\n> ### ERROR ### Please RESET the board ###\n>\n>\n> Are you able to reproduce this on your board(s) as well? Any ideas\n> on this? Please let me know if I can do any further tests.\n\nThis is an known issue of 'usb info' and 'usb tree' command. See\nprevious patch on ML @ http://patchwork.ozlabs.org/patch/810732/.\nHowever, that patch has serious issues and needs rework. I can rework\nthat patch and include it as part of v2 if you like.\n\nRegards,\nBin","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"RjX/BAuB\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xx5BC3yypz9ryQ\n\tfor ;\n\tTue, 19 Sep 2017 11:38:42 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid B429CC21DA1; Tue, 19 Sep 2017 01:38:37 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id B9092C21C3F;\n\tTue, 19 Sep 2017 01:38:34 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid A4EEBC21C3F; Tue, 19 Sep 2017 01:38:32 +0000 (UTC)","from mail-wm0-f41.google.com (mail-wm0-f41.google.com\n\t[74.125.82.41])\n\tby lists.denx.de (Postfix) with ESMTPS id 39BE5C21C34\n\tfor ; Tue, 19 Sep 2017 01:38:31 +0000 (UTC)","by mail-wm0-f41.google.com with SMTP id 13so1184178wmq.2\n\tfor ; Mon, 18 Sep 2017 18:38:31 -0700 (PDT)","by 10.223.145.3 with HTTP; Mon, 18 Sep 2017 18:38:30 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"**","X-Spam-Status":"No, score=2.5 required=5.0 tests=DRUGS_MUSCLE, FREEMAIL_FROM, \n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=D3oJUe5NAS/5TpQ1iL0xtZrEnLk0ZLvQkeHqIjBTuZo=;\n\tb=RjX/BAuBl3AwFbPefQ2bcmhNASjZlK4FSwmXNQnFAnVDmyvs0y6cu25oHDWUPMUAp6\n\teUAWYUv+ISnqR0FkCa03V7/GmpTBQBlLA7iaAWYXUYxoUkiCTksX9x/0hn/Jw7tDjtcu\n\t0veuaqytQHVkkHwZmwyKq//vT4zCrYwjMyfWZm3ihUD2mm0U7a6FQWJ6fGgOR9Kdno8G\n\tww2aRlVsVoCWxLZq8leoA8leFIZEdPHLGz+nl//ax4K7qiC8KWr6EpjH929FrUxEIJkI\n\tQLy8sHIUkNM6piWmtMbW0ZlA5QL0iaEHn9rkCTFTi5juw/dj9tgQY3pMoRGraHaWa5NV\n\tB32Q==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=D3oJUe5NAS/5TpQ1iL0xtZrEnLk0ZLvQkeHqIjBTuZo=;\n\tb=db3SHGFlAsIjdzBqHq21fLcHaPrUGbE88MwUocteT7XRiisj0BVCJ9FL1x7QeB0Xwa\n\tc/vBM5TvSpD1Ydt9ewgFcPeYDsA+/bZcAQEdTaCfUGHonIRbKEwNrMFoS4MJ9vCNT4m+\n\tA45gxdO9p3xPRDd+aZKHXQ6KfkoXNNM8Nz78GnE/lcn+1HkTYDlfl6AWGxSERVuqDsB7\n\t9bt9rHmwMcthqm3Pi2p5sngOthaFVntxICTpBs7V46lfhBD7FRnegmqAvTWuGcXWwXd9\n\tuTMIwLtfrAHyHAkpE3zdctbBvw+qSZwMg+6FWpZhakskWcFIRmN5PFbvHGcJ/O9gw0on\n\toWJw==","X-Gm-Message-State":"AHPjjUgP8AsULbvM+RQ452HuyX5SUNxgpPPUzx+fi45M3eIW2LB/1fFh\n\t8IUPHerZOG+y0dHM69nVi9TqA6Q/WyaeEhGAGkPewA==","X-Google-Smtp-Source":"AOwi7QAr4Yybags8xIkXFCqRSkkUST+yC2WqWautYgzoLvifdnix1Yo8qcQpELY9reTLTMe34tI4jtMEVWO88Ak4Fa8=","X-Received":"by 10.28.63.145 with SMTP id m139mr71890wma.5.1505785110753; Mon,\n\t18 Sep 2017 18:38:30 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"","References":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>\n\t<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>\n\t\n\t","From":"Bin Meng ","Date":"Tue, 19 Sep 2017 09:38:30 +0800","Message-ID":"","To":"Stefan Roese ","Cc":"Marek Vasut , U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1770651,"web_url":"http://patchwork.ozlabs.org/comment/1770651/","msgid":"<31c23c74-d376-69e2-3628-de954ccf67f5@denx.de>","list_archive_url":null,"date":"2017-09-19T04:54:22","subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","submitter":{"id":13,"url":"http://patchwork.ozlabs.org/api/people/13/","name":"Stefan Roese","email":"sr@denx.de"},"content":"Hi Bin,\n\nOn 19.09.2017 03:38, Bin Meng wrote:\n> On Mon, Sep 18, 2017 at 11:38 PM, Stefan Roese wrote:\n>> Hi Bin,\n>>\n>> On 18.09.2017 17:26, Stefan Roese wrote:\n>>> On 18.09.2017 17:13, Marek Vasut wrote:\n>>>> On 09/18/2017 03:40 PM, Bin Meng wrote:\n>>>>>\n>>>>> This adds the missing interrupt transfer support to xHCI driver, so\n>>>>> that devices like USB keyboard that uses interrupt transfer when\n>>>>> CONFIG_SYS_USB_EVENT_POLL is defined can work.\n>>>>>\n>>>>> This also adds full speed device support. Unlike low/high/super speed\n>>>>> devices, full speed device may report its endpoint 0 max packet size\n>>>>> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the\n>>>>> initial 8 bytes device descriptor, and later adjusts the context for\n>>>>> endpoint 0.\n>>>>>\n>>>>> This series also made several updates to xHCI driver to conform with\n>>>>> the spec.\n>>>>>\n>>>>> This series is available at u-boot-x86/xhci-working for testing.\n>>>> Looks good to me. Stefan, can you please give A-B/R-B on the series ?\n>>>> I'd like to pick it for 2017.09 :)\n>>>\n>>> That would be 2017.11, right?\n>>>\n>>> Sure, I'll review and test this series most likely tomorrow.\n>>\n>> I did a quick test and most really looks very good. Thanks for all\n>> this great work on this xHCI stuff.\n>>\n>> BUT, issuing a few commands, I was able to hang my board. This\n>> sequence is reproducible on my board:\n>>\n>>\n>> U-Boot 2017.09-00246-g28758a2992 (Sep 18 2017 - 17:29:52 +0200)\n>>\n>> CPU: x86_64, vendor Intel, device 30679h\n>> DRAM: 4 GiB\n>> MMC: pci_mmc: 0, pci_mmc: 1, pci_mmc: 2\n>> SF: Detected w25q64cv with page size 256 Bytes, erase size 4 KiB, total 8 MiB\n>> Model: theadorable-x86-DFI-BT700\n>> Net: No ethernet found.\n>> scanning bus for devices...\n>> Target spinup took 0 ms.\n>> SATA link 1 timeout.\n>> AHCI 0001.0300 32 slots 2 ports 3 Gbps 0x3 impl SATA mode\n>> flags: 64bit ncq stag pm led clo pio slum part sxs\n>> Device 0: (0:0) Vendor: ATA Prod.: SanDisk Ultra II Rev: X411\n>> Type: Hard Disk\n>> Capacity: 457862.8 MB = 447.1 GB (937703088 x 512)\n>> 960138 bytes read in 72 ms (12.7 MiB/s)\n>> Video: 800x600x16\n>> Hit any key to stop autoboot: 0\n>> =>\n>> =>\n>> => usb reset\n>> resetting USB...\n>> USB0: Register 7000820 NbrPorts 7\n>> Starting the controller\n>> USB XHCI 1.00\n>> scanning bus 0 for devices... cannot reset port 1!?\n>> 7 USB Device(s) found\n>> scanning usb for storage devices... 2 Storage Device(s) found\n>> => usb tree\n>> USB device tree:\n>> 1 Hub (5 Gb/s, 0mA)\n>> | U-Boot XHCI Host Controller\n>> |\n>> +-2 Hub (480 Mb/s, 100mA)\n>> | |\n>> | +-5 Hub (12 Mb/s, 100mA)\n>> |\n>> +-3 Hub (480 Mb/s, 2mA)\n>> | |\n>> | +-6 Mass Storage (480 Mb/s, 98mA)\n>> | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n>> | |\n>> | +-7 Mass Storage (480 Mb/s, 500mA)\n>> | General USB Flash Disk 000000000000034B\n>> |\n>> +-4 Vendor specific (5 Gb/s, 64mA)\n>> Realtek USB 10/100/1000 LAN 000002000000\n>>\n>> => ls usb 1\n>> System Volume Information/\n>> 16779264 PRIME-B350-PLUS-ASUS-0613.CAP\n>> 16779264 PRIME-B350-PLUS-ASUS-0805.CAP\n>>\n>> 3 file(s), 1 dir(s)\n>>\n>> => usb tree\n>> USB device tree:\n>> 1 Hub (5 Gb/s, 0mA)\n>> | U-Boot XHCI Host Controller\n>> |\n>> +-2 Hub (480 Mb/s, 100mA)\n>> | |\n>> | +-5 Hub (12 Mb/s, 100mA)\n>> |\n>> +-3 Hub (480 Mb/s, 2mA)\n>> | |\n>> | +-6 Mass Storage (480 Mb/s, 98mA)\n>> | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n>> | |\n>> | +-7 Mass Storage (480 Mb/s, 500mA)\n>> | | General USB Flash Disk 000000000000034B\n>> | |\n>> | |General Protection\n>> EIP: 0010:[<7b56f724>] EFLAGS: 00010246\n>> Original EIP :[]\n>> EAX: 7b33ac18 EBX: 00000000 ECX: ffffffbf EDX: 7b33ac18\n>> ESI: 7b33ad30 EDI: 00000006 EBP: 7b37b188 ESP: 7b33ac08\n>> DS: 0018 ES: 0018 FS: 0020 GS: 0018 SS: 0018\n>> CR0: 00000033 CR2: 00000000 CR3: 00000000 CR4: 00000600\n>> DR0: 00000000 DR1: 00000000 DR2: 00000000 DR3: 00000000\n>> DR6: ffff0ff0 DR7: 00000400\n>> Stack:\n>> 0x7b33ac48 : 0x7b37b160\n>> 0x7b33ac44 : 0x00000000\n>> 0x7b33ac40 : 0x00000000\n>> 0x7b33ac3c : 0x00000000\n>> 0x7b33ac38 : 0x00000000\n>> 0x7b33ac34 : 0x20202020\n>> 0x7b33ac30 : 0x52474d54\n>> 0x7b33ac2c : 0x7b37baa8\n>> 0x7b33ac28 : 0x7b37baa8\n>> 0x7b33ac24 : 0x7b33ad30\n>> 0x7b33ac20 : 0x7b5aa84e\n>> 0x7b33ac1c : 0x7b55f008\n>> 0x7b33ac18 : 0xffffffbf\n>> 0x7b33ac14 : 0xcc0a06e4\n>> 0x7b33ac10 : 0xc0e88a40\n>> 0x7b33ac0c : 0x568ad686\n>> --->0x7b33ac08 : 0x7b56f776\n>> 0x7b33ac04 : 0x00010246\n>> 0x7b33ac00 : 0x00000010\n>> 0x7b33abfc : 0x7b56f724\n>> ### ERROR ### Please RESET the board ###\n>>\n>>\n>> Are you able to reproduce this on your board(s) as well? Any ideas\n>> on this? Please let me know if I can do any further tests.\n> \n> This is an known issue of 'usb info' and 'usb tree' command. See\n> previous patch on ML @ http://patchwork.ozlabs.org/patch/810732/.\n> However, that patch has serious issues and needs rework. I can rework\n> that patch and include it as part of v2 if you like.\n\nAh, thanks. I had noticed this patch but was not aware of the\nseverity here.\n\nI've tested with the v2 and it seems to work just fine now.\n\nThanks,\nStefan","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xx9X93LVFz9s5L\n\tfor ;\n\tTue, 19 Sep 2017 14:54:33 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 2B6F8C21E27; Tue, 19 Sep 2017 04:54:29 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id AECB9C21C54;\n\tTue, 19 Sep 2017 04:54:25 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 1E801C21C54; Tue, 19 Sep 2017 04:54:25 +0000 (UTC)","from mx2.mailbox.org (mx2.mailbox.org [80.241.60.215])\n\tby lists.denx.de (Postfix) with ESMTPS id A8753C21C3F\n\tfor ; Tue, 19 Sep 2017 04:54:24 +0000 (UTC)","from smtp1.mailbox.org (smtp1.mailbox.org [80.241.60.240])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mx2.mailbox.org (Postfix) with ESMTPS id 827904B33B;\n\tTue, 19 Sep 2017 06:54:24 +0200 (CEST)","from smtp1.mailbox.org ([80.241.60.240])\n\tby hefe.heinlein-support.de (hefe.heinlein-support.de\n\t[91.198.250.172]) (amavisd-new, port 10030)\n\twith ESMTP id nPmoyYHMiU3f; Tue, 19 Sep 2017 06:54:22 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"*","X-Spam-Status":"No, score=1.8 required=5.0 tests=DRUGS_MUSCLE,\n\tRCVD_IN_DNSWL_LOW\n\tautolearn=no autolearn_force=no version=3.4.0","X-Virus-Scanned":"amavisd-new at heinlein-support.de","To":"Bin Meng ","References":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>\n\t<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>\n\t\n\t\n\t","From":"Stefan Roese ","Message-ID":"<31c23c74-d376-69e2-3628-de954ccf67f5@denx.de>","Date":"Tue, 19 Sep 2017 06:54:22 +0200","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","Cc":"Marek Vasut , U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1770652,"web_url":"http://patchwork.ozlabs.org/comment/1770652/","msgid":"","list_archive_url":null,"date":"2017-09-19T04:58:47","subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","submitter":{"id":64981,"url":"http://patchwork.ozlabs.org/api/people/64981/","name":"Bin Meng","email":"bmeng.cn@gmail.com"},"content":"Hi Stefan,\n\nOn Tue, Sep 19, 2017 at 12:54 PM, Stefan Roese wrote:\n> Hi Bin,\n>\n>\n> On 19.09.2017 03:38, Bin Meng wrote:\n>>\n>> On Mon, Sep 18, 2017 at 11:38 PM, Stefan Roese wrote:\n>>>\n>>> Hi Bin,\n>>>\n>>> On 18.09.2017 17:26, Stefan Roese wrote:\n>>>>\n>>>> On 18.09.2017 17:13, Marek Vasut wrote:\n>>>>>\n>>>>> On 09/18/2017 03:40 PM, Bin Meng wrote:\n>>>>>>\n>>>>>>\n>>>>>> This adds the missing interrupt transfer support to xHCI driver, so\n>>>>>> that devices like USB keyboard that uses interrupt transfer when\n>>>>>> CONFIG_SYS_USB_EVENT_POLL is defined can work.\n>>>>>>\n>>>>>> This also adds full speed device support. Unlike low/high/super speed\n>>>>>> devices, full speed device may report its endpoint 0 max packet size\n>>>>>> as 8/16/32/64. xHCI driver guesses 64 for the first attempt to get the\n>>>>>> initial 8 bytes device descriptor, and later adjusts the context for\n>>>>>> endpoint 0.\n>>>>>>\n>>>>>> This series also made several updates to xHCI driver to conform with\n>>>>>> the spec.\n>>>>>>\n>>>>>> This series is available at u-boot-x86/xhci-working for testing.\n>>>>>\n>>>>> Looks good to me. Stefan, can you please give A-B/R-B on the series ?\n>>>>> I'd like to pick it for 2017.09 :)\n>>>>\n>>>>\n>>>> That would be 2017.11, right?\n>>>>\n>>>> Sure, I'll review and test this series most likely tomorrow.\n>>>\n>>>\n>>> I did a quick test and most really looks very good. Thanks for all\n>>> this great work on this xHCI stuff.\n>>>\n>>> BUT, issuing a few commands, I was able to hang my board. This\n>>> sequence is reproducible on my board:\n>>>\n>>>\n>>> U-Boot 2017.09-00246-g28758a2992 (Sep 18 2017 - 17:29:52 +0200)\n>>>\n>>> CPU: x86_64, vendor Intel, device 30679h\n>>> DRAM: 4 GiB\n>>> MMC: pci_mmc: 0, pci_mmc: 1, pci_mmc: 2\n>>> SF: Detected w25q64cv with page size 256 Bytes, erase size 4 KiB, total 8\n>>> MiB\n>>> Model: theadorable-x86-DFI-BT700\n>>> Net: No ethernet found.\n>>> scanning bus for devices...\n>>> Target spinup took 0 ms.\n>>> SATA link 1 timeout.\n>>> AHCI 0001.0300 32 slots 2 ports 3 Gbps 0x3 impl SATA mode\n>>> flags: 64bit ncq stag pm led clo pio slum part sxs\n>>> Device 0: (0:0) Vendor: ATA Prod.: SanDisk Ultra II Rev: X411\n>>> Type: Hard Disk\n>>> Capacity: 457862.8 MB = 447.1 GB (937703088 x 512)\n>>> 960138 bytes read in 72 ms (12.7 MiB/s)\n>>> Video: 800x600x16\n>>> Hit any key to stop autoboot: 0\n>>> =>\n>>> =>\n>>> => usb reset\n>>> resetting USB...\n>>> USB0: Register 7000820 NbrPorts 7\n>>> Starting the controller\n>>> USB XHCI 1.00\n>>> scanning bus 0 for devices... cannot reset port 1!?\n>>> 7 USB Device(s) found\n>>> scanning usb for storage devices... 2 Storage Device(s) found\n>>> => usb tree\n>>> USB device tree:\n>>> 1 Hub (5 Gb/s, 0mA)\n>>> | U-Boot XHCI Host Controller\n>>> |\n>>> +-2 Hub (480 Mb/s, 100mA)\n>>> | |\n>>> | +-5 Hub (12 Mb/s, 100mA)\n>>> |\n>>> +-3 Hub (480 Mb/s, 2mA)\n>>> | |\n>>> | +-6 Mass Storage (480 Mb/s, 98mA)\n>>> | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n>>> | |\n>>> | +-7 Mass Storage (480 Mb/s, 500mA)\n>>> | General USB Flash Disk 000000000000034B\n>>> |\n>>> +-4 Vendor specific (5 Gb/s, 64mA)\n>>> Realtek USB 10/100/1000 LAN 000002000000\n>>>\n>>> => ls usb 1\n>>> System Volume Information/\n>>> 16779264 PRIME-B350-PLUS-ASUS-0613.CAP\n>>> 16779264 PRIME-B350-PLUS-ASUS-0805.CAP\n>>>\n>>> 3 file(s), 1 dir(s)\n>>>\n>>> => usb tree\n>>> USB device tree:\n>>> 1 Hub (5 Gb/s, 0mA)\n>>> | U-Boot XHCI Host Controller\n>>> |\n>>> +-2 Hub (480 Mb/s, 100mA)\n>>> | |\n>>> | +-5 Hub (12 Mb/s, 100mA)\n>>> |\n>>> +-3 Hub (480 Mb/s, 2mA)\n>>> | |\n>>> | +-6 Mass Storage (480 Mb/s, 98mA)\n>>> | | USBest Technology USB Mass Storage Device 09092207fbf0c4\n>>> | |\n>>> | +-7 Mass Storage (480 Mb/s, 500mA)\n>>> | | General USB Flash Disk 000000000000034B\n>>> | |\n>>> | |General Protection\n>>> EIP: 0010:[<7b56f724>] EFLAGS: 00010246\n>>> Original EIP :[]\n>>> EAX: 7b33ac18 EBX: 00000000 ECX: ffffffbf EDX: 7b33ac18\n>>> ESI: 7b33ad30 EDI: 00000006 EBP: 7b37b188 ESP: 7b33ac08\n>>> DS: 0018 ES: 0018 FS: 0020 GS: 0018 SS: 0018\n>>> CR0: 00000033 CR2: 00000000 CR3: 00000000 CR4: 00000600\n>>> DR0: 00000000 DR1: 00000000 DR2: 00000000 DR3: 00000000\n>>> DR6: ffff0ff0 DR7: 00000400\n>>> Stack:\n>>> 0x7b33ac48 : 0x7b37b160\n>>> 0x7b33ac44 : 0x00000000\n>>> 0x7b33ac40 : 0x00000000\n>>> 0x7b33ac3c : 0x00000000\n>>> 0x7b33ac38 : 0x00000000\n>>> 0x7b33ac34 : 0x20202020\n>>> 0x7b33ac30 : 0x52474d54\n>>> 0x7b33ac2c : 0x7b37baa8\n>>> 0x7b33ac28 : 0x7b37baa8\n>>> 0x7b33ac24 : 0x7b33ad30\n>>> 0x7b33ac20 : 0x7b5aa84e\n>>> 0x7b33ac1c : 0x7b55f008\n>>> 0x7b33ac18 : 0xffffffbf\n>>> 0x7b33ac14 : 0xcc0a06e4\n>>> 0x7b33ac10 : 0xc0e88a40\n>>> 0x7b33ac0c : 0x568ad686\n>>> --->0x7b33ac08 : 0x7b56f776\n>>> 0x7b33ac04 : 0x00010246\n>>> 0x7b33ac00 : 0x00000010\n>>> 0x7b33abfc : 0x7b56f724\n>>> ### ERROR ### Please RESET the board ###\n>>>\n>>>\n>>> Are you able to reproduce this on your board(s) as well? Any ideas\n>>> on this? Please let me know if I can do any further tests.\n>>\n>>\n>> This is an known issue of 'usb info' and 'usb tree' command. See\n>> previous patch on ML @ http://patchwork.ozlabs.org/patch/810732/.\n>> However, that patch has serious issues and needs rework. I can rework\n>> that patch and include it as part of v2 if you like.\n>\n>\n> Ah, thanks. I had noticed this patch but was not aware of the\n> severity here.\n>\n> I've tested with the v2 and it seems to work just fine now.\n\nYep, I posted some comments to that v2 patch.\n\nFor this xHCI series, I've moved the patch state back to \"New\".\n\nRegards,\nBin","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"RsMzTwSl\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xx9dH4zGgz9s5L\n\tfor ;\n\tTue, 19 Sep 2017 14:58:59 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 633E5C21E1C; Tue, 19 Sep 2017 04:58:54 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id AA145C21C54;\n\tTue, 19 Sep 2017 04:58:50 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid EB5C4C21C54; Tue, 19 Sep 2017 04:58:48 +0000 (UTC)","from mail-wm0-f47.google.com (mail-wm0-f47.google.com\n\t[74.125.82.47])\n\tby lists.denx.de (Postfix) with ESMTPS id 965D2C21C3F\n\tfor ; Tue, 19 Sep 2017 04:58:48 +0000 (UTC)","by mail-wm0-f47.google.com with SMTP id e71so1733629wmg.4\n\tfor ; Mon, 18 Sep 2017 21:58:48 -0700 (PDT)","by 10.223.145.3 with HTTP; Mon, 18 Sep 2017 21:58:47 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"**","X-Spam-Status":"No, score=2.5 required=5.0 tests=DRUGS_MUSCLE, FREEMAIL_FROM, \n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=Izh/BHZvssUFX7HgXw9oEeuYeQpnnlY+55C0m8pPvIY=;\n\tb=RsMzTwSlBcrtBtsJ/wEjRX6PNcT9fUYTb3sTx5L7H1eIIJWfPdI2iurEqGQ+CG9owd\n\t3xRwzdQGvwL+fJbLxBZ3weMA/fBGiOguocBa1XJsWSGAx9xR86Bc0UxrMLDBvaxZeaMA\n\tMqWupIBElR7cEoNQ0TzXz4YfK/J6vDXGfFxVBchPhoyF5zt150MnW6zhxpD5it0VpAtJ\n\trLW8tfjzm8pCLERhFxMmDeCXg4UMJ3yW41iPnxGN5pnEaB7NUuXZJz/rkE9/PiG7xovK\n\tjwrc6mwOY6Pjnm4coeUg/KtmEoZ38QgIgs8LWoDmG97I4borTx7FOvMGClYPoyPqyWx+\n\tWfDw==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=Izh/BHZvssUFX7HgXw9oEeuYeQpnnlY+55C0m8pPvIY=;\n\tb=QvhbbdowtDpOljOL06fW3v3yCix4/5MQBCgQPf6jxf54YrxGdOSdcaAwP8sf9kT/Bl\n\t8qFCnXAI2QrVMrAm/gzDZZS0NqoM6xDh2ngTobJC6TzpPU+6h6M/JXcsbBRpUwJ6Oys+\n\tJInes/Xx8+pIT43lphRoE2pyAtz0/UkWsExis8pf+1nLmHR28BnJCksq1/86wHYt3kxk\n\taakfwlyjp+LX6c42OmcNIqeTCYUNU34b5cmnSQ0tMp2lH0TkC2W1iFP4XIYt35MpFKTH\n\tj3cgR37IZ0YuJYgCTlhlwBNf1NcrCCGYx7Kzp+FI2mrQtkvmf1GjOEyi3i4BkqZgGmDy\n\tRQIg==","X-Gm-Message-State":"AHPjjUgETaIX/o7d+5m8iCDjbXEu4OYbQlarQr7FKH6xwqxFgpby9U20\n\tn5C4+CeIDkMBjDwo2VEc4z28N+BdQwRyUGOqWwI=","X-Google-Smtp-Source":"AOwi7QAEgoU2EWl171RfP1UJy4BOY5EcXAPq0lvQkrAOly6XRuESRQg2Y9qUeZjkVLda0TJ03Z9OcmTUa0+wEWNLcbQ=","X-Received":"by 10.28.238.218 with SMTP id j87mr355891wmi.44.1505797128103;\n\tMon, 18 Sep 2017 21:58:48 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<31c23c74-d376-69e2-3628-de954ccf67f5@denx.de>","References":"<1505742050-5697-1-git-send-email-bmeng.cn@gmail.com>\n\t<32060654-a956-3e4f-c8df-bdf50c43e5a8@denx.de>\n\t\n\t\n\t\n\t<31c23c74-d376-69e2-3628-de954ccf67f5@denx.de>","From":"Bin Meng ","Date":"Tue, 19 Sep 2017 12:58:47 +0800","Message-ID":"","To":"Stefan Roese ","Cc":"Marek Vasut , U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 00/14] usb: xhci: Add interrupt transfer\n\tsupport and full speed device support","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]