Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752110AbbH0DDN (ORCPT ); Wed, 26 Aug 2015 23:03:13 -0400 Received: from mail-bl2on0127.outbound.protection.outlook.com ([65.55.169.127]:8033 "EHLO na01-bl2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751413AbbH0DDM (ORCPT ); Wed, 26 Aug 2015 23:03:12 -0400 X-Greylist: delayed 858 seconds by postgrey-1.27 at vger.kernel.org; Wed, 26 Aug 2015 23:03:11 EDT Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=freescale.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=none action=none header.from=freescale.com; Date: Thu, 27 Aug 2015 09:37:21 +0800 From: Peter Chen To: Heikki Krogerus CC: Felipe Balbi , David Cohen , , Subject: Re: [PATCH 0/5] usb: common and dwc3: converting to unified device property Message-ID: <20150827013720.GA24001@shlinux2> References: <1440500675-55872-1-git-send-email-heikki.krogerus@linux.intel.com> <20150826045319.GB10607@shlinux2> <20150826101322.GA12746@kuha.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20150826101322.GA12746@kuha.fi.intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;BN1AFFO11FD023;1:bSws+JLjei/50t9+BMpsHASA+qORoy6C35izZY7gK6+R/sWzFLWwehUkW4YGJiMsGM6C8VY9VPhkbVw2AfkZPoKTBF/pZHRzlSeKz6nHFEJyk6yQePLoLEp15alfYgJO4iPJEoELb95uZJr4bESTWwKvzMHYhINaDrG74JLoOx+sVxNq2daCW3w6943Pe0siQv2iQbMDOQGd23kBf9ljGkSQ4oh/3CB8qj3w2q5RbBPkoZ656rfJVbKWmy67gNGG2ArlXWtcGTpEUJamDNLsoHkXTCqu7R7E5y69HsyWW4P1aU9DVflpAfXJdutj7nF0hA21K6WjRwXcrp5lc3vyCL/Qa67pZZ72HYTJQ3LbEfzqrgoTU2jy+FlBUOh8QhMJew3q+FUkW1mVQROKq9wBAg== X-Forefront-Antispam-Report: CIP:192.88.168.50;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10019020)(6009001)(2980300002)(3050300001)(339900001)(24454002)(164054003)(199003)(189002)(6806004)(87936001)(76176999)(50986999)(97756001)(4001350100001)(86362001)(64706001)(5001830100001)(81156007)(47776003)(4001540100001)(5001860100001)(54356999)(97736004)(105606002)(189998001)(46102003)(106466001)(110136002)(23726002)(5001960100002)(33716001)(83506001)(46406003)(33656002)(77096005)(85426001)(62966003)(92566002)(50466002)(2950100001)(5007970100001)(104016003)(77156002)(68736005);DIR:OUT;SFP:1102;SCL:1;SRVR:BY2PR03MB491;H:tx30smr01.am.freescale.net;FPR:;SPF:Fail;PTR:InfoDomainNonexistent;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BY2PR03MB491;2:dYAcozZlbn2nmmXn5CVtbTMfChYqeiSw3n0KDpFwR62iDXfhfPhaplj6LwQJaHpSr94XN+JKyax7jQbiAQhaag+GWkgEaixuQyZsYdpy9JxfeESnT2WYuK5D/QqcElBE251A90tAIcCaL5OBf4eK6kuf1tfsBp3wz/RcYAUGnVQ=;3:kSDm94PhEaDmXj0UbBQNFHvF27BF8tez5lBFq+JA3HrprCBKwhCLofrJcWzoJrLIrbAVVnDilM+zYkxrm6XYT4FW4Fuui4NPC60TIEdxWkzMuM20y6+QZ9DA1dEkH6hxe+DLMPh1tXwfgnKPuuu4/AM6FWKt+thHsP1xpVq9mmEEqkBaGXuyF+djzSR/cUXDwuZiGNpmzsgg20qk+YmkStq4oOv+zSigUcvo07SZ8IE=;25:0HcfvNb3DNmXSvQBSjHJbOULBl9LAuKqmMJUxnmr6xQCrgNHs1cmuvzmdWL/PJQ+xwWGHecROfIW2cH94MrWclThVvgAoBQ4DzIfgHAiW9QqkhpAne3TRobvRaWzyL3qNv7Mvm0yxlwbdfVXinmaAs0OuxBa/atLuQEz8U+LsYWCHQom5LkMEEUI4JRJF3rOf0yQlZ86X8+D1OWlb+/izVmmrOvUzms5CCsk9dYoUgG5kKv0EK7MU/TxBHgmx9kR X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY2PR03MB491; X-Microsoft-Exchange-Diagnostics: 1;BY2PR03MB491;20:479eFNUAC5YcpxTIpWBduHotBzLpuqntS3C5sVZAG0iLD/x7dGAxrNP+kJbqN4uzzR35NVRRayvb90d4whj4wvhkvD+sc4ME51CHLt4j+RXJ8zuf3aQYuuPIKKaO8nlh2aYOP9zaAPDIkzjlq8gPVUW/DS1eAimeyu9Vv1J3rRj4p7/gsGLoep/Phteur82IhCf8RnZ8Nq2XRNJRVRFAfcl2sGMzfCy9cyNhNRgDxmG2I5pwfvoRIHOak8zZrVr2+dPwn3bSTs2voMsjPBqHpMXVBJJwRDNdBDKC3fgSNKu7vOHe8Rcbgk0l4WzKEj3tu0FcBXdGPX7Cvs+YPtI3d2FCBn0H+wRT9JAV3rOHrnE=;4:ZQAcVFXvgg3R6NqR9r+1cfCnUIu2O9lL/bGmNLb2aLQDC2zsjLI7pGNc6Yk6TiPneObTn76pG/AQd6kurhcl5zv6IgRuDf2pRZghoHtSso0dpHD2PBeSTwof932+MOJiyh/KqbMUBIxOXKEmQyfJBq3wd0YYmAD0MAUnKjtptOW667G7j5EZ2XM3VUgV7doti9saae7oV9H7/dkKsReBGkskCLGaXV7Fp1fTF7Pg0PxhSxp5jy/nlVQUMA0wcAH86G78/3YwogLMS3pdjH9sNHldkHGKWyI0yPWCkR7lbK7ViLiZfswhcx6Ikb1BaBGG X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(8121501046)(5005006)(3002001);SRVR:BY2PR03MB491;BCL:0;PCL:0;RULEID:;SRVR:BY2PR03MB491; X-Forefront-PRVS: 06818431B9 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BY2PR03MB491;23:QPn59rXt12Nug50hV7cPUp6NA/KnbmcJiPR8nK30Gy?= =?us-ascii?Q?ONmXdG2xj+KS4Kijd/JSt7w2H2Jygw7PmcTYYDicGSmc1LYrLwTTmbdlVSb7?= =?us-ascii?Q?6jBp15LH0oEOWT7vpScBaiZFDhRNgU7PBvAKo03x+mELWJiAxiBXzf59YQT1?= =?us-ascii?Q?cnf8LpUJaCc7U5Jmti5imoREAZLOkFOoOo7g62CHFkCtWKhVCTOpxBWgL9dt?= =?us-ascii?Q?4SGc+V1stgO9ZNK1hKtMOTfcRPIvzix8Ftj5f4q7485ysF6Dr82Gp6kb62aw?= =?us-ascii?Q?gYGJC8ar1H2CQ1lL0xPcKRcG6zNIzAM5TPzplqP1J+Ciz1X5EiiPaYXWREro?= =?us-ascii?Q?2jB8P8KPu6911vafp3VG5Y1+DEYB9nVZ+FlBoJDnR83LWd8gALHmU5YS2vpM?= =?us-ascii?Q?0BEYPqMWXkAle2nfuT55t5p1skr1jUv0ZCYYQNUVPmLtj6xBfC3A+BtNP0KJ?= =?us-ascii?Q?myUIIutyKON5s3rqez8ORFC4iFuNoyx0PlkaxjtNEMBsw6/r0U4CUnFRcKKF?= =?us-ascii?Q?MWthyk1CIJOtzHdGvg6tq5Vy+1ZfCULZuRukwCBuDYxG3npx/tzNxpKBVqOR?= =?us-ascii?Q?iTyBEp7r3kXGYh9YEJFfC+rpE3SwJXe2JztBLVqeOeNNOu+cOafuwLN1p3A5?= =?us-ascii?Q?bqGcJQOVT6PM03teO0hd22yhEjhpFo322OYVPEsGYY3oF6VTrqBJ+mXgPKqF?= =?us-ascii?Q?5u4L+QuHjdDsFKbnCVREWdSWhAV2KnPz/zQqUpCS9aHh2j/ya05RW+Hk3uo+?= =?us-ascii?Q?bwvgasSkEzamWxQHibFsFFj4Qi87vQ6r6XRYM9jkntFN45AeuBk1btf3FDVP?= =?us-ascii?Q?ITSzqwm+6ejDmEwjHhe2aivCEv66S4AIw10839D7Gxg/9vtTj/3NfISuU/TV?= =?us-ascii?Q?yrtvtuLa2y8kgQazLRWXSf3rrTQod9GE+HAVYoZWauj1uL1nY1suWthy4ppr?= =?us-ascii?Q?DrqlzhovS1uI0Q4bklwAu3iB79G70+JzOoAD5k4xN50bpVYX39O26a7WGhLF?= =?us-ascii?Q?hDk6zwAOhxY4lxv1mtCi6VNEODlCn1IGOMQ3aAD1NPNrDpw2SniZZi0iSHSB?= =?us-ascii?Q?TlMBizC1RVkgqSdKif1eh/bgp7c/pwC7U8W//+7mDjX2uczRqdUO/FPstEUk?= =?us-ascii?Q?o/i80sY4vBCQLhm96BH2j+6SRdaw32A6jLDthLbN1wIkHvoypdUnl8oD4ieh?= =?us-ascii?Q?dM20rNGveTsaQ=3D?= X-Microsoft-Exchange-Diagnostics: 1;BY2PR03MB491;5:1QcZrXio0PF1+QWdQ77jXCRw8wSY6BF0hooAztFyUCHAUPA2gqoC/bg10G7/QAnqdbpy6JpJY9N2N2stmlfExnRcLodMXqV9SADJkqt2w1HkHtaUVJ2+NhU78B20Dr0QGaeDSllJ9xdilhFz2XnAaQ==;24:CKAdxGIPa7btAqD41EgFlNVdJc9HsdG5gHg4uowp/hg6Ix1DgIxQM4qdBug0VfC0JC00nGjj2+z+cUTA8lXdvTHx23WaEozu+1O0EmBMcLA=;20:rb3F0dhKwT9WZMc5ns/LfrHWZqx8mcsXnQw86JlS+qCylKab/tAJHkYurwzXKgkUiRBis46ZpRPE0K6s0GmNSA== SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Aug 2015 02:48:50.7136 (UTC) X-MS-Exchange-CrossTenant-Id: 710a03f5-10f6-4d38-9ff4-a80b81da590d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=710a03f5-10f6-4d38-9ff4-a80b81da590d;Ip=[192.88.168.50];Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR03MB491 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1606 Lines: 45 On Wed, Aug 26, 2015 at 01:13:22PM +0300, Heikki Krogerus wrote: > Hi Peter, > > On Wed, Aug 26, 2015 at 12:53:20PM +0800, Peter Chen wrote: > > On Tue, Aug 25, 2015 at 02:04:30PM +0300, Heikki Krogerus wrote: > > > Hi, > > > > > > While converting dwc3 to the unified device property interface, I > > > noticed that there is really nothing preventing of_usb_get_dr_mode and > > > of_usb_get_maximum_speed from being converted as well. Hope that's OK. > > > > > > > Place the reference for usb_get_dr_mode and usb_get_maximum_speed > > at otg.h and ch9.h may not be good, why not add a common.h which > > is at include/linux/usb/ too. > > Why? I'm not going to introduce new header file for prototypes which > depend on constants defined in other header files, unless there is a > really good reason. Please note that the prototypes for the existing > generic functions in common.c are defined in ch9.h and otg.h. Ok, I just had considered host controller driver may also call usb_get_maximum_speed to get platform available speed, so I am wonder if it is suitable to place it at ch9.h. > > These functions are _generic_ helpers for dealing with definitions in > ch9.h and otg.h, so I don't really see any reason for putting their > prototypes anywhere else then into those same headers. > > > Thanks, > > -- > heikki -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/