[oe] [PATCH] rename SOC_FAMILY to TARGET_SOC_FAMILY and make it global

[Frans Meulenbroeks]

2010/9/10 Denys Dmytriyenko <denis at denix.org>:
:>
> Frans,
>
> I guess then you haven't read the original discussion closely enough. Chase
> only made a change to add existing SOC_FAMILY override into COMPATIBLE_MACHINE
> variable... I keep repeating myself - SOC_FAMILY feature is about a year old.
> It was introduced as part of a large merge from Arago. Here is the original
> commit and some description:
>
> http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=b788499815bfc1974d4277c0a261b9a7c24aa816

I did some more research on this, as I could not recall any specific info.
The patch you give above is from dec, so 9 months old.
At that time it was not discussed at the mailing list as far as I can
see. As the commit message said "recipes/ti: merge in updates"
I (and I guess most of the non-ti devs) gave little attention to it.

Also I did a quick grep on the source tree. Currently it is only used
in the following files

frans at linux-suse:~/oe/openembedded> grep -r SOC_FAMILY .
./conf/local.conf:SOC_FAMILY_local = "none"
./conf/distro/include/angstrom.inc:SOC_FAMILY ?= "Unknown"
./conf/distro/include/angstrom.inc:# Add FEED_ARCH and SOC_FAMILY to overrides
./conf/distro/include/angstrom.inc:OVERRIDES .= ":${FEED_ARCH}:${SOC_FAMILY}"
./conf/machine/include/dm6467.inc:SOC_FAMILY = "dm6467"
./conf/machine/include/omap3.inc:SOC_FAMILY = "omap3"
./conf/machine/include/omapl137.inc:SOC_FAMILY = "omapl137"
./conf/machine/include/dm365.inc:SOC_FAMILY = "dm365"
./conf/machine/include/dm6446.inc:SOC_FAMILY = "dm6446"
./conf/machine/include/dm355.inc:SOC_FAMILY = "dm355"
./conf/machine/include/omapl138.inc:SOC_FAMILY = "omapl138"
./recipes/ti/ti-linuxutils.inc:    case ${SOC_FAMILY} in
./recipes/ti/ti-linuxutils.inc:    if [ ${SOC_FAMILY} == "dm365" ] ; then
./recipes/ti/ti-local-power-manager.inc:# SOC_FAMILY configuration
./recipes/ti/ti-codec-engine.inc:# SOC_FAMILY configuration
./recipes/ti/ti-dsplink.inc:# SOC_FAMILY configuration
./classes/base.bbclass:                this_soc_family =
bb.data.getVar('SOC_FAMILY', d, 1)

Anyway, wrt the breakage, I'll push the patch for now as it has been
out for review quite a while (and I got an ack and a looks good).
If SOC_FAMILY is not desired we can also revert that one.

Frans