From patchwork Fri Mar 27 17:33:42 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Przemyslaw Marczak X-Patchwork-Id: 455535 X-Patchwork-Delegate: sjg@chromium.org Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 2E7E9140146 for ; Sat, 28 Mar 2015 04:34:36 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id CC6C6ABFA6; Fri, 27 Mar 2015 18:34:31 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id oy9Lkop66wJY; Fri, 27 Mar 2015 18:34:31 +0100 (CET) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 7B4CAA748E; Fri, 27 Mar 2015 18:34:27 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 3EEA0A7428 for ; Fri, 27 Mar 2015 18:34:07 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id rkMGnufIkNIY for ; Fri, 27 Mar 2015 18:34:07 +0100 (CET) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from mailout2.w1.samsung.com (mailout2.w1.samsung.com [210.118.77.12]) by theia.denx.de (Postfix) with ESMTPS id 8E867A7438 for ; Fri, 27 Mar 2015 18:34:02 +0100 (CET) Received: from eucpsbgm1.samsung.com (unknown [203.254.199.244]) by mailout2.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0NLV000FXSZBWB00@mailout2.w1.samsung.com> for u-boot@lists.denx.de; Fri, 27 Mar 2015 17:37:59 +0000 (GMT) X-AuditID: cbfec7f4-b7f126d000001e9a-74-551593dfe235 Received: from eusync3.samsung.com ( [203.254.199.213]) by eucpsbgm1.samsung.com (EUCPMTA) with SMTP id 29.6B.07834.FD395155; Fri, 27 Mar 2015 17:31:11 +0000 (GMT) Received: from AMDC1186.digital.local ([106.116.147.185]) by eusync3.samsung.com (Oracle Communications Messaging Server 7u4-23.01(7.0.4.23.0) 64bit (built Aug 10 2011)) with ESMTPA id <0NLV009D4SSKREB0@eusync3.samsung.com>; Fri, 27 Mar 2015 17:33:59 +0000 (GMT) From: Przemyslaw Marczak To: u-boot@lists.denx.de Date: Fri, 27 Mar 2015 18:33:42 +0100 Message-id: <1427477624-23127-2-git-send-email-p.marczak@samsung.com> X-Mailer: git-send-email 1.9.1 In-reply-to: <1427477624-23127-1-git-send-email-p.marczak@samsung.com> References: <1425983444-18565-1-git-send-email-p.marczak@samsung.com> <1427477624-23127-1-git-send-email-p.marczak@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrDJMWRmVeSWpSXmKPExsVy+t/xq7r3J4uGGuy+yGSx4859Zos3Dzcz Wuy4fIPF4tuWbYwWb/d2slvcnPCD2WLvvc+MDuwesxsusnjMm3WCxePy9zfMHmfv7GD06Nuy itHjxPTvLAFsUVw2Kak5mWWpRfp2CVwZb9/fZilYzlGxZ1dkA+NDti5GTg4JAROJYw1zmSBs MYkL99YDxbk4hASWMkrMvdPNDuH0MUm8vvIbrIpNwEBiz6UzzCC2iICExK/+q4wgRcwCnxgl TjdvYQVJCAskSEw+Op8dxGYRUJXov38WrIFXwFWid99KVoh1chInj00GszkF3CSub/0EdpKQ QCOjxKJrzBMYeRcwMqxiFE0tTS4oTkrPNdQrTswtLs1L10vOz93ECAmxLzsYFx+zOsQowMGo xMNrcFA4VIg1say4MvcQowQHs5II7+RG0VAh3pTEyqrUovz4otKc1OJDjEwcnFINjIKtElEu /wqP717cLr5ZgS21yOrnuodHH4c2Sv3faiMgG7Y8PmoK+1Sb2HV3kid4BBTrP165MHD9RMuw c30TV7fb16Q/1FjzecKJnLuP1s6O3XGXeeHBiSxi0msD7n8v89Q2TjcKP1O2kLHDb56014qG BdHfO8UO7c+9FxlxzMCC5XzOk/v1R5VYijMSDbWYi4oTAcbWcNoPAgAA Cc: Mike Frysinger , Przemyslaw Marczak Subject: [U-Boot] [PATCH V2 1/3] dm: gpio: request list: return the count if requests max_count reached X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.15 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" The function gpio_request_list_by_name_nodev() returned -ENOSPC error, when the loop count was greater than requested count. This was wrong, because function should return the requested gpio count, when meets the call request without errors. Now, the loop ends on requested max_count. Signed-off-by: Przemyslaw Marczak Cc: Simon Glass Acked-by: Simon Glass --- drivers/gpio/gpio-uclass.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpio/gpio-uclass.c b/drivers/gpio/gpio-uclass.c index a69bbd2..4b63025 100644 --- a/drivers/gpio/gpio-uclass.c +++ b/drivers/gpio/gpio-uclass.c @@ -590,11 +590,7 @@ int gpio_request_list_by_name_nodev(const void *blob, int node, int count; int ret; - for (count = 0; ; count++) { - if (count >= max_count) { - ret = -ENOSPC; - goto err; - } + for (count = 0; count < max_count; count++) { ret = _gpio_request_by_name_nodev(blob, node, list_name, count, &desc[count], flags, true); if (ret == -ENOENT)