[lkml]   [2012]   [Jan]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[PATCH] kbuild: Fix comment in Makefile.lib
    KBUILD_MODNAME is not defined for files that are linked into multiple
    modules, and trying to change reality to match documentation would
    result in all sorts of trouble. E.g. options for built-in modules would
    be called either foo_bar.param, foo.param, or bar.param, depending on
    the configuration. So just change the comment.

    Signed-off-by: Michal Marek <>
    scripts/Makefile.lib | 6 +++---
    1 files changed, 3 insertions(+), 3 deletions(-)

    diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib
    index 5d986d9..2ce2e21 100644
    --- a/scripts/Makefile.lib
    +++ b/scripts/Makefile.lib
    @@ -93,9 +93,9 @@ obj-dirs := $(addprefix $(obj)/,$(obj-dirs))
    # already
    # $(modname_flags) #defines KBUILD_MODNAME as the name of the module it will
    # end up in (or would, if it gets compiled in)
    -# Note: It's possible that one object gets potentially linked into more
    -# than one module. In that case KBUILD_MODNAME will be set to foo_bar,
    -# where foo and bar are the name of the modules.
    +# Note: Files that end up in two or more modules are compiled without the
    +# KBUILD_MODNAME definition. The reason is that any made-up name would
    +# differ in different configs.
    name-fix = $(subst $(comma),_,$(subst -,_,$1))
    basename_flags = -D"KBUILD_BASENAME=KBUILD_STR($(call name-fix,$(basetarget)))"
    modname_flags = $(if $(filter 1,$(words $(modname))),\

     \ /
      Last update: 2012-01-08 22:19    [W:0.017 / U:8.800 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site