On Fri, 2012-08-24 at 23:34 -0500, tom.zanu...@intel.com wrote: > This is the ia32-base.inc moved over from meta-intel. See meta-intel > for the complete history of contributions to this file. > > Here's the initial commit text that explains the purpose of this file: > > The meta-intel BSPs currently have a number of machine settings common > to all - factor these out into a common include file. >
It doesn't look as though the name "ia32-base" really reflects the contents of this file very accurately. Most of the bits in here seem to have more to do with generic PC architecture and would equally apply to x86-64 (or indeed any architecture -- e.g. "ext2" which doesn't really seem to be a machine property in any meaningful sense). Also, a lot of the stuff in this file seems rather more like distro policy than architecture/machine definition. That's not to say that I think the idea of having an include file to capture a "PC-type" platform (or, at least, the common use-cases for one) is a bad idea. If there are multiple platforms which want to do that then putting the definitions in a common place does seem sensible, but I think describing it as "ia32-base" is at the same time too specific and too generic. > Also add several new intel-specific XSERVER variables for building > XSERVER variables in BSPs. If these are really intel-specific (and the i915/i965 bits do look like they are), shouldn't they stay in meta-intel? It seems like having variables defined in oe-core, but not referenced by anything in oe-core, is possibly a recipe for future confusion. p. _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
