Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932239AbcCRSQO (ORCPT ); Fri, 18 Mar 2016 14:16:14 -0400 Received: from mail-am1on0080.outbound.protection.outlook.com ([157.56.112.80]:22414 "EHLO emea01-am1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750811AbcCRSQL convert rfc822-to-8bit (ORCPT ); Fri, 18 Mar 2016 14:16:11 -0400 From: Scott Wood To: Rob Herring , Yangbo Lu CC: "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linuxppc-dev@lists.ozlabs.org" , "linux-clk@vger.kernel.org" , "linux-i2c@vger.kernel.org" , "iommu@lists.linux-foundation.org" , "netdev@vger.kernel.org" , "linux-mmc@vger.kernel.org" , Russell King , "Jochen Friedrich" , Joerg Roedel , Claudiu Manoil , "ulf.hansson@linaro.org" , Bhupesh Sharma , "Zhao Qiang" , Kumar Gala , "Santosh Shilimkar" , Yang-Leo Li , Xiaobo Xie Subject: Re: [v6, 3/5] dt: move guts devicetree doc out of powerpc directory Thread-Topic: [v6, 3/5] dt: move guts devicetree doc out of powerpc directory Thread-Index: AQHRee0M0SKIxUGJDUePEtCYXtWyoQ== Date: Fri, 18 Mar 2016 18:16:06 +0000 Message-ID: References: <1457518131-11339-1-git-send-email-yangbo.lu@nxp.com> <1457518131-11339-4-git-send-email-yangbo.lu@nxp.com> <20160317170640.GB21009@rob-hp-laptop> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: kernel.org; dkim=none (message not signed) header.d=none;kernel.org; dmarc=none action=none header.from=nxp.com; x-originating-ip: [2601:448:8100:702f:12bf:48ff:fe84:c9a0] x-ms-office365-filtering-correlation-id: f77ef0c5-5a55-4823-9dc4-08d34f595f9a x-microsoft-exchange-diagnostics: 1;DB5PR04MB0887;5:zKU3Tq9xrjR++BST6bQQc58jnfKv59rqucwiuy+Bs0wqRtAvs3szYfYmsquS3DrPmT2kV8s3rnuNem56I90noiAGM1/RssH4vyqFHCOp4bDeJZLfFnAwo2s8/BTVUnd9PC8n5vN+XfnaYBHOlUhmoQ==;24:IaUWe54Hirrh22K7uMW+3/AnricQCyInBWpOlUISboEQfAzBKITm9cJka51i979YiCUeOq2lnRFiH/khO27KGr9jEoGmPA6jPIyomVow+XM= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB5PR04MB0887; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001);SRVR:DB5PR04MB0887;BCL:0;PCL:0;RULEID:;SRVR:DB5PR04MB0887; x-forefront-prvs: 088552DE73 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(377454003)(24454002)(5001770100001)(19580405001)(19580395003)(5004730100002)(1220700001)(1096002)(3280700002)(2906002)(6116002)(4326007)(586003)(76576001)(3660700001)(189998001)(102836003)(122556002)(92566002)(74316001)(5008740100001)(50986999)(2900100001)(87936001)(33656002)(5003600100002)(5002640100001)(11100500001)(77096005)(81166005)(86362001)(106116001)(54356999)(76176999)(10400500002)(7059030)(3826002);DIR:OUT;SFP:1101;SCL:1;SRVR:DB5PR04MB0887;H:HE1PR0401MB1931.eurprd04.prod.outlook.com;FPR:;SPF:None;MLV:sfv;LANG:en; spamdiagnosticoutput: 1:23 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Mar 2016 18:16:06.1992 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB5PR04MB0887 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2376 Lines: 61 On 03/17/2016 12:06 PM, Rob Herring wrote: > On Wed, Mar 09, 2016 at 06:08:49PM +0800, Yangbo Lu wrote: >> Move guts devicetree doc to Documentation/devicetree/bindings/soc/fsl/ >> since it's used by not only PowerPC but also ARM. And add a specification >> for 'little-endian' property. >> >> Signed-off-by: Yangbo Lu >> --- >> Changes for v2: >> - None >> Changes for v3: >> - None >> Changes for v4: >> - Added this patch >> Changes for v5: >> - Modified the description for little-endian property >> Changes for v6: >> - None >> --- >> Documentation/devicetree/bindings/{powerpc => soc}/fsl/guts.txt | 3 +++ >> 1 file changed, 3 insertions(+) >> rename Documentation/devicetree/bindings/{powerpc => soc}/fsl/guts.txt (91%) >> >> diff --git a/Documentation/devicetree/bindings/powerpc/fsl/guts.txt b/Documentation/devicetree/bindings/soc/fsl/guts.txt >> similarity index 91% >> rename from Documentation/devicetree/bindings/powerpc/fsl/guts.txt >> rename to Documentation/devicetree/bindings/soc/fsl/guts.txt >> index b71b203..07adca9 100644 >> --- a/Documentation/devicetree/bindings/powerpc/fsl/guts.txt >> +++ b/Documentation/devicetree/bindings/soc/fsl/guts.txt >> @@ -25,6 +25,9 @@ Recommended properties: >> - fsl,liodn-bits : Indicates the number of defined bits in the LIODN >> registers, for those SOCs that have a PAMU device. >> >> + - little-endian : Indicates that the global utilities block is little >> + endian. The default is big endian. > > The default is "the native endianness of the system". So absence on an > ARM system would be LE. No. For this binding, the default is big-endian, because that's what existed for this device before an endian property was added. "endianness of the system" is not a well-defined concept. > This property is valid for any simple-bus device, Since when does simple-bus mean anything more than that the nodes underneath it can be used without bus-specific knowledge? > so it isn't really required to document per device. You can, but > your description had better match the documented behaviour. Documented where? In fact, Documentation/devicetree/bindings/common-properties.txt explicitly says of the endian properties, "If a binding supports these properties, then the binding should also specify the default behavior if none of these properties are present." -Scott