From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D04D3EB64D7 for ; Tue, 20 Jun 2023 09:25:59 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id E5425861CE; Tue, 20 Jun 2023 11:25:57 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=denx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1687253158; bh=yYcbzE1o+qSg+x+DGmykmAgHj67+jhZws5i4lWTgiFI=; h=Date:Subject:To:Cc:References:From:In-Reply-To:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=inYWNHGEboJZCl66SNjADA4UjmCWebOJ4astzinJxum72e5ypO/+RhmXgsfSA+Jlw AqvogGEAyS4FTalDCLzGbEZusEPeqW2uq0d0lFYjpaOftMnv03VhwlL4r6zldrb+tB oZt6XOwjUUpBkn8diqcnijZChjhRFfkXowM8WKJwOMXacfT8Hr31pBlnYbQDVP7obE YXmjRugMTXgjiNp6xLJoASMkWHg1ds8xvt7kXA8pHNZZcuhbwYQCtkOZPnL3gNp6GQ gZHUzoxg61Io6Q4Zku8aGlf1M9NmFjZXH5Q6p+SD95EqwOOcLOeQu+DK2MTG8rBli+ Liy6GjB3bTCQg== Received: from [127.0.0.1] (p578adb1c.dip0.t-ipconnect.de [87.138.219.28]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: marex@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id DF6EE84688; Tue, 20 Jun 2023 11:25:55 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1687253156; bh=yYcbzE1o+qSg+x+DGmykmAgHj67+jhZws5i4lWTgiFI=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=QYPrZgnr7qLVMUEgjsiZnpU5nESlBfVpUhBhj1P064/e2veg68dIHwFDdLFTdSAHy RhcbDTwkwtm+z9JPuvDOH/gcAyHzdXrkgTDGNVtKaSvOzK7LeATXF0p0Wqd+UVwsKi 2FmLew2nYi0C8l88sk0Lyk2h58z1iCc0rkw5NvO2ZI0vi2AOc2HisRVLcyvF/Qj5lO 42pnAgai+9k3OeshX11fWX0imIUVMU7h2AmfrrS1/DtO4SgEO2I43Ts4Fr/0YGn3Rm Ryzf+bX1LJGSf2+gFyCT6FmdmwrZOhHBUPtP9Ed+DE7FDTb0YqQ7RgKew0Hy5Fc3L7 eqP4jAAS4Q0Tw== Message-ID: <7eaf7fcf-9869-dd87-e60b-f0e4fc536220@denx.de> Date: Tue, 20 Jun 2023 11:13:41 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [SPAM] Re: [PATCH] cmd: usb: Prevent reset in usb tree/info command To: Xavier Drudis Ferran Cc: u-boot@lists.denx.de, Simon Glass , Lukasz Majewski , Sean Anderson References: <78623f65-0cb5-fad8-7a85-f7370dda022c@denx.de> Content-Language: en-US From: Marek Vasut In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean On 6/20/23 09:03, Xavier Drudis Ferran wrote: > El Tue, Jun 20, 2023 at 02:50:48AM +0200, Marek Vasut deia: >> On 6/13/23 08:52, Xavier Drudis Ferran wrote: >>> >>> U-Boot TPL 2023.07-rc2-00144-g497967f1ee (Jun 12 2023 - 11:15:47) >> >> Next is already at rc4 , what's this rc2 ? > > It's a test I did last week (June 12th). What do you think has changed > in next between rc2 and rc4 so that this shouldn't happen now ? The beginning of the email states: " Ok. New test. This uses yesterday morning's next branch. commit 5b589e139620214f " So, that is not the case ? Also, commit 497967f1ee does not exist in upstream U-Boot, was this some patched U-Boot tree ? > My last patch (sent yesterday) was tested with the next branch as of > yesterday morning. But you can test it yourself with any version you like. > Do I have to send the logs of each test I do to the list every time ? Ideally the claims you make about what you tested and logs from the test should match, that would be a good starting point. > I thought Simon Glass reply last week to the mail you quoted already > showed agreement that the issue exists, and the adding of the > UCLASS_BOOTDEV device is per design, so cmd/usb.c needs fixing to deal > with this change. I am not disputing that. What bothers me is the still missing consistent test case, esp. if this is a fix which should be added this late in release.