Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753616Ab2KIO1i (ORCPT ); Fri, 9 Nov 2012 09:27:38 -0500 Received: from ozlabs.org ([203.10.76.45]:49388 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752851Ab2KIO1h (ORCPT ); Fri, 9 Nov 2012 09:27:37 -0500 Date: Sat, 10 Nov 2012 01:29:08 +1100 From: David Gibson To: Joel A Fernandes Cc: Pantelis Antoniou , Kevin Hilman , Matt Porter , Koen Kooi , linux-kernel , Felipe Balbi , Deepak Saxena , Scott Wood , Russ Dill , linux-omap@vger.kernel.org, devicetree-discuss@lists.ozlabs.org Subject: Re: [RFC] Device Tree Overlays Proposal (Was Re: capebus moving omap_devices to mach-omap2) Message-ID: <20121109142908.GB4696@truffula.fritz.box> References: <02FF5400-9F97-4B8A-AEF0-267B01C8099F@antoniou-consulting.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3575 Lines: 81 On Fri, Nov 09, 2012 at 12:32:09AM -0500, Joel A Fernandes wrote: > Hi Pantelis, > > I hope I'm not too late to reply as I'm traveling. > > On Nov 6, 2012, at 5:30 AM, Pantelis Antoniou > wrote: > > >> Joanne has purchased one of Jane's capes and packaged it into a rugged > >> case for data logging. As far as Joanne is concerned, the BeagleBone and > >> cape together are a single unit and she'd prefer a single monolithic FDT > >> instead of using an FDT overlay. > >> Option A: Using dtc, she uses the BeagleBone and cape .dts source files > >> to generate a single .dtb for the entire system which is > >> loaded by U-Boot. -or- > > > > Unlikely. > >> Option B: Joanne uses a tool to merge the BeagleBone and cape .dtb files > >> (instead of .dts files), -or- > > Possible but low probability. > > > >> Option C: U-Boot loads both the base and overlay FDT files, merges them, > >> and passes the resolved tree to the kernel. > >> > > > > Could be made to work. Only really required if Joanne wants the > > cape interface to work for u-boot too. For example if the cape has some > > kind of network interface that u-boot will use to boot from. > > > > I love Grant's hashing idea a lot keeping the phandle problem for > compile time and not requiring fixups. Well, using a hash only moves the problem of fixed phandles to a problem of fixed node paths. The details of node paths are, if anything, more mutable than phandles. [snip] > Alternatively to hashing, reading David Gibson's paper I followed, > phandle is supposed to 'uniquely' identity node. I wonder why the node > name itself is not sufficient to uniquely identify. Node names are not unique, not even close. If you have two similar NICs in slot 0 of two different PCI domains, they'll almost certainly both be called 'ethernet@0,0'. Similar examples abound on other buses. Node paths are unique, but they are long. The other big reason for phandles in OF history is that they would be more stable than paths. The device tree could be manipulated during OF runtime, but phandles would generally be internal pointers in OF and so remain a consistent handle even if the node moved in the tree. That's not really relevant for flat trees, but we need to work with the same structures. > The code that does > the tree walking can then just strcmp the node name while it walks the > tree instead of having to find a node with a phandle number. I guess > the reason is phandles are small to store as data values. Another > approach can be to arrange the string block in alphabetical order > (unless it already is), They're not, and doing so would be a painful change to maintain compatibility across. And in any case only property names use the strings block, not node names. > and store phandle as index of the node name > referenced relative to the starting of the strong block. This will not > affect nodes in dtb being moved around since they will still have the > same index value. the problem being adding or removing nodes Changes > the index of all other nodes in the string block as well.. Hmm. -- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/