From patchwork Mon Jun 1 13:03:15 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Pali_Roh=C3=A1r?= X-Patchwork-Id: 11582007 X-Patchwork-Delegate: lorenzo.pieralisi@arm.com Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 21401739 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 F193D207D5 for ; Mon, 1 Jun 2020 13:03:41 +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=1B05qLtHTBY05X2LczsT+n9mrhVxBOU78pDepUGFBCczMUsxeUuTZ7okerZChAgDx Jhz4dqbLGwU+h1tJxvRuNUU+2Mc7yri1FvpD3NKgg+5y3ASHBIauvgB/smJ684qaQU sJT4RkfYde5owtHyRmMd14/STtw+v6L1n0OH/3DM= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726094AbgFANDl (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_Roh=C3=A1r?= To: Bjorn Helgaas , Thomas Petazzoni , Lorenzo Pieralisi , Andrew Murray , =?utf-8?q?Marek_Beh=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 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@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 Reported-by: Bjorn Helgaas --- 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);