From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: To: David Gibson Subject: Re: [PATCH 2/4 v4] POWERPC: Add StorCenter DTS first draft. In-Reply-To: Your message of "Fri, 01 Feb 2008 18:50:33 +1100." <20080201075033.GB18684@localhost.localdomain> References: <20080201075033.GB18684@localhost.localdomain> Date: Fri, 01 Feb 2008 08:34:54 -0600 From: Jon Loeliger Message-Id: Cc: linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , So, like, the other day David Gibson mumbled: > > +/ { > > + model = "StorCenter"; > > + compatible = "storcenter"; > > This really needs a vendor prefix. Hey Grant! Here's how dumb I am: You were thinking "Something's wrong here..." and kept saying "Needs a model name!", and I kept saying "That _is_ the model name!" And finally Dave comes along and with the ever-obvious "Needs a vendor prefix!" I'm like, "D'oh!" > > + chosen { > > + linux,stdout-path = "/soc/serial@4500"; > > You can now use a path reference here. Ah, good point. I'll get around to an update patch! Thanks! jdl