From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754235AbbINDTM (ORCPT ); Sun, 13 Sep 2015 23:19:12 -0400 Received: from mail-by2on0063.outbound.protection.outlook.com ([207.46.100.63]:24736 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753417AbbINDTI (ORCPT ); Sun, 13 Sep 2015 23:19:08 -0400 Authentication-Results: spf=pass (sender IP is 149.199.60.100) smtp.mailfrom=xilinx.com; rock-chips.com; dkim=none (message not signed) header.d=none;rock-chips.com; dmarc=bestguesspass action=none header.from=xilinx.com; Date: Sun, 13 Sep 2015 19:47:02 -0700 From: =?utf-8?B?U8O2cmVu?= Brinkmann To: Shawn Lin CC: Michal Simek , Ulf Hansson , , , , Subject: Re: [PATCH 2/2] Documentation: bindings: add description of phy for sdhci-of-arasan Message-ID: <20150914024702.GA19678@xsjsorenbubuntu> References: <1441961698-10516-1-git-send-email-shawn.lin@rock-chips.com> <1441961729-10594-1-git-send-email-shawn.lin@rock-chips.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1441961729-10594-1-git-send-email-shawn.lin@rock-chips.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-TM-AS-Product-Ver: IMSS-7.1.0.1224-8.0.0.1202-21812.005 X-TM-AS-User-Approved-Sender: Yes;Yes X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11FD054;1:s6kde/CcWeycnl40b5S7LDcARAk6qNTKii3dHYQULYKgMFVSzLENpmWCg93b7k/LdaSj5QTazW+CxTtenzmiZC7huiGWbrGpeRoj+kJlL0N1NK1nUhITKlelXWJQZ2WB/GQcMcl8iGXkZCNLrM64Qj5TPq4SXN4cJsa6dIIRqy4t/YyR5H/YtUSc+jSfBySjKYB+TBd7wlKb83ZtLJPW2BnraAC48IWSASDmM1kSzLpMaqI7BX6GyG12sbsr8Qwldf69PONRytQK0PgDTDnTFFHWOvjfzFR7Qktl3rBAMWqx2eidSvuamihy+SddOC7xmH3WI6VQ87eXGwnqlpsZHkPE3H5nUWVI2T1Nkqp11RI= X-Forefront-Antispam-Report: CIP:149.199.60.100;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(2980300002)(438002)(377424004)(24454002)(199003)(189002)(4001350100001)(54356999)(83506001)(86362001)(76176999)(64706001)(19580395003)(2950100001)(85182001)(33716001)(6806004)(11100500001)(50986999)(50466002)(23676002)(106466001)(47776003)(230783001)(92566002)(85202003)(19580405001)(46102003)(76506005)(189998001)(110136002)(63266004)(5007970100001)(77096005)(16796002)(36386004)(33656002)(87936001)(81156007)(4001540100001)(57986006)(5001860100001)(5001960100002)(5001830100001)(77156002)(62966003)(107986001)(5001870100001);DIR:OUT;SFP:1101;SCL:1;SRVR:BL2FFO11HUB040;H:xsj-pvapsmtpgw02;FPR:;SPF:Pass;PTR:unknown-60-100.xilinx.com,xapps1.xilinx.com;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11HUB040;2:e+8p4SG8DNKD2QioMFD0qfFeB6B5PSSTE+u/COkkmfYxAz5ngZnzfe6+MO/ODpwaQxAQeSmv0crbJrC0ap0rPWywH8LLudm836jkR2591rgJTgIyXYJ0wzmcvL3DKOeknmcNjBtOOK9E8Y7LyAVv9SxqOaSM+/qZvkn9Vd5gw+E=;3:Urtz2OvdYo4uyuB8cSJT1u/K3/fbNyXVSN3Y6IgbjFC8IenN32p8QBJFw7jpy/OiZuxL9zEuAipfUnpdUX7MRAW6vAl302PwCbtTCgV/AvrGf5Qcqar5jCrBvnBEt7+fhNYuUZu7xDVJ/A/pc1+o4IuHYB0gCu8Cl6sQPr+5SXJ0HY1iVSEzHY8NK51pBSeUr1YUBdSF8ijNxM4tAa3Okwn9qxrAP/bugebazk9O/scLGI/XtXVLATcz5Z0ukplPSUjzLz+V54FLxjguFKUYMQ==;25:EFPzR8eGLD6uwEZVcW/T/GqvtkuibAYpsn5CFuARVa8FFrYH+k6zsdXyXiKb+n2WVUyGdzro2I4fWM4yriIHHzpiGSV9MkoTNITedqJjj/2FyaC6cZhPTubXEURv9NzM59NAxzq7YQElJW4zbNyp5TvKmNS9X63uQ7l5DEfRXAceWCOBtXMwmZdLwBQsOF2caiiKtfEvW+3BdZAiipfer63cSPOajTa8/3Gw5BtnpILx4j/G8C8rkXraM4sqWcYEATmf3owwKmMvb7HLjhl6TA== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(8251501001);SRVR:BL2FFO11HUB040; X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11HUB040;20:Dc3ESclcZXtMEdSUHbIsbcukmKet8iJEGg4oJrAzyLGiDwETXnsO8QUE0mxN8PwydLY6Tg0d0LsGMy41tAPnHCUsgN0wgbUBuokUU2xSC+A3cpSzprdwSyuukCBCuTT4T8DdF+nqZn5vHJHwp60tBL3sTmOh5yByEtWYD6jJ5n6pOMpI1/ok7yKmlQkX/vx97WUzdVi393N4VDwr3Yiwi1V3Ex233L2tqeY+HU2zK5G99shuR19Jtb6+x0JFBM0QS6Tp2yfrTnIongQ3jS6poyPjEUC2cJE/wBvMkrY2dNN6CiP+/hL0Da8YrMsUPBYMGdt8QZfFEXPsOydfTy+aTo7wtvLxWSkAgymF89JXTSfOv+Arf0+JKfj3rvKCGDKvHM1854T7VpzbmVWGAlPLyHjZOMqhcg0KFNrSZW/5VHodmzgaYI2ielRg9gI+XJKq8XD6VwnXDuaLnIQHZ347b3ih8WjmTvwdE1K+QyT9Rj2lqnSGaSu9boXRs7m2kFz5;4:+sh7WwzoFkVAIZSgBp8HkqUJ8ncrawPeBGxzm6mNhQkAPvpg4hYjEjt2LgQ3z5HuUCaVclG5yyfqXOAFpscuLrrMsy0PLJ08f6+NM/y2XmbneafBdHXaxRKBpyGsm7Bl5AEj0Ozw3e9ifXiKwxYX65qShHlqyCSgSnCc8McTbu6E892HkV/fWdRy73hOxOq5uegNIkHLADOO1ZXFhixCWu8Si8p1KRYfmC+kTkkWaF1jwCe2qDiom0YganqcDBSKUCcEfXxCSFCGVbWN9IETEq0YK9Ot+n0ury74cQLvD/eolTQiI3IrXfxndeIqCthWnCQFgrwuJA+Na9R8NJ1M5g== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(520048)(5005006)(8121501046)(3002001);SRVR:BL2FFO11HUB040;BCL:0;PCL:0;RULEID:;SRVR:BL2FFO11HUB040; X-Forefront-PRVS: 0699FCD394 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCTDJGRk8xMUhVQjA0MDsyMzp3MStIaTZJWG84TGNKVkIrZUFQSnpoVzRn?= =?utf-8?B?WHFncG4xekJSS01lZVk0K253ZDFVZkxBYzc4UDI1emloaCsxKzBiU1BSUjdF?= =?utf-8?B?d0JrQVhrQXlYenQ1WU9Kd29RM0Q2SVFBRTh4YXB6NTBlSXNpZ1BRYXE2TG5h?= =?utf-8?B?VzZlNWxHVFFTUUNaK0UxZS9QRkZuc3N5WkVLaCtuN2FrSHpCOWJmOURMbUlK?= =?utf-8?B?b1cvREhSbTQyWGpQRHB3eXdXSTVmSlZBYk9jMnRuMlpUTDNhSHVxRG12MjVh?= =?utf-8?B?SExteXBBdWltUjR2VTR0LzRncGRDZStoam1pYnNZOFhxazd0U1dLUHhSME9v?= =?utf-8?B?SHo5eG1wZUdaMG43V2Q3K3BGQ2hVaGlHcjVjQTlvMzNudDhiVlE1SzRvU2pE?= =?utf-8?B?ZUpraTF3ejVkZmhwZDJXSVJTeXZ4VlZJczVIbFU4OU9SdlgrMmc2SXgyT0th?= =?utf-8?B?MmxvMmJVY3NqU2hzbkxJL0Z4OEh2dGVaTEdTSytjclc4eFZlR3FrMTByZWpz?= =?utf-8?B?N2EzQTlTTDczRGJSMDA3VkpaY3diSGVUdVJPQlVURWpaQTFQbWF2RVZINEtU?= =?utf-8?B?OGFLbkYwbUJyQVdmRktFSzlmSEtmLy8vZnIxb1hEWUw3Y1ZocXZNdjZGSmpP?= =?utf-8?B?SHA1Q253STR3WjF6YmlvUy9NNmNoT3oyUzgwWTdlVkcySWxqRTgvT0RTbWZF?= =?utf-8?B?UDNIdFROU3FLNWRaNU5palE2K2l1MHRwUmVTajRkcnBBWUFoTHJHY09FaEo3?= =?utf-8?B?UTNjRDJnMFFoNmZvOFdmQ2NkOUxIRVB5SmJEdVlJZzEyY3JEa1huS3hxSC91?= =?utf-8?B?UkVrTm9HZ3lvdTg4NlVhRmdraHhRVzlNZTYxR2VIUjhEZjdxdWd4Sk40VXhs?= =?utf-8?B?enA2cWtiWHFvZlc2M1g2a0ovMHBOT1hzK2dhbGJrUUJRKzE5aFd4YkM4Wi9G?= =?utf-8?B?UUhLbzhUalkxQ2FnMXg2OWtINHpydzl5WlhLeDZVMkxZN0hVbEszNzhJaUZ1?= =?utf-8?B?VHJ3WjZGUmc4SkE1dkVxWlN3a1VqT2JYY1pwQ1p1eklmcHJFMnVxOUhqQTBo?= =?utf-8?B?N05MWVJaRG5Ea2xGQjFYSllHc2pJdHYzaGJoSlh2UzdzazRucGUzejBBZDBW?= =?utf-8?B?UU5kVy83S2ZpVGxYeE1sQXVGZXU4d3IzbEZ1NlF3UGp5Ti9ZakpKUHg4eU5Z?= =?utf-8?B?YUY4ekhYVXQzZEoveUxXRU1LRWN0YXRBSEFuVUExMzI4QllmQ2N6MVFVS0k3?= =?utf-8?B?OXgwOXN3NmpMYTRzM3VEbDlaOHVhOU1vWGJpSlpaZDNJbnRxWWtzTWpYa3p6?= =?utf-8?B?dVcrRDRWd0hBNkwrRzVlaTlPUXR5N09QUUE5N2gwVlpCYmd0QldFdFlvK01t?= =?utf-8?B?eXg1U0NzaVFvNDkzYjlWWldqcmc0aU1VT25NeUtVS2lUWTNIMm9TdzFHTmRV?= =?utf-8?B?ZGRieTFMSjlLU0xsMEI2ellFOTMwazJaM3J3M2ZFU1FPS3MxUHFsbE5PTTM1?= =?utf-8?B?WWpWQXhONjdUQmpmaDBBZEt6cGQvbnRhaUhYOFVFdDVCVGZSdHY3VnVQWEtL?= =?utf-8?B?YWxxVHhXcjV2OWRkTEpHRWljQ2dobXBhaHBrQlFOdE96OVpBMjBxRTBuVUpU?= =?utf-8?B?emVGMkYrSFhRK0FwNUE4TmVUV0RtU0ljY3hOR1M4Kzl6aWs5YXpxNjFBYitW?= =?utf-8?B?MXd0U1lwOHhycWQyYlVzaDRKNm1rUnM3YVZaTjJnVEtsRm9oenB5LzFCQUlt?= =?utf-8?Q?9IbNqGUzdg/6y6arEm0MyiwJtWTEsEnxbpAD4M=3D?= X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11HUB040;5:zLrC6X2VtL4Y7cFGgfa2YUbaiAWwujbNppFSxtw7T4F7Y91ZTnKt6RDW6pmrWbrVqCc1gXVMh10ovpWZQsM5jhGZs/AsaryStvp/MffYFyUoE1ggwU/K/f2uyZMuLtDDVBk9hsTgsy8FcfWt+9wSTQ==;24:FIel1eGwA3QFUIa0/Y9Y00H3TGSCaAaw1P2QR+ustAkx7fmOWhllmzdKGAk37fB0FMJ/k0fi2V7Sm9xYaxK+x3HoxBMTiQ6/AmehNAeWYh0= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Sep 2015 02:47:07.4513 (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.100];Helo=[xsj-pvapsmtpgw02] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2FFO11HUB040 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Shawn, On Fri, 2015-09-11 at 04:55PM +0800, Shawn Lin wrote: > This patch adds phys and phy-names for sdhci-of-arasan as optional > properties, and details the example as well. > > Signed-off-by: Shawn Lin > --- > > Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > index da541c3..0264d5f 100644 > --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > @@ -1,11 +1,12 @@ > Device Tree Bindings for the Arasan SDHCI Controller > > - The bindings follow the mmc[1], clock[2] and interrupt[3] bindings. Only > - deviations are documented here. > + The bindings follow the mmc[1], clock[2], interrupt[3] and phy[4] bindings. > + Only deviations are documented here. > > [1] Documentation/devicetree/bindings/mmc/mmc.txt > [2] Documentation/devicetree/bindings/clock/clock-bindings.txt > [3] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt > + [4] Documentation/devicetree/bindings/phy/phy-bindings.txt > > Required Properties: > - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' or > @@ -16,6 +17,9 @@ Required Properties: > - interrupts: Interrupt specifier > - interrupt-parent: Phandle for the interrupt controller that services > interrupts for this device. > +Optional Properties: > + - phys: From PHY bindings: Phandle for the Generic PHY for arasan. > + - phy-names: MUST be "phy_arasan". This might be a dumb question, but, is the external phy actually optional? Or is it mandatory for certain implementations of the IP. In the latter case, it should probably be a mandatory property depending on the compatible string matching an implementation that requires an external phy (accordingly, the implementation might have to treat it that way too). Sören