From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964843AbcLOPmk (ORCPT ); Thu, 15 Dec 2016 10:42:40 -0500 Received: from mail-bn3nam01on0066.outbound.protection.outlook.com ([104.47.33.66]:54752 "EHLO NAM01-BN3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S938470AbcLOPmf (ORCPT ); Thu, 15 Dec 2016 10:42:35 -0500 X-Greylist: delayed 12727 seconds by postgrey-1.27 at vger.kernel.org; Thu, 15 Dec 2016 10:42:35 EST Authentication-Results: spf=pass (sender IP is 149.199.60.83) smtp.mailfrom=xilinx.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=bestguesspass action=none header.from=xilinx.com; X-IncomingTopHeaderMarker: OriginalChecksum:;UpperCasedChecksum:;SizeAsReceived:2362;Count:27 From: Manish Narani To: Greg KH CC: "mathias.nyman@intel.com" , "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "Anurag Kumar Vulisha" , Punnaiah Choudary Kalluri , Anirudha Sarangi Subject: RE: [RFC PATCH] usb: host: xhci: plat: add support for otg_set_host() call Thread-Topic: [RFC PATCH] usb: host: xhci: plat: add support for otg_set_host() call Thread-Index: AQHSVjunbh9ctfNV1UCr3AG6lSr806EHZyaAgAGEPwA= Date: Thu, 15 Dec 2016 12:10:18 +0000 Message-ID: References: <1481741708-7946-1-git-send-email-mnarani@xilinx.com> <20161214205847.GA16930@kroah.com> In-Reply-To: <20161214205847.GA16930@kroah.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.23.97.88] Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 X-RCIS-Action: ALLOW X-TM-AS-Product-Ver: IMSS-7.1.0.1224-8.0.0.1202-22762.006 X-TM-AS-User-Approved-Sender: Yes;Yes X-IncomingHeaderCount: 27 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:149.199.60.83;IPV:NLI;CTRY:US;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(7916002)(39450400003)(39860400002)(39410400002)(39840400002)(39850400002)(2980300002)(438002)(199003)(189002)(377454003)(13464003)(24454002)(110136003)(7696004)(23726003)(5250100002)(97756001)(107886002)(189998001)(63266004)(106466001)(106116001)(92566002)(76176999)(54356999)(55846006)(50986999)(626004)(3846002)(2900100001)(33656002)(6116002)(47776003)(5660300001)(2920100001)(4326007)(2906002)(8746002)(38730400001)(5890100001)(81156014)(8676002)(81166006)(305945005)(7736002)(8936002)(4001430100002)(50466002)(2950100002)(229853002)(6916009)(102836003)(46406003)(356003)(107986001);DIR:OUT;SFP:1101;SCL:1;SRVR:BN3PR0201MB0882;H:xsj-pvapsmtpgw01;FPR:;SPF:Pass;PTR:unknown-60-83.xilinx.com;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;SN1NAM02FT026;1:mkhfn5YbbkdQLJiPtuAFoI/w0VQxtdUQCfuvk7Zgx8nLD6hXwYDhcY3x7aH/mbBbsXhUUe9XeJiGbOMiv7toxPB+9kzJ11OpHf4TgPSLNNHCvU5NEMHomwmvYEH0SCLjLlcTojh8SxBrKlxU6/oCx7q1B8EpzMYSSxwiNiYrflHn6PB/aV1YTYQjH6sS+erxRvpqq2tZSCDYqBhfl7r+VAjOx404VEh6qHt1CrgJ7nCu3hjtHy2X8JvkpGrhRmoNTEh4ictfF1W6mlwhwsPBfdrBv/ljrvPJHIEb8gBr607Bg81xKYtsO1+jFv5w94iGQDBPRfylLFJZRxtu1voYXBV8zklpqngrI1fd0AXhbfmfd4aqTH8Nqjw3W3wna+oJ3b8qUwW3WTOpnpaBESqfVCXlTj4B2wq9ltSBSGDFMQhW6EUpqftyFr4K21ndDv1C/FPui4ZpxZtZF3paOlWewsow0YnBuvupb9aSykNyeL2iC9XN6jyozFqHEuDsXinQqzHmNUb34PdzJzKNxJiKACilwjD+ji6BRRfK0NL38e83fVTAJ9Wrvlcq2nyDf1L8l6LNFwBQXBrlLSrZdIuI8djl0lYl/pSCtDU/NWpKgc8lGKTyMMAsWJrODs7mFKDSCGgHMoRXxGVlHdod6bjmuA== X-MS-Office365-Filtering-Correlation-Id: 02d6ceb4-0917-4c9f-c6bd-08d424e35907 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(8251501002);SRVR:BN3PR0201MB0882; X-Microsoft-Exchange-Diagnostics: 1;BN3PR0201MB0882;3:gFTkqC2h/+92v8syeC4/5xddAthnQlVc86onoAUVflUUP47b3cETQOF7FTmB4qJDwFCB6zdupkPrR04NLRZ3ujeXdFtHG6yrClrv3dgxCvbV+vmDVG7kOM8SD4JdLR2EYLWu3nZWnNH3F56Hi1RgUE8t5WbYM51i6Ri5lLqXveTmaJCQSQE3KxJf8TfGPBSz6GlDVuQrcF4t20AnOb3ZF9GW4qI48uMrhZJoMsfeI+pLBcTiUY9b2oWjZu+SFdm9l1Fez2n+dlmkwmypv9z/A301P2/84bCGDEoARZJz2hJTm2yve5ca0mUVM0JUOytjSJ58BM8InrpFirlsDbkn8DyrLhy82pnPeUn6Dyw4dFNtrcMPueSeyMReI/mcMVkhLTxCIbZONRzI5J5x4y1a1Q==;25:Q2An7nTS1PQDL+mcsrLBxeNZWGng91C7G5dfZ3Qo9y8CHRkNi+t9hthvEYQ+WLiUOJayiYRsR129EUb7GCo9jDmQkuV6lXJ2vy/gFoEpDXIqZqq2Szuv6sMm+r1Vj9EOSSbPNFUkj/6V9Cc/ZZgI5zh+QKAvEYfYaH2qob0HM5nXyff7sEFJlGEnHq2Sbp6zrW5fW1mIN0p2/SX7kv8M17zL/va9TWN4Pu7nTQDetE7hAPSfVxquS3Fuu6Qijzyyrwg7JUowMN7jRVuZ8LE3gJfIj0a7/qUrnI1VJ7BHBM1x71dVadBiP3/hm/WfMyYJKld4D45LpO5yJ6TBDmFPq6z0GK6qV7cx5x7sSNj0x/KF+I/nBsgyT8iB+9zxgrz3yejpAaJY3dyDkb3CSDxoEThkd6iq/OjaHFcjC2SANKT/HjNtGhf2PVMFLSY6MsEAVt0P85+2BHGniWX7cPnItg== X-Microsoft-Exchange-Diagnostics: 1;BN3PR0201MB0882;31:cW0RddfGuLI2gozrEUnMEhBQ/7EqCPC8WL+RFJcE+Od8w3nRkf+c+M8r17m/9aYxqc2DAwmcCetjBy+CXLcDKtRi+rpHcJZED5yslhT/Jua6PbLHJqvhOvPHdCGbB2ULG/kNqaUIFOrqTVlurxAgkKWTGwbmOUdE/p5J/B8YblrG8U9pea7gXSHwe1YEfSKGSch+dy1QtYHF1oa2E5J2u2Y/sLAlZ7hSAEwrC0JL0HmPK/96P9OcVaU1YMii7dcAqscr4ucHjij6bxY7VFjszWZrVzaKiw16A8y+AyQE9xw=;20:hc9yi9CAwc9SoG8yp/ZG05g7YA7jlRS1UIlgeZ1HYg2uv3YbGJoexB7atbciANG0DBTNF382r4apD1fVwwQF6nmFhiheZw3n8SwKi0gnB4DfxdFrYD6Rq1xkVMCrigOCIzBoN6V2i3IOfRVYehppGSLi4YS4hOvkabHUv+ioJdvRStTC27WlWHyZ4w/PKiXvhnx5dURm/P93xWpvUMOu2NTAC02RZE/bQepNOjccL/rgn4dI4l/C4Bs8bFJHy6A0tB69nsrEO7vQWQYCSB0hg6BaEPYuGgkQkFugL/B0Jt1VTrfWk29LySnSobdBlP7vu6aEZhw2bVynynpefdbUGQsLB4Uk7myEpmO6h7YpLwYWqZFa2o+mvAoNe0J27u9wJ1ZCsPYYQ+aPv4tpSQr9HBuvgEGC596Uu45MdQb0vj8pSBEICD2XXgufvqjGqo5StN8jI76BW1PHu5eU3VQc/0hZ7RUFn6IJWrTkL/+VtbSEOu3UjI4oC64rwtke4prx X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(9452136761055)(192813158149592)(228905959029699); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(13023025)(13024025)(13015025)(13018025)(13017025)(5005006)(8121501046)(3002001)(10201501046)(6055026)(6041248)(20161123564025)(20161123560025)(20161123555025)(20161123562025)(6072148);SRVR:BN3PR0201MB0882;BCL:0;PCL:0;RULEID:;SRVR:BN3PR0201MB0882; X-Microsoft-Exchange-Diagnostics: 1;BN3PR0201MB0882;4:HNvGBHDwbwKsmoPUK1Bwu3o4ete6viD9KvMoY80GIo8J12S51THOBY/mxBELOJvoBZWT9+/1JQkm2MbLbwaEtF/CaRkqL+YPhh4fiqC3NG+bIN4Ecmh8BYtqiVtmXEHsT725zOB5jevO+fEMosBd+ksjyMspP7IlEy/pEIdr58NJOci+36skcNxSQBdBIrlp4q2qYvDD/NbOodVv7Is2IilXcXIBnOrMCWH36cntdelcHKrV13nwYNwFQp+6mmattFz+PdZDjc6qs+4JO638oD/4x37pT0sNhzn4TQiUWBG9Qo4ht8SH2lMyqfnOIfoPTqI4Wq5NnFUpz9h59kStiWv7g+s3rU/2ywQaMIvZ4J3S9LzbG1yYCtBodjx+hNjWJu0AzK99V3HJtJPGeQDrTrDCV6Q2fyOcRFScNXL70/JwjR9r+U4sh/8JQeKmkSCqeHBXfv2wwdUFKhS7TR/D+EvlFa7BWLURj0ShpYaQbvq5wjsPwXs/c9rSB738Lhdc+xhGv+FZullBxKJlWz41Q1ofa8y9VERpkmI0EsprEtFGREXp6td3WWhz8rwRUBME8IHpg4Kqwt2j/gtcPZOu45XH/J5U3BID89MdkmrH42ke49eRmdr59ntreyAMtLyDUUD132P9f1ntuloIwgshOHc0uQ07kCXA+5KVrLFIX5he/JVk7+3iI0UAvmjMz2hr9ooiD5p0wrE8yn3tFCICs6SMXQ3JU+3KlRRFre3mm9jwE/WwvrU1WUxzq4EtuHYT4TCL/eiYc9CuQgqmiJa1/TLssKwifI+V5Qg6/jf5Ucc= X-Forefront-PRVS: 0157DEB61B X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BN3PR0201MB0882;23:arbnBFjMswlmvs+bWp0AKlNrLe93DlmG7wgUpck?= =?us-ascii?Q?H0eaAGtPoZrv2avkEugZcnToewcRERbt2/yQAQEm2PUkXtWRfuibDUZMALdi?= =?us-ascii?Q?Op5PoyId6RhHy7JnaoqG3D98zRUiuGTzTPj2vV7HgHsnglzdXotvTTo5pP6w?= =?us-ascii?Q?bxyLEZMyarp/k7VFsWVjAFPZbyxlEwDpl72NW5EwrYE79QqYc1YhGEBVh1U2?= =?us-ascii?Q?RLTtEqrnvg+8nZNbkRR1GxD92f3p327kz6eCbTfaVKkMylypqPVpCfqUBt4w?= =?us-ascii?Q?JNihkvXmvaz8l97FVva+myNTYTmZBV4cAMB/TJD0QKg6y+ZpihmfoRLvQxOf?= =?us-ascii?Q?+0J6hYWsXkuWrWlgVkS4ag7vU4HkOPugprA4M9Lrhhh5mPeHizL87b9xSrW1?= =?us-ascii?Q?J9g06PxUOA5kC3eKUEUua5AA9z4KM+vJAZfjpxFjVaSHI6c62e+rfYatYNrT?= =?us-ascii?Q?uVdmL/uMU6/4vvD8MynY9Cj+addqvTVGBGtWpeGS7V221Y7Y/OiuV/hd4uKQ?= =?us-ascii?Q?Gkvn+rxrB1XU/PWGPFyOs6QYWgRBOK1bzA0FPAQepmcLgZyduozdkHpkeiQt?= =?us-ascii?Q?UlAQYq++Oya1sC9FlMxm4Vazr7cMkKFnQbwrstTFXJdhFenHaTH/4nBii892?= =?us-ascii?Q?X/SeviTls9P0ayNtm9n5XXhdRfPDGqg5IXCHf/JeCOUbOg3XSOg7fSsuHPMr?= =?us-ascii?Q?9jql7HT1xtSfYNO1DhvBIKZLoyc/9HklK68bugo2Eya1EUkUtTFV87ji1nxW?= =?us-ascii?Q?t/mApyIkyzyu8V/o3yBsCtKVfNsS/uL0yYoAGy537CbbcVD7ISLBIbCCoF1X?= =?us-ascii?Q?8m7iTjXDFlK+FW5kIbWctyF9msowqhcMcsjoNB38pKqC9fqJY6K7g5hu+fas?= =?us-ascii?Q?bEpB/084hj8Ow3HCIodO9IvwFSL4Mtxc6owgyNq8+oiZqoGy5WpnURaO6gqp?= =?us-ascii?Q?//KGRxq41Mzqb2CujallV6wgizQ5gb5rryvPg3w74nl0d7stnF2QuJr5hDHr?= =?us-ascii?Q?fzqHq3zWoTd3qW5uiDraaXE8HM7r2bx9Dz5fxQsvPDNb3VxvzRwyeQedWnRI?= =?us-ascii?Q?bY7hqT5lQSfzX4PhQycHH9QZ980emlmxX7FR7Ag0Rqm3cXjLisebwn2BMPuJ?= =?us-ascii?Q?Bq5RdAbw/AuRve87TJcx6ziBIvQna3k5FIPjlP9QUNRNCrZ4g4deSRCZ9kJ3?= =?us-ascii?Q?GNU3LHsd7xguN0BICsY7XgyJX3sN5RM4jMFXg3xNy1tq8o8DV40Ue9TMWAQt?= =?us-ascii?Q?Adxjg85kSHYHhRNtQ+V5bRj/07gUHa6c2CgtxStpqGoG83SrT0b9DR8v4zxe?= =?us-ascii?Q?NGstVbN76AyMEHEVQMoGI5x2ygfbA0ERdSxAetD483rzOTOv4GRKv3pP5xzA?= =?us-ascii?Q?WPJP8s9FrRGqaQ8IOO4dxLsDjJ0Y=3D?= X-Microsoft-Exchange-Diagnostics: 1;BN3PR0201MB0882;6:9tURQkEZhRmF9sJ3rQWQ9hIRvifJTr6Fn5fvRUn0TFyqZ2nVR8rESTES6IXCN7vJIRV9vwtxeXCcMzIfkwUgbaIb6ZvsUbMpY3VZx7+It4roS3CcMHdfLbQBe4hJg8Y3TohbZgBaLazYkW21JGT1LC0aCIbEUg0rlkJsFD2R0Pu9hAqRp1KNJApcvyUoAX6aHzcNmQltmJjnExRX8OpnuWhT24aN9yWVBb/68EK59Dz59Aye3mNrBFaxRENdylcunvsdOvVq1V+ncspIYFlEndvbaI8C8Liju8NCOVgKBpNIG2tyJsWl2E9NhwwgCtg7F+Ip2lb02LxVPAwY5EwBKnPqfQLdG39A7zubhcKdebapqhh9FgmKgWHZBpcQqv4FKMN9IfnShxAQLJ68m6pwknbBZSMN6wa/5tNGexYa6spL5FVAw306rEm69rtD8Xr1aCHQKBPlYgZHBHexQ5nD0Q==;5:JSN/SbJb3jd3OoGA4BVI2/QH4ErKZ/8KcqY5Qw7cNAhtQdItuZADnkZsBlGcYIbCa1043QbCuaHnA9OhY/zHcrKDNa9NFJHEUdfgd0sTarUYmupA8WTpD8IJ5oks2s3GWZsdw0WpeODpa0mevsLXXQ==;24:+p5gLo6SsPQTqyj9nSiho9atfV+C6UognP6y6fTDo/NVkFJ2Op6rcZAhwkwFkmbh8u6rFtV9gqj7Nk4b9NAhY9IiTNe3sngeDCIt48BEJ1M= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;BN3PR0201MB0882;7:4hGAzeb/ny3WLls7Hx767vJL2FlN60i3NjnmcJPJ8Wsf9D0rP4C4wnF1IVGOMYAx4QXtt9AxGdv8/tcGy2nAgv3XOr/WbetTiZbvjZdJvlcAUFF/qKbrKvwY2/kIClFEE2iiP8YB5EfhHNXQXbNL7KvuJirzLgrOMtMrIL8AIAFfPzcv7amcVYUhJyV8MVaRrKMtjCTgWI+qzh2nYf4tD7H7dyTEkEUyDhQsn/ns1l1pPnDbqi0uZDfNEySFTDVHMubPHaMCdGfkp04MXrdPaE5VzyhJQYU0Rmbn/5luIieQ5Jo783Czz1dYxu5laTEXdQL/WSLuosk/rQKvW1+6Cy75/Z3nYQcr+VC0dqCc6TMiWpp19SWtpt69ol/McMN7+hQN7cyQxkIhI1KYTDnWA2KXWgnvX2I1+7UbZPI1WFQCAK+QdiPhibunh7XioqnS+sdutOMik7oB0hjLp6S5Gg== X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2016 12:10:22.0002 (UTC) X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.60.83];Helo=[xsj-pvapsmtpgw01] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR0201MB0882 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id uBFFgqgp022279 Thanks Greg. I have send another patch with changes as per your suggestions. Please review it and let me know if you have any concerns. - Manish -----Original Message----- From: Greg KH [mailto:gregkh@linuxfoundation.org] Sent: Thursday, December 15, 2016 2:29 AM To: Manish Narani Cc: mathias.nyman@intel.com; linux-usb@vger.kernel.org; linux-kernel@vger.kernel.org; Anurag Kumar Vulisha ; Punnaiah Choudary Kalluri ; Anirudha Sarangi ; Manish Narani Subject: Re: [RFC PATCH] usb: host: xhci: plat: add support for otg_set_host() call On Thu, Dec 15, 2016 at 12:25:08AM +0530, Manish Narani wrote: > This patch will add support for OTG host initialization. This will > help OTG drivers to populate their host subsystem. > > Signed-off-by: Manish Narani > --- > drivers/usb/host/xhci-plat.c | 35 +++++++++++++++++++++++++++++++++++ > 1 files changed, 35 insertions(+), 0 deletions(-) > > diff --git a/drivers/usb/host/xhci-plat.c > b/drivers/usb/host/xhci-plat.c index ddfab30..b4cadbd 100644 > --- a/drivers/usb/host/xhci-plat.c > +++ b/drivers/usb/host/xhci-plat.c > @@ -20,6 +20,10 @@ > #include > #include > > +#ifdef CONFIG_USB_OTG > +#include > +#endif never use a #ifdef in a .c file if at all possible. Here you don't need it at all. > + > #include "xhci.h" > #include "xhci-plat.h" > #include "xhci-mvebu.h" > @@ -255,6 +259,24 @@ static int xhci_plat_probe(struct platform_device *pdev) > if (ret) > goto dealloc_usb2_hcd; > > +#ifdef CONFIG_USB_OTG > + hcd->usb_phy = usb_get_phy(USB_PHY_TYPE_USB3); > + if (!IS_ERR_OR_NULL(hcd->usb_phy) && hcd->usb_phy->otg) { > + dev_dbg(&pdev->dev, "%s otg support available\n", __func__); > + ret = otg_set_host(hcd->usb_phy->otg, &hcd->self); > + if (ret) { > + dev_err(&pdev->dev, "%s otg_set_host failed\n", > + __func__); > + usb_put_phy(hcd->usb_phy); > + hcd->usb_phy = NULL; > + goto dealloc_usb2_hcd; > + } > + } else { > + usb_put_phy(hcd->usb_phy); > + hcd->usb_phy = NULL; > + } > +#endif Can't you wrap this in a function to get rid of this #ifdef mess? > + > return 0; > > > @@ -283,6 +305,19 @@ static int xhci_plat_remove(struct platform_device *dev) > struct xhci_hcd *xhci = hcd_to_xhci(hcd); > struct clk *clk = xhci->clk; > > +#ifdef CONFIG_USB_OTG > + if (hcd->usb_phy) { > + if (!IS_ERR(hcd->usb_phy)) { > + if (hcd->usb_phy->otg) > + otg_set_host(hcd->usb_phy->otg, NULL); > + usb_put_phy(hcd->usb_phy); > + } > + hcd->usb_phy = NULL; > + if (xhci->shared_hcd) > + xhci->shared_hcd->usb_phy = NULL; > + } > +#endif same here. thanks, greg k-h This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.