From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757189Ab2ADWY2 (ORCPT ); Wed, 4 Jan 2012 17:24:28 -0500 Received: from oproxy4-pub.bluehost.com ([69.89.21.11]:45760 "HELO oproxy4-pub.bluehost.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1757115Ab2ADWYZ (ORCPT ); Wed, 4 Jan 2012 17:24:25 -0500 Message-ID: <4F04DF50.4070201@xenotime.net> Date: Wed, 04 Jan 2012 15:22:56 -0800 From: Randy Dunlap Organization: YPO4 User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.15) Gecko/20110323 Thunderbird/3.1.9 MIME-Version: 1.0 To: Kyle Moffett CC: linuxppc-dev@lists.ozlabs.org, Grant Likely , Rob Herring , Benjamin Herrenschmidt , Paul Mackerras , Kumar Gala , Lucas De Marchi , Stuart Yoder , Josh Boyer , devicetree-discuss@lists.ozlabs.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] fsl/mpic: Document and use the "big-endian" device-tree flag References: <1324571123-11577-1-git-send-email-Kyle.D.Moffett@boeing.com> In-Reply-To: <1324571123-11577-1-git-send-email-Kyle.D.Moffett@boeing.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Identified-User: {1807:box742.bluehost.com:xenotime:xenotime.net} {sentby:smtp auth 50.53.38.135 authed with rdunlap@xenotime.net} Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/22/2011 08:25 AM, Kyle Moffett wrote: > The MPIC code checks for a "big-endian" property and sets the flag > MPIC_BIG_ENDIAN if one is present. Unfortunately, the PowerQUICC-III > compatible device-tree does not specify it, so all of the board ports > need to manually set that flag when calling mpic_alloc(). > > Document the flag and add it to the pq3 device tree. Existing code > will still need to pass the MPIC_BIG_ENDIAN flag because their dtb may > not have this property, but new platforms shouldn't need to do so. > > Signed-off-by: Kyle Moffett Grant, are you merging this patch? I don't think I should merge the patch to arch/powerpc/boot/dts/fsl/pq3-mpic.dtsi. > --- > .../devicetree/bindings/powerpc/fsl/mpic.txt | 9 ++++++++- > arch/powerpc/boot/dts/fsl/pq3-mpic.dtsi | 1 + > 2 files changed, 9 insertions(+), 1 deletions(-) > > diff --git a/Documentation/devicetree/bindings/powerpc/fsl/mpic.txt b/Documentation/devicetree/bindings/powerpc/fsl/mpic.txt > index 2cf38bd..ebafba2 100644 > --- a/Documentation/devicetree/bindings/powerpc/fsl/mpic.txt > +++ b/Documentation/devicetree/bindings/powerpc/fsl/mpic.txt > @@ -56,7 +56,14 @@ PROPERTIES > to the client. The presence of this property also mandates > that any initialization related to interrupt sources shall > be limited to sources explicitly referenced in the device tree. > - > + > + - big-endian > + Usage: optional > + Value type: > + If present the MPIC will be assumed to be big-endian. Some > + device-trees omit this property on MPIC nodes even when the MPIC is > + in fact big-endian, so certain boards override this property. > + > INTERRUPT SPECIFIER DEFINITION > > Interrupt specifiers consists of 4 cells encoded as > diff --git a/arch/powerpc/boot/dts/fsl/pq3-mpic.dtsi b/arch/powerpc/boot/dts/fsl/pq3-mpic.dtsi > index 5c80460..47f2b67 100644 > --- a/arch/powerpc/boot/dts/fsl/pq3-mpic.dtsi > +++ b/arch/powerpc/boot/dts/fsl/pq3-mpic.dtsi > @@ -39,6 +39,7 @@ mpic: pic@40000 { > reg = <0x40000 0x40000>; > compatible = "fsl,mpic"; > device_type = "open-pic"; > + big-endian; > }; > > timer@41100 { -- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***