Re: [PATCH igt 1/2] tools/msm_dp_compliance: Small arg parsing cleanup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviving this:

On 12/8/2021 11:22 AM, Rob Clark wrote:
From: Rob Clark <robdclark@xxxxxxxxxxxx>

Signed-off-by: Rob Clark <robdclark@xxxxxxxxxxxx>

Reviewed-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>

Nit: shouldnt the patch prefix be "PATCH i-g-t" ?

We will test this out next week and land this.

---
  tools/msm_dp_compliance.c | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/tools/msm_dp_compliance.c b/tools/msm_dp_compliance.c
index 5e491c89..a15a8dd9 100644
--- a/tools/msm_dp_compliance.c
+++ b/tools/msm_dp_compliance.c
@@ -629,7 +629,9 @@ static const char optstr[] = "hi";
  static void __attribute__((noreturn)) usage(char *name, char opt)
  {
  	igt_info("usage: %s [-hi]\n", name);
-	igt_info("\t-i\tdump info\n");
+	igt_info("\t-h, --help           - print this usage message");
+	igt_info("\t--help-description   - print test description");
+	igt_info("\t-i, --info           - dump info\n");
  	igt_info("\tDefault is to respond to Qd980 tests\n");
  	exit((opt != 'h') ? -1 : 0);
  }
@@ -709,6 +711,7 @@ int main(int argc, char **argv)
  	struct option long_opts[] = {
  		{"help-description", 0, 0, HELP_DESCRIPTION},
  		{"help", 0, 0, 'h'},
+		{"info", 0, 0, 'i'},
  	};
enter_exec_path(argv);



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux