Skip to content

Commit

Permalink
kernel-doc: unify all EXPORT_SYMBOL scanning to one place
Browse files Browse the repository at this point in the history
Scan all input files for EXPORT_SYMBOLs along with the explicitly
specified export files before actually parsing anything.

Signed-off-by: Jani Nikula <[email protected]>
  • Loading branch information
jnikula committed Jun 10, 2016
1 parent 88c2b57 commit c9b2cfb
Showing 1 changed file with 5 additions and 13 deletions.
18 changes: 5 additions & 13 deletions scripts/kernel-doc
Original file line number Diff line number Diff line change
Expand Up @@ -61,10 +61,10 @@ Output format selection (mutually exclusive):
Output selection (mutually exclusive):
-export Only output documentation for symbols that have been
exported using EXPORT_SYMBOL() or EXPORT_SYMBOL_GPL()
in the same FILE or any -export-file FILE.
in any input FILE or -export-file FILE.
-internal Only output documentation for symbols that have NOT been
exported using EXPORT_SYMBOL() or EXPORT_SYMBOL_GPL()
in the same FILE or any -export-file FILE.
in any input FILE or -export-file FILE.
-function NAME Only output documentation for the given function(s)
or DOC: section title(s). All other functions and DOC:
sections are ignored. May be specified multiple times.
Expand Down Expand Up @@ -2792,17 +2792,6 @@ sub process_file($) {
return;
}

# two passes for -export and -internal
if ($output_selection == OUTPUT_EXPORTED ||
$output_selection == OUTPUT_INTERNAL) {
while (<IN>) {
if (/$export_symbol/o) {
$function_table{$2} = 1;
}
}
seek(IN, 0, 0);
}

$. = 1;

$section_counter = 0;
Expand Down Expand Up @@ -3110,6 +3099,9 @@ if (open(SOURCE_MAP, "<.tmp_filelist.txt")) {

if ($output_selection == OUTPUT_EXPORTED ||
$output_selection == OUTPUT_INTERNAL) {

push(@export_file_list, @ARGV);

foreach (@export_file_list) {
chomp;
process_export_file($_);
Expand Down

0 comments on commit c9b2cfb

Please sign in to comment.