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)