From patchwork Mon Sep 30 12:14:33 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 174746 Delivered-To: patch@linaro.org Received: by 2002:a92:7e96:0:0:0:0:0 with SMTP id q22csp7046677ill; Mon, 30 Sep 2019 05:15:49 -0700 (PDT) X-Google-Smtp-Source: APXvYqxKMHEoRovfw3ZCYFKzuXVHydWAdZISXP6C/oQDNjs/GS4uS9W+N53V8pEufG4Ufghs0M7n X-Received: by 2002:a17:906:3e96:: with SMTP id a22mr18838086ejj.268.1569845749350; Mon, 30 Sep 2019 05:15:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1569845749; cv=none; d=google.com; s=arc-20160816; b=LZBZ4nX69Q0kmsOoOFDCsvFHV500m82SluvF8EK9c81Dqfc8h22DGmqDqp3/4YXYI1 ANwjsaNkfoY4z+R/6lCyIzZ4ZArde6LHRlTAWtTjg8fhqd53+BZ3IkTMdI/GQpOeKIY0 0SvoyiBi9eED6O4FYeejVyw46CHqC9NhOSVBkcmhNbJ6Sh6VkpvYIHjr2mVK/2DEJX/v Zfn5SObcg+TVAEyGFeMfclzcxSZ526pFhTcV/hpR9hothhnRgFrbrFIghCenq+arh4HR KsvDubRDMOJiP1xqlYypP3lCIVtCKkBb7AsIMcpJbZkN+UvJKxHitpQ2tKpkxnVd6VPd zLNQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=ml/CTnqhVW2qBPPanZ0BKImXsly7132M8cavQjsI7e0=; b=QTC1FOUHQndFhGSqoVIHwQa/1sdA5FWObB4h4zrPZGnX/6GWvI/vMyUu6ck2mlLBrY QT8E7wldV8pU5PGgAkq7MoV0k+k+4Vv4tYRv+3dD0zmXmtXQ+q+S6A08p7nt7cUEk8W6 0gg4Nd8D3RdtC8FuOLneaxP7pv3hs/u7wInIjROfkhLqFYhuUJcg20rm8OvLCUwmhFkz 8fvren/Dcez9iRtA9e+Udb3gT14pKxuCO55c2FEi15aitx4Uq1NBRc3JGTJdrGSmYqO/ LrFU5sqVpSuz343pNhjOAhN49pqJpbJOqulenngXAbww40NpIThR628VkAjGKzZZNwNx nG3A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c20si6231777ejs.197.2019.09.30.05.15.49; Mon, 30 Sep 2019 05:15:49 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730490AbfI3MPs (ORCPT + 27 others); Mon, 30 Sep 2019 08:15:48 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:52461 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726952AbfI3MPr (ORCPT ); Mon, 30 Sep 2019 08:15:47 -0400 Received: from threadripper.lan ([149.172.19.189]) by mrelayeu.kundenserver.de (mreue108 [212.227.15.145]) with ESMTPA (Nemesis) id 1MqapC-1hjvMD2Xwm-00meW0; Mon, 30 Sep 2019 14:15:25 +0200 From: Arnd Bergmann To: Antoine Tenart , Herbert Xu , "David S. Miller" Cc: Arnd Bergmann , Pascal van Leeuwen , Pascal van Leeuwen , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/3] crypto: inside-secure - Fix a maybe-uninitialized warning Date: Mon, 30 Sep 2019 14:14:33 +0200 Message-Id: <20190930121520.1388317-1-arnd@arndb.de> X-Mailer: git-send-email 2.20.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:LInyj3Ud0A8wB1lffm2V4xf1mZe/8J3moJ6KlLFoXVopMJiU/KM KxE4gKuqXvHz5qvY0skfX00Qf9wBYigW3dERcUECUQk5t+w48wwHyk5tQnjQRvHg1s9vw1H cZ/9oBe3xJSdpS21GHrrqvAXp6zEoKW/PvheWvOURr3TQe8FHkaIuu0vfdLDy+QlRH18qie mrXaPorGX3+Eyn5xcLY5w== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V03:K0:U0vHL9TV2yo=:DiIHi5HDqbjDn+66lvh3os nzMF3Fp3UmCEhUte/68ClyljDSx7vzyaqqEWEK3TFQFqaUO2OKOiHG0v6QQruDYfewn3wNCFb u0efen0ljbb7ev/Hvu+rImGYZJ7aqaKAzCjzkZy68VBBUBnNE0DdNP9vrnIGYW5wDTDCMjqDg 0ZqHMVq/Fy6f2gXJdK1aQNMCWlldAia81jFNbJhs4IaHSpci+Gj4N5ERBVQqNYAUJnssRxxei 3+NaFM59fS8mxFIts/r+X3tzUFBalDZdjaaXLcHp5qL+EbwGiouWMl7AxfbJxN4FsJwBLpZVt j5n5Mpvh4AQijN+RYpzFl+hdyCfnL8vLJZLMGAgK9NZidxC1ItinGxHu/0SpprSCPSOxvc/i2 7o1VYme52Tde7KRHrfjkztgJO5MFe3LZbwe/iud14AEaAa59E4zgopDXjsAb5bz4H3d+LFwnL ICLYRkOIgrUuL1mTHukqdBtvE/9Lx6vABqUHymuQwwrshX9eDV/7FhAKiDh5NjOh0Mj+nTsVN JR2pmX9WvGVGppUySihKW8s1qDKKKedwhkHJyjaNDTiXxWd77ljwYX49JM8YwQf8Mp6qSWB/4 DQjNrWV11MDtCBBywbalwJl1VQ7UoVTItdfE1+PaVrsOpEB6x5cSyHMQgJu5T+ZP5ubybNd9J A0rOoVnBkAThUKeVo1NAgWtYKh6zi3bfHTyag29X2COSv1jLQzo6+CtD0791oNuOKo0ZLMtcb J1chSZt5iADXu6VIZoNlrC7b8UpGYQX8w1opW9fD8t5bhta/o9Kf3TjexUzssMFRtWKH1Qf0e tMk1goSf9Lb6jP/p+i6aXWyDLSf2kN5TD0VGDF8yUnptoPoMyijB7YRj2d+1zBN67mm10fYhJ EQZdIIA3eBVUy4Js3uXQ== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A previous fixup avoided an unused variable warning but replaced it with a slightly scarier warning: drivers/crypto/inside-secure/safexcel.c:1100:6: error: variable 'irq' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized] This is harmless as it is impossible to get into this case, but the compiler has no way of knowing that. Add an explicit error handling case to make it obvious to both compilers and humans reading the source. Fixes: 212ef6f29e5b ("crypto: inside-secure - Fix unused variable warning when CONFIG_PCI=n") Signed-off-by: Arnd Bergmann --- drivers/crypto/inside-secure/safexcel.c | 2 ++ 1 file changed, 2 insertions(+) -- 2.20.0 diff --git a/drivers/crypto/inside-secure/safexcel.c b/drivers/crypto/inside-secure/safexcel.c index 4ab1bde8dd9b..311bf60df39f 100644 --- a/drivers/crypto/inside-secure/safexcel.c +++ b/drivers/crypto/inside-secure/safexcel.c @@ -1120,6 +1120,8 @@ static int safexcel_request_ring_irq(void *pdev, int irqid, irq_name, irq); return irq; } + } else { + return -ENXIO; } ret = devm_request_threaded_irq(dev, irq, handler,