From patchwork Thu Sep 16 08:53:36 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Masami Hiramatsu X-Patchwork-Id: 512919 Delivered-To: patch@linaro.org Received: by 2002:a02:c816:0:0:0:0:0 with SMTP id p22csp1010041jao; Thu, 16 Sep 2021 01:55:33 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyrGg4ihQfT4SltxQzBXnA7MjZyY/wHIu0y4p+hEh9hMqXqberQkdbCJ902DygiAwq8Yzjo X-Received: by 2002:a17:906:ece4:: with SMTP id qt4mr5170908ejb.250.1631782533640; Thu, 16 Sep 2021 01:55:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631782533; cv=none; d=google.com; s=arc-20160816; b=BHIbOSvTu39RPfqRh3feCNY6h7l2APldibbwGWqMonlLm2GmqFl8fwkVLLKDeMVUP4 Tp0UGKzxoLDT7WptZjfNFhTRDpLrIi5PS9mQlxLaNy8zuPK+pag0S2rHFDXng4xjsCwf uM5VZsZB2fKde5Mc6ostU4RUKVTTyqaZZj1g6UajAshskJG3B60Q7nbOOdsozlekVFHK qt792uhXbY3Mm7/gstrrMxQfr+9pV+KxPwzIfYAx9kle8NVtZUHCSlIyeGXjRmf8f0ro 1gs1sIejWEo+OYNk8jBhAVtsvAvB/vOZ2/AFEFwGjTIKhkXEHi5rfZ54fi+PV7oS/KrZ p6yg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:user-agent:references:in-reply-to:message-id:date :subject:cc:to:from:dkim-signature; bh=z/P28taXTWuoZNLOu5MoRtO3G6G9hQYJRdgwPsybijU=; b=VIYu7rGjfion6lsSR2WSe2CX35o36flPcJ2Gh2aH4/kr1BnG7A4jwSCisWrnIZjtmN ss1lU6P16fdx2l4o1KfVqvHmYdjje/dRkrFobBEMWOP2FR9wlmHefDwuPo8UvJig4cUP tFcleJqlj39sFokWL2HXrzCBfegIJMPBQmtg8ZmytnhcUvy/f6D4KMqoBhM46ifkw5Er o0/Dmd2Tnk3V66q9WIodZOFP2hU1tFvyNSJDuJB00/7FxtbuYFt2Wc7PRQRNusE7TTeE QRAVkTiDyQuxCXS4MmeW+oZViKyc/cmNE5Ktz41xdViRWHmSJp69xqWJjEkPwkqOeCo5 rbuQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=yTo4MvaO; spf=pass (google.com: domain of u-boot-bounces@lists.denx.de designates 2a01:238:438b:c500:173d:9f52:ddab:ee01 as permitted sender) smtp.mailfrom=u-boot-bounces@lists.denx.de; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from phobos.denx.de (phobos.denx.de. [2a01:238:438b:c500:173d:9f52:ddab:ee01]) by mx.google.com with ESMTPS id hx14si2829418ejc.326.2021.09.16.01.55.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 16 Sep 2021 01:55:33 -0700 (PDT) Received-SPF: pass (google.com: domain of u-boot-bounces@lists.denx.de designates 2a01:238:438b:c500:173d:9f52:ddab:ee01 as permitted sender) client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=yTo4MvaO; spf=pass (google.com: domain of u-boot-bounces@lists.denx.de designates 2a01:238:438b:c500:173d:9f52:ddab:ee01 as permitted sender) smtp.mailfrom=u-boot-bounces@lists.denx.de; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 7FCCA8319C; Thu, 16 Sep 2021 10:54:46 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="yTo4MvaO"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 07C0783124; Thu, 16 Sep 2021 10:54:06 +0200 (CEST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 Received: from mail-pf1-x42a.google.com (mail-pf1-x42a.google.com [IPv6:2607:f8b0:4864:20::42a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id F20C8831D5 for ; Thu, 16 Sep 2021 10:53:40 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=masami.hiramatsu@linaro.org Received: by mail-pf1-x42a.google.com with SMTP id y8so5225318pfa.7 for ; Thu, 16 Sep 2021 01:53:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :user-agent:mime-version:content-transfer-encoding; bh=z/P28taXTWuoZNLOu5MoRtO3G6G9hQYJRdgwPsybijU=; b=yTo4MvaOlCtqn/ZpfIqUzbUYebyuX7M/wMD3xDG8Ur/dAZiY0ksj3Yp1HVsN7WdiGa V8zuH9qZKKNSokIipM1Qt+7gdP4nDTOzlINRKG7JdaDdCciFYgw1HBJA6QxKwBpUXUB9 cFurZMqxODysuLQN6rCaDwuUHP68s3S6qwnOHlZgOmH3jbCEwXaSlWy6iRT/Y0xO/vXH p+HIjM3BVVBHXDmHgdTgjj6QnJRC9uYXSlpH+Vtbq+8AKqy+5mRH79jCk5Yr+EiOWENS gkRv+WSnfDTaxz9Wj2yd1QJDLpZU60oXwQKe0iZwGgrXW8JEanCmBNBZE4EEk8qIZb+v uVMQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:user-agent:mime-version:content-transfer-encoding; bh=z/P28taXTWuoZNLOu5MoRtO3G6G9hQYJRdgwPsybijU=; b=Vg6M/ZN+L53OdB3MIckDwdubp6lTOonduG0oGKhLqcbbRhZrxLoN2xWwvcloc1SE7s SnkR0nvAeQMKY7R2FNWKT5ujHzjc5Wt4RxIO9lwTUXJ5j1zexVFsG5TVywz/7sZ5DitX +utO6RO2lKpsJfFjhMjuS+fw504e+TFbgjcISnc+nTwMC9SyK9RgNQ1XLpxQ5Qbe6N4o Z6Tmfk7kWRtwzNckXtJ4oIkIMh3McGpxsumR4i6FlBp+1vg/OK5VF5ev889tPDGz8LoR T+39wRyrDbbOnJ5ft+JRyD3LMUCFUgULFVEsk20FWUzfG/nCnuHdY1xyWeVhqBpllQsk KPUg== X-Gm-Message-State: AOAM5317imQU2TrhsT+qBHmLoXd4rAs4Je5EFDZSon80cZWmtzZYIGk+ fxyH5im8I70Ycck0Yr38z6XwLQ== X-Received: by 2002:a62:5406:0:b0:441:c46a:ccb9 with SMTP id i6-20020a625406000000b00441c46accb9mr1672137pfb.52.1631782419199; Thu, 16 Sep 2021 01:53:39 -0700 (PDT) Received: from localhost.localdomain (122x208x150x49.ap122.ftth.ucom.ne.jp. [122.208.150.49]) by smtp.gmail.com with ESMTPSA id b13sm7122786pjk.35.2021.09.16.01.53.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 16 Sep 2021 01:53:38 -0700 (PDT) From: Masami Hiramatsu To: Heinrich Schuchardt Cc: Kazuhiko Sakamoto , Masami Hiramatsu , Jassi Brar , Ilias Apalodimas , u-boot@lists.denx.de Subject: [PATCH 2/3] efi_selftest: Do not check EFI_SIMPLE_NETWORK_RECEIVE_INTERRUPT Date: Thu, 16 Sep 2021 17:53:36 +0900 Message-Id: <163178241612.65790.3191991741031171590.stgit@localhost> X-Mailer: git-send-email 2.25.1 In-Reply-To: <163178239865.65790.17211919979018423637.stgit@localhost> References: <163178239865.65790.17211919979018423637.stgit@localhost> User-Agent: StGit/0.19 MIME-Version: 1.0 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 Do not check EFI_SIMPLE_NETWORK_RECEIVE_INTERRUPT in packet receiving loop. This depends on the implementation and not related to whether the packet can be received or not. Whether the received packets are available or not is ensured by wait_for_packet, and that is already done in the loop. Signed-off-by: Masami Hiramatsu --- lib/efi_selftest/efi_selftest_snp.c | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/lib/efi_selftest/efi_selftest_snp.c b/lib/efi_selftest/efi_selftest_snp.c index cb0db7eea2..c5366c872c 100644 --- a/lib/efi_selftest/efi_selftest_snp.c +++ b/lib/efi_selftest/efi_selftest_snp.c @@ -340,8 +340,6 @@ static int execute(void) events[0] = timer; events[1] = net->wait_for_packet; for (;;) { - u32 int_status; - /* * Wait for packet to be received or timer event. */ @@ -367,15 +365,6 @@ static int execute(void) * Receive packet */ buffer_size = sizeof(buffer); - ret = net->get_status(net, &int_status, NULL); - if (ret != EFI_SUCCESS) { - efi_st_error("Failed to get status"); - return EFI_ST_FAILURE; - } - if (!(int_status & EFI_SIMPLE_NETWORK_RECEIVE_INTERRUPT)) { - efi_st_error("RX interrupt not set"); - return EFI_ST_FAILURE; - } ret = net->receive(net, NULL, &buffer_size, &buffer, &srcaddr, &destaddr, NULL); if (ret != EFI_SUCCESS) {