[U-Boot] a bunch of redundant "#define CONFIG_SH 1" lines?

Robert P. J. Day rpjday at crashcourse.ca
Sat Feb 2 00:48:34 CET 2013


  notice the following, in which a number of config files for
SH-related boards explicitly define the macro "CONFIG_SH":

$ grep -rw CONFIG_SH *
common/cmd_bdinfo.c:#elif defined(CONFIG_SH)
drivers/bios_emulator/include/x86emu.h:#elif defined (CONFIG_SH)
drivers/net/sh_eth.h:#if defined(CONFIG_SH)
drivers/net/sh_eth.h:#endif /* defined(CONFIG_SH) */
examples/standalone/stubs.c:#elif defined(CONFIG_SH)
include/configs/MigoR.h:#define CONFIG_SH		1
include/configs/ms7722se.h:#define CONFIG_SH		1
include/configs/r0p7734.h:#define CONFIG_SH		1
include/configs/rsk7269.h:#define CONFIG_SH		1
include/configs/r2dplus.h:#define CONFIG_SH		1
include/configs/sh7785lcr.h:#define CONFIG_SH		1
include/configs/ms7720se.h:#define CONFIG_SH		1
include/configs/sh7752evb.h:#define CONFIG_SH		1
include/configs/ms7750se.h:#define CONFIG_SH		1
include/configs/shmin.h:#define CONFIG_SH		1
include/configs/sh7763rdp.h:#define CONFIG_SH		1
include/configs/ap325rxa.h:#define CONFIG_SH		1
include/configs/sh7757lcr.h:#define CONFIG_SH		1
include/configs/rsk7264.h:#define CONFIG_SH		1
include/configs/mpr2.h:#define CONFIG_SH		1
include/configs/ap_sh4a_4a.h:#define CONFIG_SH		1
include/configs/espt.h:#define CONFIG_SH		1
include/configs/r7780mp.h:#define CONFIG_SH		1
include/configs/ecovec.h:#define CONFIG_SH		1
include/configs/rsk7203.h:#define CONFIG_SH		1
$

  which is fine -- it doesn't hurt -- but aren't all those define's
superfluous since that macro will be defined via this:

  arch/sh/config.mk:PLATFORM_CPPFLAGS += -DCONFIG_SH -D__SH__
                                         ^^^^^^^^^^^

  not suggesting they deserve immediate removal, just want to verify
that my understanding is correct and that they're unnecessary.
thanks.

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================


More information about the U-Boot mailing list