From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755295AbbBBJzI (ORCPT ); Mon, 2 Feb 2015 04:55:08 -0500 Received: from mailout1.samsung.com ([203.254.224.24]:35575 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755245AbbBBJzD (ORCPT ); Mon, 2 Feb 2015 04:55:03 -0500 X-AuditID: cbfee691-f79b86d000004a5a-b2-54cf497531f9 Message-id: <54CF4974.7080404@samsung.com> Date: Mon, 02 Feb 2015 18:55:00 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130106 Thunderbird/17.0.2 MIME-version: 1.0 To: Roger Quadros Cc: balbi@ti.com, tony@atomide.com, myungjoo.ham@samsung.com, george.cherian@ti.com, nsekhar@ti.com, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/7] usb: extcon: Fix USB-Host cable name References: <1422274532-9488-1-git-send-email-rogerq@ti.com> <1422274532-9488-3-git-send-email-rogerq@ti.com> <54CB6539.1040703@ti.com> <54CB8FBD.6050701@ti.com> <54CF0555.4080004@samsung.com> <54CF3EE7.2050705@ti.com> In-reply-to: <54CF3EE7.2050705@ti.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrNIsWRmVeSWpSXmKPExsWyRsSkQLfU83yIwcy9IhYH79dbzD9yjtXi 1MHlrBaXd81hs5i9pJ/FYtGyVmaL240r2Cz2925gsuh5pGWx/4qXA5fHt6+TWDz6tqxi9Dh+ YzuTx+dNcgEsUVw2Kak5mWWpRfp2CVwZS1esYCz4yFNxeu8itgbGlVxdjJwcEgImEtO3rWSH sMUkLtxbz9bFyMUhJLCUUeL7g1PsMEVNa5YyQSSmM0pcWfUcquo1o8TfO6+YQap4BbQkfr3u ZwSxWQRUJT7M/8QEYrMBxfe/uMEGYosKhEmsnH6FBaJeUOLH5HtgtoiAosS9lRCrmQVuMUqs /38fbLWwgIPEgwfHWCG2PWCUOH3gOdgGTgE1ifYvN4CKOIA69CTuX9QCCTMLyEtsXvOWGaRe QuAau8S21cugLhKQ+Db5EAtIvYSArMSmA8wQr0lKHFxxg2UCo9gsJDfNQpg6C8nUBYzMqxhF UwuSC4qT0otM9YoTc4tL89L1kvNzNzEC4/D0v2cTdzDeP2B9iFGAg1GJh9fi/bkQIdbEsuLK 3EOMpkBHTGSWEk3OB0Z7Xkm8obGZkYWpiamxkbmlmZI4r470z2AhgfTEktTs1NSC1KL4otKc 1OJDjEwcnFINjEnxm5V1toR/Y06R2p9w5uuX1IeTT071ClhzK0Qjoqz1zdctgv57ppd4Tfks d9ja4lCGnbKUmpZ4SMx89XUSx13qMh/8vMUjFPTt+c9K0zyGiibG5lzW7R5+q9/O2LLqUrap qGiNrqysdLVHqKj2yaAtii6M+yvYl2aWvGmSEbxk2JAXcv+QEktxRqKhFnNRcSIAXoi3er4C AAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrKIsWRmVeSWpSXmKPExsVy+t9jQd1Sz/MhBkde8lgcvF9vMf/IOVaL UweXs1pc3jWHzWL2kn4Wi0XLWpktbjeuYLPY37uByaLnkZbF/iteDlwe375OYvHo27KK0eP4 je1MHp83yQWwRDUw2mSkJqakFimk5iXnp2TmpdsqeQfHO8ebmhkY6hpaWpgrKeQl5qbaKrn4 BOi6ZeYAnaOkUJaYUwoUCkgsLlbSt8M0ITTETdcCpjFC1zckCK7HyAANJKxhzFi6YgVjwUee itN7F7E1MK7k6mLk5JAQMJFoWrOUCcIWk7hwbz1bFyMXh5DAdEaJK6ueQzmvGSX+3nnFDFLF K6Al8et1PyOIzSKgKvFh/iewbjag+P4XN9hAbFGBMImV06+wQNQLSvyYfA/MFhFQlLi3EmID s8AtRon1/++zgySEBRwkHjw4xgqx7QGjxOkDz8E2cAqoSbR/uQFUxAHUoSdx/6IWSJhZQF5i 85q3zBMYBWYh2TELoWoWkqoFjMyrGEVTC5ILipPSc430ihNzi0vz0vWS83M3MYKj/Jn0DsZV DRaHGAU4GJV4eG8anA8RYk0sK67MPcQowcGsJMKrYQEU4k1JrKxKLcqPLyrNSS0+xGgKDIGJ zFKiyfnABJRXEm9obGJmZGlkbmhhZGyuJM6rZN8WIiSQnliSmp2aWpBaBNPHxMEp1cDYXRt8 WZottmpSY2JKv8EiO8c7l5U/e02ac85uTZSH7eK7L7Q10vdYxE7ikwq2OKb+9cuRiNgdRn69 KW/Ddottlr+uHHVtrnbotvULOfYXut5SKLpbc932107z+H2dC+TdFk5hdb7A9b+SqaTnwhRv N54Ma1PJ6WdbVmyoOHi8Yt3BbedUd0QqsRRnJBpqMRcVJwIAdq8VSAgDAAA= DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Roger, On 02/02/2015 06:09 PM, Roger Quadros wrote: > Chanwoo, > > On 02/02/15 07:04, Chanwoo Choi wrote: >> Hi Roger, >> >> On 01/30/2015 11:05 PM, Roger Quadros wrote: >>> Hi, >>> >>> On 30/01/15 13:04, Roger Quadros wrote: >>>> Felipe & Chanwoo, >>>> >>>> On 26/01/15 14:15, Roger Quadros wrote: >>>>> The recommended name for USB-Host cable state is "USB-Host" and not >>>>> "USB-HOST" as per drivers/extcon/extcon-class.c extcon_cable_name. >>>>> >>>>> Change all instances of "USB-HOST" to "USB-Host". >>>>> >>>>> Signed-off-by: Roger Quadros >>>>> Reviewed-by: Felipe Balbi >>>>> Acked-by: Felipe Balbi >>>> >>>> This patch has no dependency to the rest so can be picked up as soon as possible. >>>> >>>> Do you think it is better to go via the USB tree? >>>> If yes then Chanwoo, can you please Ack this one? Thanks. >>>> >>>> This would mean that only the first patch needs to go through extcon tree as Tony >>>> will pick the rest. >>> >>> Hold on. Let's first decide what we really want to go ahead with >>> "USB-Host" or "USB-HOST". >> >> Currently, extcon driver have used the specific cable name(USB-Host or USB-HOST) >> without any standard way. So, I have plan to define common cable name in extcon >> header file by using capital letter. > > OK. In that case, this patch is not required. > I will resend patch 1 with cable name corrected to "USB-HOST". If you possbile, I want to use 'USB-HOST' cable name in drivers related to extcon. If we use different cable name, this cause the confusion to control cable. Thanks, Chanwoo