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=-10.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable 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 AF861C433DF for ; Mon, 1 Jun 2020 13:03:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 73A9E2077D for ; Mon, 1 Jun 2020 13:03:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1591016622; bh=3xN+KUHay1UXrM2e/q9Ewz774/uNEpSlG4BqrojgmRI=; h=From:To:Cc:Subject:Date:List-ID:From; b=0vn5S590CLGsE7hTEZI+jotzeDWwD7SUMwz4eNCzupQMwJQQgJZQwVZcD/e3/fLZz VbqYw+Fi5fEJURvpliCEhRaL3a4B0+pQKUUJfD04jmuHBbtX4A5TbjpX1jD1EM6l8h ugQwV7gUN/8onwEv+kJF0NiUQiPlkRG0PlUNH3E0= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726149AbgFANDl (ORCPT ); Mon, 1 Jun 2020 09:03:41 -0400 Received: from mail.kernel.org ([198.145.29.99]:35954 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725847AbgFANDl (ORCPT ); Mon, 1 Jun 2020 09:03:41 -0400 Received: from pali.im (pali.im [31.31.79.79]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 87BFE2068D; Mon, 1 Jun 2020 13:03:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1591016620; bh=3xN+KUHay1UXrM2e/q9Ewz774/uNEpSlG4BqrojgmRI=; h=From:To:Cc:Subject:Date:From; b=IqMBJBhQg9sgMy1Kj9AZ4jYM8Ip+nSuoT3fd6gBMgEx80FyJItuwxsBcoenoDpY15 VOIxthju/gClsXmTLYHVrge9bc6iuzuFZmvQFnjq28zVFeYWP6rMrt1sXlnyOhrIDs WBGXWie/++Uuu/wovmm60x9u68QpuN24q8oiSnw8= Received: by pali.im (Postfix) id BA9BDCB0; Mon, 1 Jun 2020 15:03:38 +0200 (CEST) From: =?UTF-8?q?Pali=20Roh=C3=A1r?= To: Bjorn Helgaas , Thomas Petazzoni , Lorenzo Pieralisi , Andrew Murray , =?UTF-8?q?Marek=20Beh=C3=BAn?= , Remi Pommarel , Tomasz Maciej Nowak , Xogium Cc: linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH] PCI: aardvark: Indicate error in 'val' when config read fails Date: Mon, 1 Jun 2020 15:03:15 +0200 Message-Id: <20200601130315.18895-1-pali@kernel.org> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Most callers of config read do not check for return value. But most of the ones that do, checks for error indication in 'val' variable. This patch updates error handling in advk_pcie_rd_conf() function. If PIO transfer fails then 'val' variable is set to 0xffffffff which indicates failture. Signed-off-by: Pali Rohár --- drivers/pci/controller/pci-aardvark.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/pci/controller/pci-aardvark.c b/drivers/pci/controller/pci-aardvark.c index 53a4cfd7d377..783a7f1f2c44 100644 --- a/drivers/pci/controller/pci-aardvark.c +++ b/drivers/pci/controller/pci-aardvark.c @@ -691,8 +691,10 @@ static int advk_pcie_rd_conf(struct pci_bus *bus, u32 devfn, advk_writel(pcie, 1, PIO_START); ret = advk_pcie_wait_pio(pcie); - if (ret < 0) + if (ret < 0) { + *val = 0xffffffff; return PCIBIOS_SET_FAILED; + } advk_pcie_check_pio_status(pcie); -- 2.20.1