From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from de01egw02.freescale.net (de01egw02.freescale.net [192.88.165.103]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "de01egw02.freescale.net", Issuer "Thawte Premium Server CA" (verified OK)) by ozlabs.org (Postfix) with ESMTP id 422F6DDEEE for ; Fri, 18 May 2007 04:48:51 +1000 (EST) Message-ID: <464CA380.2020705@freescale.com> Date: Thu, 17 May 2007 13:48:32 -0500 From: Timur Tabi MIME-Version: 1.0 To: Timur Tabi Subject: Re: [PATCH 5/5] PCI fixes for the MPC8641 Rev 2.0 silicon and Rev 1.02hardware References: <1179245829.8132.100.camel@rhino> <1179247809.8132.138.camel@rhino> <46B96294322F7D458F9648B60E15112C23441B@zch01exm26.fsl.freescale.net> <1179417813.8132.250.camel@rhino> <744CD970-5421-47D6-A30A-C7C79BE21BE8@kernel.crashing.org> <1179421139.8132.256.camel@rhino> <464CA302.9060707@freescale.com> In-Reply-To: <464CA302.9060707@freescale.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Cc: linuxppc-dev , Zhang Wei-r63237 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Timur Tabi wrote: > I think two DTS files is the best approach for now. A few of us had an > idea to introduce conditional statements in to the DTS, and U-Boot would > examine the board and/or environment variables and then apply the > conditions to the device tree before booting the kernel. This would > allow you to merge the two DTS files into one, but we're quite a ways > off from implementing this feature. In the meantime, two DTS files is > okay. One more thing - I think each DTS file should specify which revision of the board it supports, and there should be code in U-Boot and/or Linux to check that revision and reject the device tree if there's a mismatch. We don't want people accidentally using the wrong device tree and wondering why it doesn't work. -- Timur Tabi Linux Kernel Developer @ Freescale