In /usr/bin/scsi_ch_swp line 43: s|-set) shift ; let swp="$1" ;; ^-----------^ SC2219: Instead of 'let expr', prefer (( expr )) . In /usr/bin/scsi_ch_swp line 69: sdparm ${rdonly} ${verbose} ^-------^ SC2086: Double quote to prevent globbing and word splitting. Did you mean: sdparm "${rdonly}" ${verbose} In /usr/bin/scsi_ch_swp line 81: echo sdparm --quiet ${rdonly} ${verbose} --get=SWP=1 "$1" ^-------^ SC2086: Double quote to prevent globbing and word splitting. Did you mean: echo sdparm --quiet "${rdonly}" ${verbose} --get=SWP=1 "$1" In /usr/bin/scsi_ch_swp line 83: sdparm --quiet ${rdonly} ${verbose} --get=SWP=1 "$1" ^-------^ SC2086: Double quote to prevent globbing and word splitting. Did you mean: sdparm --quiet "${rdonly}" ${verbose} --get=SWP=1 "$1" In /usr/bin/scsi_ch_swp line 91: echo sdparm --quiet ${rdonly} ${verbose} --clear=SWP "$1" ^-------^ SC2086: Double quote to prevent globbing and word splitting. Did you mean: echo sdparm --quiet "${rdonly}" ${verbose} --clear=SWP "$1" In /usr/bin/scsi_ch_swp line 93: sdparm --quiet ${rdonly} ${verbose} --clear=SWP "$1" ^-------^ SC2086: Double quote to prevent globbing and word splitting. Did you mean: sdparm --quiet "${rdonly}" ${verbose} --clear=SWP "$1" For more information: https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ... https://www.shellcheck.net/wiki/SC2219 -- Instead of 'let expr', prefer (( ...