diff mbox

[1/2] gmsl-yavta: Remove bash-isms

Message ID 20180131210108.28048-1-laurent.pinchart@ideasonboard.com (mailing list archive)
State Not Applicable
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Laurent Pinchart Jan. 31, 2018, 9:01 p.m. UTC
The script contains two bash-specific constructs:

- Replace ${!var} expansion with eval
- Initialize conf1234 and conf5678 variables to avoid empty argument to
  -ne test

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 gmsl-yavta.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Niklas Söderlund Jan. 31, 2018, 11:09 p.m. UTC | #1
Hi Laurent,

Thanks for your patch.

On 2018-01-31 23:01:08 +0200, Laurent Pinchart wrote:
> The script contains two bash-specific constructs:
> 
> - Replace ${!var} expansion with eval
> - Initialize conf1234 and conf5678 variables to avoid empty argument to
>   -ne test
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Applied.

> ---
>  gmsl-yavta.sh | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/gmsl-yavta.sh b/gmsl-yavta.sh
> index 635ab72feb07..a123bf52a16d 100755
> --- a/gmsl-yavta.sh
> +++ b/gmsl-yavta.sh
> @@ -22,13 +22,17 @@ function conf() {
>  
>  function capture() {
>      VID="$1"
> +    vid=$(eval echo \$$VID)
>  
> -    yavta -f YUYV -s 1280x800 -c10 --skip 7 --file="$out/$VID-#.bin" /dev/${!VID} &
> +    yavta -f YUYV -s 1280x800 -c10 --skip 7 --file="$out/$VID-#.bin" /dev/$vid &
>  }
>  
>  mc_reset
>  
>  # Need to configure all formats going through each MAX9286
> +conf1234=0
> +conf5678=0
> +
>  for cam in "$@"; do
>  	case $cam in
>          1|2|3|4)
> -- 
> Regards,
> 
> Laurent Pinchart
>
diff mbox

Patch

diff --git a/gmsl-yavta.sh b/gmsl-yavta.sh
index 635ab72feb07..a123bf52a16d 100755
--- a/gmsl-yavta.sh
+++ b/gmsl-yavta.sh
@@ -22,13 +22,17 @@  function conf() {
 
 function capture() {
     VID="$1"
+    vid=$(eval echo \$$VID)
 
-    yavta -f YUYV -s 1280x800 -c10 --skip 7 --file="$out/$VID-#.bin" /dev/${!VID} &
+    yavta -f YUYV -s 1280x800 -c10 --skip 7 --file="$out/$VID-#.bin" /dev/$vid &
 }
 
 mc_reset
 
 # Need to configure all formats going through each MAX9286
+conf1234=0
+conf5678=0
+
 for cam in "$@"; do
 	case $cam in
         1|2|3|4)