Hi David, Thanks for the comments.
On Thu, 2009-06-11 at 11:32 +1000, David Gibson wrote: > These last two aren't standard properties, so should probably be > "xes,form-factor" and "xes,boot-bank". I'll just delete them for now since they're not critical. > > + pmcslots { > > What does this structure model? Without any reg properties it's kind > of hard to see what you could do with it. This would be a bit more obvious to those familiar with CompactPCI and PMC modules. It aims to describe how many PMC slots are in the system, and gives information on whether or not a module is installed and if it's monarch of the bus or not. So it's really just a description of the hardware. The properties themselves are useful because you may have to read from GPIO lines to get this information (which may be accessible only via I2C, which can't be done too early in the kernel). A reg property could be used just for indexing the slots, but there are no real memory-mapped resources involved for the slot itself (that information would go in the PCI nodes). It'd be nice if we could get a device tree standard for this type of information in the future, but for now I can just delete it since we don't yet rely on it in the kernel. > > + > > + tbi2: tbi-...@11 { > > + reg = <0x11>; > > + device_type = "tbi-phy"; > > And this one, too. Although this node should probably have a > compatible property instead. I don't think I can drop that device_type yet. All the other device trees for Freescale boards use it, and the Freescale MDIO driver appears to rely on it. A future patch I suppose? > Analagous comments for the other device trees. Thanks again. I'll fix up the other things you and Kumar mentioned and re-submit today. -- Nate Case <nc...@xes-inc.com> _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev