From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754596AbcB2WiN (ORCPT ); Mon, 29 Feb 2016 17:38:13 -0500 Received: from host.buserror.net ([209.198.135.123]:38637 "EHLO host.buserror.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750910AbcB2WiL (ORCPT ); Mon, 29 Feb 2016 17:38:11 -0500 Message-ID: <1456785480.5360.60.camel@buserror.net> From: Scott Wood To: Alessio Igor Bogani , Kumar Gala Cc: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Date: Mon, 29 Feb 2016 16:38:00 -0600 In-Reply-To: <1456397825-1786-2-git-send-email-alessio.bogani@elettra.eu> References: <1456397825-1786-1-git-send-email-alessio.bogani@elettra.eu> <1456397825-1786-2-git-send-email-alessio.bogani@elettra.eu> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.16.5-1ubuntu3.1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 75.72.173.242 X-SA-Exim-Mail-From: oss@buserror.net X-Spam-Report: * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP * -15 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] Subject: Re: [RFC PATCH v1 2/2] powerpc/86xx: Introduce and use common dtsi X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:57:07 +0000) X-SA-Exim-Scanned: Yes (on host.buserror.net) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2016-02-25 at 11:57 +0100, Alessio Igor Bogani wrote: > Signed-off-by: Alessio Igor Bogani > --- > arch/powerpc/boot/dts/fsl/gef_ppc9a.dts | 284 +++------------------- > arch/powerpc/boot/dts/fsl/gef_sbc310.dts | 276 +++------------------ > arch/powerpc/boot/dts/fsl/gef_sbc610.dts | 284 +++------------------- > arch/powerpc/boot/dts/fsl/mpc8641_hpcn.dts | 318 ++++------------------ > --- > arch/powerpc/boot/dts/fsl/mpc8641_hpcn_36b.dts | 317 ++++------------------ > -- > arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi | 124 ++++++++++ > arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi | 59 +++++ > arch/powerpc/boot/dts/fsl/sbc8641d.dts | 314 +++------------------- > -- > 8 files changed, 400 insertions(+), 1576 deletions(-) > create mode 100644 arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi > create mode 100644 arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi Could you post a diff of what the decompiled trees look like before and after this change (e.g. interrupts went from 2-cell to 4-cell)? It is very hard to review in this form. Or better, like the config change, have a commit that first makes changes to what the unified trees will be, and then a second commit that generates the same output using includes. > +/include/ "qoriq-mpic.dtsi" > + pic@40000 { > + compatible = "chrp,open-pic"; > + }; This is removing the fsl,mpic compatible, which is required for 4-cell interrupt specifiers. > +&pci0 { > + compatible = "fsl,mpc8641-pcie"; > + device_type = "pci"; > + #size-cells = <2>; > + #address-cells = <3>; > + bus-range = <0x0 0xff>; > + clock-frequency = <33333333>; The clock frequency of PCI Express is not 33 MHz. > diff --git a/arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi > b/arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi > new file mode 100644 > index 0000000..026fa58 > --- /dev/null > +++ b/arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi > @@ -0,0 +1,59 @@ > +/* > + * MPC8641 Silicon/SoC Device Tree Source (pre include) > + * > + * Copyright 2016 Elettra-Sincrotrone Trieste S.C.p.A. > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or (at your > + * option) any later version. > + * > + */ > + > +/dts-v1/; > + > +/ { > + compatible = "fsl,MPC8641"; This compatible is pointless -- it will be overwritten by the board compatible. -Scott