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 X-Spam-Level: X-Spam-Status: No, score=-5.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F3CFDC433EF for ; Mon, 13 Sep 2021 10:42:54 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id AFA8060FF2 for ; Mon, 13 Sep 2021 10:42:53 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org AFA8060FF2 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id DEC9683A3F; Mon, 13 Sep 2021 12:42:50 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (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=1631529771; bh=6iLE27Cpc2YDlyRon0QcbRBBOtV4Qyh7PmRswVpCqY8=; h=To:cc:From:Subject:In-reply-to:References:Date:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=ESpSy2z1InZshcr2NUtMilAKUx0AN041/1Iq/xlClfOMbfjemkD0VDPQHFOnsfHFG zCjsaypMidOHOC8G5XzsYe4JTSB/yvTd5FauaZhCW6JQXVEsI88xJC/ZpCxN2BX5vi NRPsTUIuJtg8J4Beroyh4g6zkegil3cip133ef7DnGu8BP/qgBPUTxORM50td9bH98 MCzdVtoIETq8TsY1Hpxwk7gIgC6yN2iwiEywxPVzh9HCEJzBrfgcXn/ZBA/u8ZsPDp QvEy8Z0e/qZWiTZcokL3kc426q6OxJQDNDKejwgTAABVhKSltmrInIL2mzC2RfAfFa rU9VlzQezVZ2A== Received: from janitor.denx.de (unknown [62.91.23.180]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: noc@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id 5D60883A3B for ; Mon, 13 Sep 2021 12:42:49 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1631529769; bh=6iLE27Cpc2YDlyRon0QcbRBBOtV4Qyh7PmRswVpCqY8=; h=To:cc:From:Subject:In-reply-to:References:Date:From; b=xoE3p4RCRwCxF8pDLMGe7WDtdTcqE0XswhgcgaUMRYx8KKzly66B+OUnlMyIJ0DzM 21XodtjXvM7B0OHKED6Jpgig/AqH1bbVe4SZOQ1ob5h42VPEsALMWuhXpnMvBJW52v 8OMwNrG7xmdV0M0mY7y0QNp4bh7dIqvzDhSTW1kFT0FJJcFIuOI/2FtHbHYiRZPd5J 1DzZX7Q1k8VgrHtoAcv7QZvkvyzmPl3wKD6L/Teo7gpLNxUADqkx7DvwH9/EfGDqRN AlZ1cMrG9/V86wT6ifkgoyXHLmYlZCsTiohkNo6G7JioY6MabciL5BCXLd7NLU8ij4 TK24idpGvGDiQ== Received: by janitor.denx.de (Postfix, from userid 108) id 00A90A0126; Mon, 13 Sep 2021 12:42:48 +0200 (CEST) Received: from gemini.denx.de (gemini.denx.de [10.4.0.2]) by janitor.denx.de (Postfix) with ESMTPS id C4681A0036; Mon, 13 Sep 2021 12:42:42 +0200 (CEST) Received: from gemini.denx.de (localhost [IPv6:::1]) by gemini.denx.de (Postfix) with ESMTP id 831721E0AE8; Mon, 13 Sep 2021 12:42:42 +0200 (CEST) To: =?UTF-8?q?Pali=20Roh=C3=A1r?= cc: Simon Glass , Heinrich Schuchardt , u-boot@lists.denx.de From: Wolfgang Denk Subject: Re: [PATCH 1/2] xyz-modem: Wait infinitely for initial y-modem packet MIME-Version: 1.0 Content-type: text/plain; charset=UTF-8 Content-transfer-encoding: 8bit In-reply-to: <20210910204653.3066-1-pali@kernel.org> References: <20210910204653.3066-1-pali@kernel.org> Comments: In-reply-to =?UTF-8?q?Pali=20Roh=C3=A1r?= message dated "Fri, 10 Sep 2021 22:46:52 +0200." Date: Mon, 13 Sep 2021 12:42:42 +0200 Message-ID: <1325940.1631529762@gemini.denx.de> X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 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.2 at phobos.denx.de X-Virus-Status: Clean Dear Pali Rohár, In message <20210910204653.3066-1-pali@kernel.org> you wrote: > Now when command loady can be aborted / cancelled by CTRL+C, change wait > timeout for initial packet to infinite. This would allow user to not be > hurry when locating file which want to send. Commands loadb and loads > already waits infinitely too. I'm not sure if this is a good idea. If you use loady in any kind of scripts, this would now hard hang the system, while until now it was possible to recover from the error. This is a change to the user interface that is not really necessary, so I recommend NOT to change the behaviour here, especially as it does not hurt. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de "I've finally learned what `upward compatible' means. It means we get to keep all our old mistakes." - Dennie van Tassel