Looks good. Reviewed-by : Chaitanya Kulkarni <chaitanya.kulkarni@xxxxxxx> On 3/13/19 12:05 AM, Xiao Liang wrote: > Signed-off-by: Xiao Liang <xiliang@xxxxxxxxxx> > --- > common/rc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/common/rc b/common/rc > index 0c354a2..71e27c3 100644 > --- a/common/rc > +++ b/common/rc > @@ -33,7 +33,7 @@ _have_modules() { > local module > > for module in "$@"; do > - if ! modprobe -n -q "$1"; then > + if ! modprobe -n -q "$module"; then > missing+=("$module") > fi > done >