From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 10 Jan 2008 10:42:16 +1100 From: David Gibson To: Grant Likely Subject: Re: [PATCH 1/3] add default device trees for MPC837x MDS board Message-ID: <20080109234215.GA17153@localhost.localdomain> References: <1199707400-23588-1-git-send-email-leoli@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Cc: linuxppc-dev@ozlabs.org, Li Yang , paulus@samba.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, Jan 08, 2008 at 08:58:17AM -0700, Grant Likely wrote: > Hi Leo, > > comments below. I've made my comments on the first file, but they > apply to the other two also. > > Cheers, > g. [snip] > > + ranges = <0 e0000000 00100000>; > > + reg = ; > > + bus-frequency = <0>; > > + > > + wdt@200 { > > + compatible = "mpc83xx_wdt"; > > "fsl,mpc8377_wdt", "fsl,mpc83xx_wdt" as per generic names recommended practice. Uh.. that's reasonable advice, but it has nothing to do with the generic names recommended practice. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson