From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH 15/16] Add device tree for Ebony From: Benjamin Herrenschmidt To: Segher Boessenkool In-Reply-To: <692d02b69479653f689a50a1257f43e9@kernel.crashing.org> References: <20070213061026.5837FDDDE9@ozlabs.org> <20070214002210.GE11491@localhost.localdomain> <45afe653a3f963e21e58a063c09b1b22@kernel.crashing.org> <1171488643.20192.177.camel@localhost.localdomain> <7fa77edce7aeb8a41d03b8b422f7f71b@kernel.crashing.org> <1171500782.20192.204.camel@localhost.localdomain> <692d02b69479653f689a50a1257f43e9@kernel.crashing.org> Content-Type: text/plain Date: Thu, 15 Feb 2007 13:22:43 +1100 Message-Id: <1171506164.20192.228.camel@localhost.localdomain> Mime-Version: 1.0 Cc: linuxppc-dev@ozlabs.org, David Gibson List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 2007-02-15 at 02:00 +0100, Segher Boessenkool wrote: > >>> Note that the "open-pic" node does not have a "#address-cells" > >>> property, so that the number of cells for the parent unit interrupt > >>> specifiers is 2 (which is the value of its "#interrupt-cells" > >>> property). > > > Yes, "missing" is equivalent to 0, > > ...and the reason for that is that nodes without physical > children should not _have_ a "#address-cells" property. > > > but the common practice has always > > been to specify it explicitely. Check existing Apple and IBM > > device-trees for example. > > You do realise how many other borderline (or outright > wrong) things are done in those trees, right? > > It doesn't really hurt to have a "#a = 0" prop, but it's > better to just not have a #a prop if you shouldn't have one. Let's have one for now. I think that's how I documented it and it works fine while I'm not sure the kernel code will cope with not having it (it may ... or not). There are various issues with the OF bindings, especially the imap one (and some pretty bad bugs in it) and that's one of the reason I prefer not leaving anything to be "implicit" like this case. It want explicit mention of #address-cells/#size-cells/#interrupt-cells at all levels where they might enter the parser. Ben.