Skip to content

Commit

Permalink
sof_{insert/remove}.sh: Add logging for default case
Browse files Browse the repository at this point in the history
Added some messages for the case in which VENDOR is not specified.

The purpose of this change is to clearly show which removal/insertion
script is running at some time. Before this change, the output of
the insertion/removal scripts made it look like it was one big
insertion/removal script containing all the modules which is
clearly not the case.

Signed-off-by: Laurentiu Mihalcea <[email protected]>
  • Loading branch information
LaurentiuM1234 committed Jul 13, 2022
1 parent 6c6350b commit d112f2d
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tools/kmod/sof_insert.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@

TOPDIR=$(cd "$(dirname "${BASH_SOURCE[0]}")/../.." && pwd)

# shellcheck source=case-lib/lib.sh
source "$TOPDIR"/case-lib/lib.sh

# shellcheck source=tools/kmod/lib.sh
source "$TOPDIR"/tools/kmod/lib.sh

Expand All @@ -22,7 +25,10 @@ main()
return $?
fi

dlogi "VENDOR not specified. Running all insertion subscripts."

for insert_subscript in "$TOPDIR"/tools/kmod/sof_insert_*.sh; do
dlogi "Currently running: $insert_subscript"
"$insert_subscript"
done
}
Expand Down
3 changes: 3 additions & 0 deletions tools/kmod/sof_remove.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,10 @@ main()
return $?
fi

dlogi "VENDOR not specified. Running all removal subscripts."

for remove_subscript in "$TOPDIR"/tools/kmod/sof_remove_*.sh; do
dlogi "Currently running: $remove_subscript"
"$remove_subscript" || true
done
}
Expand Down

0 comments on commit d112f2d

Please sign in to comment.