The option itself was considered correctly, but a small but crucial break statement was missing. --data-offset option does not take an argument either.
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com> --- yavta.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/yavta.c b/yavta.c index 3bec0be..7c21db2 100644 --- a/yavta.c +++ b/yavta.c @@ -1770,7 +1770,7 @@ static struct option opts[] = { {"buffer-type", 1, 0, 'B'}, {"capture", 2, 0, 'c'}, {"check-overrun", 0, 0, 'C'}, - {"data-prefix", 1, 0, OPT_DATA_PREFIX}, + {"data-prefix", 0, 0, OPT_DATA_PREFIX}, {"delay", 1, 0, 'd'}, {"enum-formats", 0, 0, OPT_ENUM_FORMATS}, {"enum-inputs", 0, 0, OPT_ENUM_INPUTS}, @@ -2034,6 +2034,7 @@ int main(int argc, char *argv[]) break; case OPT_DATA_PREFIX: dev.write_data_prefix = true; + break; default: printf("Invalid option -%c\n", c); printf("Run %s -h for help.\n", argv[0]); -- 2.1.0.231.g7484e3b -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html