From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755850AbeEaQzS (ORCPT ); Thu, 31 May 2018 12:55:18 -0400 Received: from mail-lf0-f68.google.com ([209.85.215.68]:42653 "EHLO mail-lf0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755829AbeEaQzN (ORCPT ); Thu, 31 May 2018 12:55:13 -0400 X-Google-Smtp-Source: ADUXVKKBFfN96EClLwNKiSTdY9pPFrcVB8oXsHk1RQc6kxEx0j4B/zqIkU/aNzDxyCuXuO/JLhNs/w== Subject: Re: [PATCH v2 01/21] usb: phy: use match_string() helper To: Yisheng Xie , linux-kernel@vger.kernel.org Cc: andy.shevchenko@gmail.com, linux-usb@vger.kernel.org, Felipe Balbi , Greg Kroah-Hartman References: <1527765086-19873-1-git-send-email-xieyisheng1@huawei.com> <1527765086-19873-2-git-send-email-xieyisheng1@huawei.com> From: Sergei Shtylyov Organization: Cogent Embedded Message-ID: Date: Thu, 31 May 2018 19:55:10 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <1527765086-19873-2-git-send-email-xieyisheng1@huawei.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-MW Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello! On 05/31/2018 02:11 PM, Yisheng Xie wrote: > match_string() returns the index of an array for a matching string, > which can be used instead of open coded variant. > > Cc: linux-usb@vger.kernel.org > Cc: Felipe Balbi > Cc: Greg Kroah-Hartman > Signed-off-by: Yisheng Xie > --- > v2: > - donot rename err to ret - per Andy Hm... > > drivers/usb/phy/of.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/usb/phy/of.c b/drivers/usb/phy/of.c > index 1ab134f..9d74081 100644 > --- a/drivers/usb/phy/of.c > +++ b/drivers/usb/phy/of.c > @@ -28,16 +28,16 @@ > enum usb_phy_interface of_usb_get_phy_mode(struct device_node *np) > { > const char *phy_type; > - int err, i; > + int err; > > err = of_property_read_string(np, "phy_type", &phy_type); > if (err < 0) > return USBPHY_INTERFACE_MODE_UNKNOWN; > > - for (i = 0; i < ARRAY_SIZE(usbphy_modes); i++) > - if (!strcmp(phy_type, usbphy_modes[i])) > - return i; > + err = match_string(usbphy_modes, ARRAY_SIZE(usbphy_modes), phy_type); > + if (err < 0) This is one of the few cases when 'err' is not the best name for such a variable. I'd prefer to see something like 'match' or even 'rc' or 'ret'... :-) > + return USBPHY_INTERFACE_MODE_UNKNOWN; > > - return USBPHY_INTERFACE_MODE_UNKNOWN; > + return err; > } > EXPORT_SYMBOL_GPL(of_usb_get_phy_mode); > MBR, Sergei From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [v2,01/21] usb: phy: use match_string() helper From: Sergei Shtylyov Message-Id: Date: Thu, 31 May 2018 19:55:10 +0300 To: Yisheng Xie , linux-kernel@vger.kernel.org Cc: andy.shevchenko@gmail.com, linux-usb@vger.kernel.org, Felipe Balbi , Greg Kroah-Hartman List-ID: SGVsbG8hCgpPbiAwNS8zMS8yMDE4IDAyOjExIFBNLCBZaXNoZW5nIFhpZSB3cm90ZToKCj4gbWF0 Y2hfc3RyaW5nKCkgcmV0dXJucyB0aGUgaW5kZXggb2YgYW4gYXJyYXkgZm9yIGEgbWF0Y2hpbmcg c3RyaW5nLAo+IHdoaWNoIGNhbiBiZSB1c2VkIGluc3RlYWQgb2Ygb3BlbiBjb2RlZCB2YXJpYW50 Lgo+IAo+IENjOiBsaW51eC11c2JAdmdlci5rZXJuZWwub3JnCj4gQ2M6IEZlbGlwZSBCYWxiaSA8 YmFsYmlAa2VybmVsLm9yZz4KPiBDYzogR3JlZyBLcm9haC1IYXJ0bWFuIDxncmVna2hAbGludXhm b3VuZGF0aW9uLm9yZz4KPiBTaWduZWQtb2ZmLWJ5OiBZaXNoZW5nIFhpZSA8eGlleWlzaGVuZzFA aHVhd2VpLmNvbT4KPiAtLS0KPiB2MjoKPiAgLSBkb25vdCByZW5hbWUgZXJyIHRvIHJldCAgLSBw ZXIgQW5keQoKICAgSG0uLi4KCj4gCj4gIGRyaXZlcnMvdXNiL3BoeS9vZi5jIHwgMTAgKysrKyst LS0tLQo+ICAxIGZpbGUgY2hhbmdlZCwgNSBpbnNlcnRpb25zKCspLCA1IGRlbGV0aW9ucygtKQo+ IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL3VzYi9waHkvb2YuYyBiL2RyaXZlcnMvdXNiL3BoeS9v Zi5jCj4gaW5kZXggMWFiMTM0Zi4uOWQ3NDA4MSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL3VzYi9w aHkvb2YuYwo+ICsrKyBiL2RyaXZlcnMvdXNiL3BoeS9vZi5jCj4gQEAgLTI4LDE2ICsyOCwxNiBA QAo+ICBlbnVtIHVzYl9waHlfaW50ZXJmYWNlIG9mX3VzYl9nZXRfcGh5X21vZGUoc3RydWN0IGRl dmljZV9ub2RlICpucCkKPiAgewo+ICAJY29uc3QgY2hhciAqcGh5X3R5cGU7Cj4gLQlpbnQgZXJy LCBpOwo+ICsJaW50IGVycjsKPiAgCj4gIAllcnIgPSBvZl9wcm9wZXJ0eV9yZWFkX3N0cmluZyhu cCwgInBoeV90eXBlIiwgJnBoeV90eXBlKTsKPiAgCWlmIChlcnIgPCAwKQo+ICAJCXJldHVybiBV U0JQSFlfSU5URVJGQUNFX01PREVfVU5LTk9XTjsKPiAgCj4gLQlmb3IgKGkgPSAwOyBpIDwgQVJS QVlfU0laRSh1c2JwaHlfbW9kZXMpOyBpKyspCj4gLQkJaWYgKCFzdHJjbXAocGh5X3R5cGUsIHVz YnBoeV9tb2Rlc1tpXSkpCj4gLQkJCXJldHVybiBpOwo+ICsJZXJyID0gbWF0Y2hfc3RyaW5nKHVz YnBoeV9tb2RlcywgQVJSQVlfU0laRSh1c2JwaHlfbW9kZXMpLCBwaHlfdHlwZSk7Cj4gKwlpZiAo ZXJyIDwgMCkKCiAgIFRoaXMgaXMgb25lIG9mIHRoZSBmZXcgY2FzZXMgd2hlbiAnZXJyJyBpcyBu b3QgdGhlIGJlc3QgbmFtZSBmb3Igc3VjaCBhCnZhcmlhYmxlLiBJJ2QgcHJlZmVyIHRvIHNlZSBz b21ldGhpbmcgbGlrZSAnbWF0Y2gnIG9yIGV2ZW4gJ3JjJyBvciAncmV0Jy4uLiA6LSkKCj4gKwkJ cmV0dXJuIFVTQlBIWV9JTlRFUkZBQ0VfTU9ERV9VTktOT1dOOwo+ICAKPiAtCXJldHVybiBVU0JQ SFlfSU5URVJGQUNFX01PREVfVU5LTk9XTjsKPiArCXJldHVybiBlcnI7Cj4gIH0KPiAgRVhQT1JU X1NZTUJPTF9HUEwob2ZfdXNiX2dldF9waHlfbW9kZSk7Cj4gCgpNQlIsIFNlcmdlaQotLS0KVG8g dW5zdWJzY3JpYmUgZnJvbSB0aGlzIGxpc3Q6IHNlbmQgdGhlIGxpbmUgInVuc3Vic2NyaWJlIGxp bnV4LXVzYiIgaW4KdGhlIGJvZHkgb2YgYSBtZXNzYWdlIHRvIG1ham9yZG9tb0B2Z2VyLmtlcm5l bC5vcmcKTW9yZSBtYWpvcmRvbW8gaW5mbyBhdCAgaHR0cDovL3ZnZXIua2VybmVsLm9yZy9tYWpv cmRvbW8taW5mby5odG1sCg==