From patchwork Fri Jun 17 05:19:21 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Srinivas Neeli X-Patchwork-Id: 12885106 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E2190C43334 for ; Fri, 17 Jun 2022 05:20:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:CC :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=wLUfNTn8sqDFaT3nbCCscu3YPfcDC0IR1vQY88s0gjk=; b=N/YGf2T2yWtbDV +yer9uoFQ9lLQ1xRdRpz11Htm6U5J2y4kKXuBD9YznfyxrhvKh3snZ7kF+wvrhGtht1umAVyJzE48 MgrCq3BJbRBfOO/I3UPS32hSLj75rtxKSN655ikxgmjA/8eTR+xaWOQ/Ub+NYQMh08ilGxwX3cMQm Xd4eoxHnlFbAjE1rC5X3XKCfcWx09Ga6UfDaZZxa6pmVJ9vicJoUlkBvG9yAevhOAr8SjWurH796a tgUDgBuOqaf7a0S+LB1DxTPKWLKvepVTkfCEx9fcVMsj+9sfvmVfWXFQqqx6FOaMCMR/B4+9XjhnD SMvvPTgjmEoZc2ywzuvA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o24Ob-005Yrh-EC; Fri, 17 Jun 2022 05:19:49 +0000 Received: from mail-dm6nam10on2060a.outbound.protection.outlook.com ([2a01:111:f400:7e88::60a] helo=NAM10-DM6-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1o24OU-005YpB-9x for linux-arm-kernel@lists.infradead.org; Fri, 17 Jun 2022 05:19:47 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bbBLmWz5O5WrwXvWq603KvuQULtkmBRZm7HmKwc9Dn+8EMBC8Fyt5Y8vWUTYWu1rplWjAMXwCOqgEz9MtvlkFLn/Pu5oxnbiDqUUNpd6R6wL43L6zM5cXa6pw/UHlw4zEHu5v6nmW0mSkKxBBzpgBTUsq8g1qozNdfY6IUmU/+qMUfs2Hl6Y75dXjYPT3BHhlfAnTeYX6ukeBeOw7Nk+F8lpENfSkLDFTbrRa9hU/nDOCSQf/3aI2QAx6nsg2qYm8TEg+uO3xyycaVXyZ/QdHoy08PhfremEwVv3ma/xSV4kulWrpDgKoGGW04hsyki663mcQhaS2jngvQazHxDO8Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=t0l8R/le4LsHqFokvaHVS/vNOCwp2qTDoLGHw4wjkBw=; b=ZtGFIqjB0MWosIM0SZ6hICnTqOSDDbvfxf5rX+LQTBJhGg4lBCbWGOK0VPq01KwogT9OGmuiizocu7MXY8JljQ9gzS2H71iqiwtwzX5A00ggiE1FF12KaHdQKUU9E/jXYXtDUtKkphOjdvB1WHX20iz7PATaeDIt9O5r11jOXRxxjbyuZknsUfjVzTOOo+OGHf6OvNo8DYf3Ca/8sv2QUUx39q3Du59qDLnmp9Gs7pdXar6ebjE8JCBeSALWHtZvdkvBdg1jqERZXSgvtams4+ebxDvKb6nLUwcqhG91VTHMCkVcxbhL/GMeMUYXUbctGwwN2Ygmf2svffZpxbCB8A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 149.199.62.198) smtp.rcpttodomain=linaro.org smtp.mailfrom=xilinx.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=xilinx.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xilinx.onmicrosoft.com; s=selector2-xilinx-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=t0l8R/le4LsHqFokvaHVS/vNOCwp2qTDoLGHw4wjkBw=; b=icaLcPmU4/OCQ6AAXnwfzsxvLjKAsjVFkF5Wb4BEIHkn9YKNYEWYWzNykR9B0e0A+ODSTnlLkdcFbVUbYX+xwHixWPJE6BmnKtu2riKJlyPh7EHOnfC1VByaUf9EBjD0OTBo6NkuxwmCV5/A8XNpzftfIgydXjPj5X0W7gf8oHA= Received: from DM5PR07CA0144.namprd07.prod.outlook.com (2603:10b6:3:13e::34) by PH0PR02MB8630.namprd02.prod.outlook.com (2603:10b6:510:109::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5353.15; Fri, 17 Jun 2022 05:19:31 +0000 Received: from DM3NAM02FT003.eop-nam02.prod.protection.outlook.com (2603:10b6:3:13e:cafe::7) by DM5PR07CA0144.outlook.office365.com (2603:10b6:3:13e::34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5353.15 via Frontend Transport; Fri, 17 Jun 2022 05:19:31 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 149.199.62.198) smtp.mailfrom=xilinx.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=xilinx.com; Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.62.198 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.62.198; helo=xsj-pvapexch02.xlnx.xilinx.com; pr=C Received: from xsj-pvapexch02.xlnx.xilinx.com (149.199.62.198) by DM3NAM02FT003.mail.protection.outlook.com (10.13.4.168) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.5353.14 via Frontend Transport; Fri, 17 Jun 2022 05:19:31 +0000 Received: from xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.14; Thu, 16 Jun 2022 22:19:29 -0700 Received: from smtp.xilinx.com (172.19.127.95) by xsj-pvapexch02.xlnx.xilinx.com (172.19.86.41) with Microsoft SMTP Server id 15.1.2176.14 via Frontend Transport; Thu, 16 Jun 2022 22:19:29 -0700 Received: from [10.140.6.39] (port=47594 helo=xhdsgoud40.xilinx.com) by smtp.xilinx.com with esmtp (Exim 4.90) (envelope-from ) id 1o24OG-000DMd-B4; Thu, 16 Jun 2022 22:19:28 -0700 From: Srinivas Neeli To: , , , , , , , CC: , , , , Srinivas Neeli Subject: [PATCH] gpio: gpio-xilinx: Check return value of of_property_read_u32 Date: Fri, 17 Jun 2022 10:49:21 +0530 Message-ID: <20220617051921.3801832-1-srinivas.neeli@xilinx.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 43c92f3e-4b16-4f4b-f23f-08da5020f55e X-MS-TrafficTypeDiagnostic: PH0PR02MB8630:EE_ X-Microsoft-Antispam-PRVS: X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: JQ/0j7CU6OR/Mwzf4tgL3FkflU894zxXanVV5cHlsIY3yQTSbBPW2NcDJ6eEIpZiENA/U8yO+Och87XdV4gMUY7mVUmdOMnx6zsZhF73RixoWZxB20jGknhPRGRn9QO1N6TQX8/xb9CdF7QWoAVSQVDTC7gTOiMl1hhaysyvEqy1S8aelny81XAyWNuCXSiEfuGAKQRqR9T75Yz89jJJAnV5KT26jXhIlSpQ0v8KaeML5QKZKK4AmCRRc01jSff1zD4nOKmWfzhZjIpDgLVt6Xe0kN5NgXD3QlMWO7oUtwMPC/GLLqESIz3IKZ2pZeEXUJW2g4UoKoi12/THaNTUEgE684gJsicPKkF/ogFfgAufgd3JjGPq1GaJ3zS7Zw2G6xQ3U6jsXMxkD24PnVWvuqsr6dlbBcqm81H2yosHJ0hhYXn2sIGauau66GqxJ9RhP7p+qKjmeWIj9bJhJjMWtmdgzb9di+kWHKqKM9H/wILtGKJkWCrPaGdndozd24sJ2nsgjdPluxOzhlPIQOp0jxoebhGag7+yTBK3CAkGIrtjIWzpvDjLRsHMP7/ytHNlmQR5SkVjxHhRyIrlCMKXGC3iBm1k/xXeLtFPzQIN8DRa+ioTKRBYspHShQDuNqzMJVwSGGYqW146YmNpT6vjjsbwmx4WuhjB6rpvB3YBvZrGTIhq2fR6Gm7/Rhp+BP7yVlplrmplsww9raaItpG7DKxBJ/luyOaUtvmpzp8ngYc= X-Forefront-Antispam-Report: CIP:149.199.62.198;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:xsj-pvapexch02.xlnx.xilinx.com;PTR:unknown-62-198.xilinx.com;CAT:NONE;SFS:(13230016)(4636009)(40470700004)(36840700001)(46966006)(70206006)(4326008)(40460700003)(44832011)(2906002)(7696005)(82310400005)(498600001)(6666004)(426003)(6636002)(54906003)(26005)(7636003)(2616005)(1076003)(36756003)(83380400001)(70586007)(316002)(110136005)(107886003)(356005)(186003)(36860700001)(8676002)(8936002)(5660300002)(9786002)(47076005)(336012)(102446001)(2101003);DIR:OUT;SFP:1101; X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jun 2022 05:19:31.1381 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 43c92f3e-4b16-4f4b-f23f-08da5020f55e X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.62.198];Helo=[xsj-pvapexch02.xlnx.xilinx.com] X-MS-Exchange-CrossTenant-AuthSource: DM3NAM02FT003.eop-nam02.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR02MB8630 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220616_221942_387531_B77C0DFD X-CRM114-Status: UNSURE ( 8.97 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org In five different instances the return value of "of_property_read_u32" API was neither captured nor checked. Fixed it by capturing the return value and then checking for any error. Signed-off-by: Srinivas Neeli Addresses-Coverity: "check_return" --- drivers/gpio/gpio-xilinx.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/gpio/gpio-xilinx.c b/drivers/gpio/gpio-xilinx.c index b6d3a57e27ed..268c7b0e481d 100644 --- a/drivers/gpio/gpio-xilinx.c +++ b/drivers/gpio/gpio-xilinx.c @@ -570,7 +570,8 @@ static int xgpio_probe(struct platform_device *pdev) platform_set_drvdata(pdev, chip); /* First, check if the device is dual-channel */ - of_property_read_u32(np, "xlnx,is-dual", &is_dual); + if (of_property_read_u32(np, "xlnx,is-dual", &is_dual)) + is_dual = 0; /* Setup defaults */ memset32(width, 0, ARRAY_SIZE(width)); @@ -578,14 +579,18 @@ static int xgpio_probe(struct platform_device *pdev) memset32(dir, 0xFFFFFFFF, ARRAY_SIZE(dir)); /* Update GPIO state shadow register with default value */ - of_property_read_u32(np, "xlnx,dout-default", &state[0]); - of_property_read_u32(np, "xlnx,dout-default-2", &state[1]); + if (of_property_read_u32(np, "xlnx,dout-default", &state[0])) + state[0] = 0x0; + if (of_property_read_u32(np, "xlnx,dout-default-2", &state[1])) + state[1] = 0x0; bitmap_from_arr32(chip->state, state, 64); /* Update GPIO direction shadow register with default value */ - of_property_read_u32(np, "xlnx,tri-default", &dir[0]); - of_property_read_u32(np, "xlnx,tri-default-2", &dir[1]); + if (of_property_read_u32(np, "xlnx,tri-default", &dir[0])) + dir[0] = 0xFFFFFFFF; + if (of_property_read_u32(np, "xlnx,tri-default-2", &dir[1])) + dir[1] = 0xFFFFFFFF; bitmap_from_arr32(chip->dir, dir, 64);