TLV_PRIORITY1 and TLV_PRIORITY2 cases in do_set_action() use the same repeated
piece of generic code for setting one-value parameter. Remove the duplicated
code and let both cases use the same code.

Signed-off-by: Petr Kulhavy <br...@jikos.cz>
---
 pmc.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/pmc.c b/pmc.c
index 261f2a2..cefa771 100644
--- a/pmc.c
+++ b/pmc.c
@@ -515,14 +515,6 @@ static void do_set_action(int action, int index, char *str)
        }
        switch (code) {
        case TLV_PRIORITY1:
-               cnt = sscanf(str,  " %*s %*s %hhu", &mtd.val);
-               if (cnt != 1) {
-                       fprintf(stderr, "%s SET needs 1 value\n",
-                               idtab[index].name);
-                       break;
-               }
-               pmc_send_set_action(pmc, code, &mtd, sizeof(mtd));
-               break;
        case TLV_PRIORITY2:
                cnt = sscanf(str,  " %*s %*s %hhu", &mtd.val);
                if (cnt != 1) {
-- 
2.7.4


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to