[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-wpan
Subject:    [PATCH wpan-tools 1/2] info: remove confusing supported commands printout
From:       Stefan Schmidt <stefan () osg ! samsung ! com>
Date:       2016-09-14 12:03:52
Message-ID: 1473854633-4391-2-git-send-email-stefan () osg ! samsung ! com
[Download RAW message or body]

This is confusing for the user as it used the raw netlink names and might not
match the help output we print. If we want to query the kernel for available
commands to display them we need to spent more energy to print it out
understandable.

Fixes #8

Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>
---
 src/info.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/src/info.c b/src/info.c
index 69f37f9..b06093b 100644
--- a/src/info.c
+++ b/src/info.c
@@ -472,16 +472,6 @@ static int print_phy_handler(struct nl_msg *msg, void *arg)
 			}
 		}
 	}
-
-	if (tb_msg[NL802154_ATTR_SUPPORTED_COMMANDS]) {
-		struct nlattr *nl_cmd;
-		int rem_cmd;
-
-		printf("Supported commands:\n");
-		nla_for_each_nested(nl_cmd, tb_msg[NL802154_ATTR_SUPPORTED_COMMANDS], rem_cmd)
-			printf("\t* %s\n", command_name(nla_get_u32(nl_cmd)));
-	}
-
 	return 0;
 }
 
-- 
2.5.5

--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]