- These days if_changed is used with many more commands than ld,
    objcopy, and gzip, hence add an ellipsis,
  - Any target that utilises if_changed must be listed in $(targets),
    so it needs an assignment to "targets", not "target".

Signed-off-by: Geert Uytterhoeven <[email protected]>
---
 Documentation/kbuild/makefiles.txt | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/kbuild/makefiles.txt 
b/Documentation/kbuild/makefiles.txt
index 3b416592cc44..1c6056f1b95f 100644
--- a/Documentation/kbuild/makefiles.txt
+++ b/Documentation/kbuild/makefiles.txt
@@ -1115,7 +1115,7 @@ When kbuild executes, the following steps are followed 
(roughly):
 
        Usage:
                target: source(s) FORCE
-                       $(call if_changed,ld/objcopy/gzip)
+                       $(call if_changed,ld/objcopy/gzip/...)
 
        When the rule is evaluated, it is checked to see if any files
        need an update, or the command line has changed since the last
@@ -1133,7 +1133,7 @@ When kbuild executes, the following steps are followed 
(roughly):
        significant; for instance, the below will fail (note the extra space
        after the comma):
                target: source(s) FORCE
-       #WRONG!#        $(call if_changed, ld/objcopy/gzip)
+       #WRONG!#        $(call if_changed, ld/objcopy/gzip/...)
 
     ld
        Link target. Often, LDFLAGS_$@ is used to set specific options to ld.
@@ -1165,7 +1165,7 @@ When kbuild executes, the following steps are followed 
(roughly):
 
        The ": %: %.o" part of the prerequisite is a shorthand that
        free us from listing the setup.o and bootsect.o files.
-       Note: It is a common mistake to forget the "target :=" assignment,
+       Note: It is a common mistake to forget the "targets :=" assignment,
              resulting in the target file being recompiled for no
              obvious reason.
 
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to