Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755426AbbHZGEv (ORCPT ); Wed, 26 Aug 2015 02:04:51 -0400 Received: from mail-by2on0116.outbound.protection.outlook.com ([207.46.100.116]:35347 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751216AbbHZGEs (ORCPT ); Wed, 26 Aug 2015 02:04:48 -0400 Authentication-Results: spf=fail (sender IP is 192.88.158.2) smtp.mailfrom=freescale.com; vger.kernel.org; dkim=none (message not signed) header.d=none; Date: Wed, 26 Aug 2015 12:53:20 +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: <20150826045319.GB10607@shlinux2> References: <1440500675-55872-1-git-send-email-heikki.krogerus@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <1440500675-55872-1-git-send-email-heikki.krogerus@linux.intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;BY2FFO11FD003;1:mQJAY3HUXAAqlFsNICKUoBTD/ApaCQhmvIaijqpKgqRzZrg+sj+BPBAWtPx0T23Gm8oPcnjIE26Nv/k2ZvwUDmDN6oOkXlAhcM96qfoJE+iAej1hk+kOjmIolfF9aI3u+Af2SS7UqVCeJyNJJSM33pnsk1sJu/UM3EcSmrt4Elc/N76qPjSy9nDQd3kKNoLPFPieil1sN6LlkG3ZR3OH3V/Q7LT9kcy0TmOOZPIEFxN/6L6/ZdyaETdJptyNqbY4YJUbCbOxVl999vtXHlhg5i6k4RLZ6ktKhy3g11xorgB1ME2iBzBD06Flzv+7043pEjIC9s4FlbEOd5b/JPPkqQQJoTcEwymuSc+qODeikqQ1bkG08iZicHJV/DTYQlZPyuE0b6qF782FefwNPKsNIA== X-Forefront-Antispam-Report: CIP:192.88.158.2;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10019020)(6009001)(2980300002)(3050300001)(339900001)(24454002)(189002)(199003)(33716001)(4001540100001)(50466002)(97756001)(5001860100001)(189998001)(19580405001)(5001830100001)(87936001)(23726002)(86362001)(69596002)(110136002)(6806004)(104016003)(4001350100001)(76176999)(5001960100002)(50986999)(47776003)(64706001)(83506001)(2950100001)(81156007)(54356999)(33656002)(97736004)(105606002)(85426001)(62966003)(106466001)(19580395003)(77156002)(46102003)(77096005)(68736005)(15975445007)(46406003)(92566002)(5007970100001);DIR:OUT;SFP:1102;SCL:1;SRVR:DM2PR0301MB1280;H:az84smr01.freescale.net;FPR:;SPF:Fail;PTR:InfoDomainNonexistent;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1280;2:PtNauAqRuWANiBO36JGs1h54XtbSxqGVDzXfKX6HbwfnZChYWBF8w6rItHpdh3G83rIrhYmApY6AFjKatqZp3HWHEqFyKyRegpY9bR1dt+sCWkmgtjcJzzh7c8xTkMW2aLjlxjMklZVdC/bBx2oIlEuaIxm3bYPiaPn9sKfpwIM=;3:9oFzwEsd2FN7BF/9TsuJ+EmeP2gAWSs3Wui2+1iKqPtusMrEKyuiDk6xWLVdJj8vmg9UeG87JrM3CjapqA2R2fWVgkRCYcnVpiFQHQia8O67sKfVIQ3As7iBJjjjHrIBALx8aWHhuqqBPy55tWfH88GrCJk085Zv6H6OQm9Nsyh7Ypm/gs8BznyHBbeAmnmw1RGtpC0wLfRMuV8TCzbtvz4lMDRTZf9X60E/bfBMCKd8Xa5oMUfNTfkOYgILLcXHAUPLj3Oe9hvmC4EamXf8eg==;25:2PEfUCWJO4OvQ0xgKCxvxxWiGdjh1hVwEPLDgmExsAqSo0IyoMmpVvMvD2kp9OnHsNAEOlGOp4JQlo/27h+MU299nywCE3IKjhCSWlN4PXoRhTeOmk21cdERd0zXJMR8M0qVSbtDUAtoxgyNy9kUDBU6GSyiG00QjLcfLcOQv+clNI6H/hECqTrG9GrMWKdERRVK+WpxUhlpizI4Zs4YnRL1QpLMp6MH5vxSfRIzpBQQ4yEPBSL41tXsSSIBhBK2luGB1xxdNcICLff4P0UNdA== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(42134001)(42139001);SRVR:DM2PR0301MB1280; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1280;20:eYkU+0SCFrgCenHDM9draj4Z7jxw6HVvv10kE5YuNESDvAbX/STPut/OPSh1fO73nk+OzoFIqgzvRQjwzWYIsR7+JvNVKAvXRbsV4SM0Y1me7x7YcvW8D7jYB4n8qh9cGEneZZTL3S5Xiv3mYsSMqIuy4j7x9P1A2tMU31YMA7F+GYpZ1dcgZPRpIrRVBZUx0N/eqAERJltRX1hkZPw+ptpvnrXtjoow3blNxd4IV4UBtVJm2eCr5ro7ujG7zkaUn9eZAk0ehyIPDeTyAa4SpObcMWeL3jQ07KFj0wQEB0+30+ZLjKYULqcMLW/SGcQh149HPYEBwbvsUHWycQkvxXX45CkgBV5c37sWZvykua8=;4:XbEsP5qrV0s1w2RSKXziU5s7y+MTyQAk5dsw1KQm6X093y+YsgrOobE8R2/w9+H48TrY776RHFdz82HEXh84MODldBAopZjH8QydUBkGlMMTHWOJmgf7EiIFOftdy0YDs8PZQU+wZS9AOE6/vm/aICOM/a1wFwiX4AQ/kuyvf49T6of37Otb74zLbSUY2FkobcSIVbVg2Pd3wY0tFymSWomKJdA+dMuaam97ZIr4xtwPGFpdcU/LfYu1Gn0mjxd7XOys0YtoRsa6dIExaPBo8AG5oRgVGHThGfVhV6fV+f8lt2DqgnPIY+d0/AP8C4xs 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:DM2PR0301MB1280;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0301MB1280; X-Forefront-PRVS: 0680FADD48 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;DM2PR0301MB1280;23:pzyjVYquLZLXb46Irwo0D1Px97te6gPgk+t+4FB?= =?us-ascii?Q?QNRgtbeJzd6n1xm4NihACrK5qtih9fG9ziVH19r8am05NljAB2ASFP3Z0mxP?= =?us-ascii?Q?qzMYoeYGthVhHGP3lXtnRcJqu6ehIgOC9fGDQkORNCHfR8fJR7aZLWKf6FkQ?= =?us-ascii?Q?GelUVNXihi38ywRvx6bVHll685Fn6AeuFBCwmP3bgXqgsWeVdZpkctJSQhw4?= =?us-ascii?Q?22mD3U2970sPB3xcKNsBhuarxjUWOK86uaMMJ441/728auW23nu+Dfov3kL7?= =?us-ascii?Q?/Is1uBApqNZd4o528PA1frCU49r5KVdmFAfw0SQXZQlSTLSfD+0mb8N8fmKn?= =?us-ascii?Q?5iTV2QTzicCHi+R6JVnRqGlFQvDDFgwEtJtZjKZGSUSouOjm7FxE5rw5rlRq?= =?us-ascii?Q?V6zb5x3Tmd9Zr+GPE4qUdAKJWGhjD9If/njmwkBry9BIYf5wnCfaAnpJC7pw?= =?us-ascii?Q?UK1jdrv2uONj/AZXaCg9tkrHgT8OcVUDLhxGL8rLEvInjLLgpvcMcgdisZ5K?= =?us-ascii?Q?qlgr58U4rXVh8PK3y/g6oD9IxmvgNgSW/0FWENFsVIN1+d8ay3JnFHO/mHVK?= =?us-ascii?Q?kUVcRgsDluof8MgDHG+wVL5znE/QAJj/58bK3JmNaPnb1+JbfvM8LM0nDtnT?= =?us-ascii?Q?p//GP+SFp119Iewv47/5+d4hq3CaQbit2MzJfHMU/i1+I+q+ALEms3OZaVC8?= =?us-ascii?Q?VGbVnCu32VFh4QZzg6psp4vWtaddKBRb+PndvqMHinwmibnFEmwRYBm0Rx/t?= =?us-ascii?Q?a3hRv3FbbXUeqCCk7XOT0RaRHhqhOhuqb8Tt141rOxhndDB+Eh7+CjcWejlA?= =?us-ascii?Q?4WDCbgK5MjmizZv/vux1UXb5xncV+Agm1f6ilPXHE0ConnnTTZnNxuIpJs6b?= =?us-ascii?Q?PQ4lv+Eh1clIIBT7ILb6P0pIfxHiS8s3yQ8X6WqFcNxl2cXx9LqSp1SHOuxB?= =?us-ascii?Q?2xgVbDbsgXcqoB0JUjukyXjup3izw/JHkfKoaU7/R92B4ihkHgSgwn+Vd/Fb?= =?us-ascii?Q?L9mwMJiQ2zIunUbHm26RQgv9DZ2Lk3A5KJwnuv/WgCIPmQ35adfROEPAT0Nv?= =?us-ascii?Q?f9m4p13pvvsqpYXB5EYua/EllxaYSNHNaSGhInbCktiJBu61SoCmGLybxELD?= =?us-ascii?Q?egJNapUsKj15h01H4AVpbKSIWcn3YRuAWy9k1/FEfByO1l7eRaBc+yeeLO9f?= =?us-ascii?Q?klCGHluD1BYrw++crhjsm37g3egCPZ7Ao63Np6Z7cuidR1WgH2FHCrRNwfz1?= =?us-ascii?Q?Wmg3qUQcI9cDoEYZ7mdNNw5ZZe8SnnUjlrWixCK3d?= X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1280;5:gjLkAUZdShd+9I6isJKTNGo6L++jSekciNHWMOpmp33RZE/jtzU2VtLPxWYJIjJ9PALTuWG2b7tmUIHj9I4iAlX3DNMyOl74LuI1F2nVDie+RMftQ0d1GsNm7BsJ/qr4G52DrmLlCwvwsHG8EF6WpA==;24:xWi5v735LXuU6Jvyayg04PwRX0Ahqpi+o2IDhiKSAcI4GPwyqqMPFhU4noRVfgoVULpSqfZJG/peoMguiYPaYIvDupdiEOi/nm2ixmaQKNM=;20:6RzjsVM8DCmONwkstCZwbW9J4SsFvryQqjMSvIEPpY4lMv8GNxiurl0hIxnIQx+UZY1jc/DKvx93LhrUuTh/tg== SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Aug 2015 06:04:45.9669 (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.158.2];Helo=[az84smr01.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0301MB1280 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2249 Lines: 57 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. > The only special case was dwc3-st.c, where the dr_mode is requested > before the platform device has been populated. I changed it so that > the dr_mode property is requested after the platform device is > populated in a separate patch. > > > Heikki Krogerus (5): > usb: common: of_usb_get_maximum_speed to usb_get_maximum_speed > usb: dwc3: st: prepare the driver for generic usb_get_dr_mode function > usb: common: of_usb_get_dr_mode to usb_get_dr_mode > usb: dwc3: core: convert to unified device property interface > usb: dwc3: pci: passing forward the ACPI companion > > drivers/usb/chipidea/core.c | 4 +-- > drivers/usb/common/common.c | 59 +++++++++++++++-------------------------- > drivers/usb/dwc2/platform.c | 2 +- > drivers/usb/dwc3/core.c | 50 +++++++++++++++++----------------- > drivers/usb/dwc3/dwc3-pci.c | 1 + > drivers/usb/dwc3/dwc3-st.c | 12 +++++++-- > drivers/usb/musb/musb_dsps.c | 2 +- > drivers/usb/musb/sunxi.c | 2 +- > drivers/usb/phy/phy-msm-usb.c | 2 +- > drivers/usb/phy/phy-tegra-usb.c | 2 +- > include/linux/usb/ch9.h | 11 +++++++- > include/linux/usb/of.h | 12 --------- > include/linux/usb/otg.h | 9 +++++++ > 13 files changed, 84 insertions(+), 84 deletions(-) > > -- > 2.5.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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/