feat: add FEAT_ARGS to use instead of NVME_ARGS

Includes save and sel options for all nvme feat plugin commands.

Signed-off-by: Tokunori Ikegami <[email protected]>

Author: ikegami-t

plugins/feat/feat-nvme.c

@@ -108,11 +108,9 @@ static int feat_power_mgmt(int argc, char **argv, struct command *cmd, struct pl
 
 	struct config cfg = { 0 };
 
-	NVME_ARGS(opts,
+	FEAT_ARGS(opts,
 		  OPT_BYTE("ps", 'p', &cfg.ps, ps),
-		  OPT_BYTE("wh", 'w', &cfg.wh, wh),
-		  OPT_FLAG("save", 's', &cfg.save, save),
-		  OPT_BYTE("sel", 'S', &cfg.sel, sel));
+		  OPT_BYTE("wh", 'w', &cfg.wh, wh));
 
 	err = parse_and_open(&dev, argc, argv, POWER_MGMT_DESC, opts);
 	if (err)
@@ -239,16 +237,14 @@ static int feat_perfc(int argc, char **argv, struct command *cmd, struct plugin
 
 	struct perfc_config cfg = { 0 };
 
-	NVME_ARGS(opts,
+	FEAT_ARGS(opts,
 		  OPT_UINT("namespace-id", 'n', &cfg.namespace_id, namespace_id_optional),
 		  OPT_BYTE("attri", 'a', &cfg.attri, attri),
 		  OPT_FLAG("rvspa", 'r', &cfg.rvspa, rvspa),
 		  OPT_BYTE("r4karl", 'R', &cfg.r4karl, r4karl),
 		  OPT_STR("paid", 'p', &cfg.paid, paid),
 		  OPT_SHRT("attrl", 'A', &cfg.attrl, attrl),
-		  OPT_FILE("vs-data", 'V', &cfg.vs_data, vs_data),
-		  OPT_FLAG("save", 's', &cfg.save, save),
-		  OPT_BYTE("sel", 'S', &cfg.sel, sel));
+		  OPT_FILE("vs-data", 'V', &cfg.vs_data, vs_data));
 
 	err = parse_and_open(&dev, argc, argv, PERFC_DESC, opts);
 	if (err)
@@ -340,11 +336,9 @@ static int feat_hctm(int argc, char **argv, struct command *cmd, struct plugin *
 
 	struct config cfg = { 0 };
 
-	NVME_ARGS(opts,
+	FEAT_ARGS(opts,
 		  OPT_SHRT("tmt1", 't', &cfg.tmt1, TMT(1)),
-		  OPT_SHRT("tmt2", 'T', &cfg.tmt2, TMT(2)),
-		  OPT_FLAG("save", 's', &cfg.save, save),
-		  OPT_BYTE("sel", 'S', &cfg.sel, sel));
+		  OPT_SHRT("tmt2", 'T', &cfg.tmt2, TMT(2)));
 
 	err = parse_and_open(&dev, argc, argv, HCTM_DESC, opts);
 	if (err)

plugins/feat/feat-nvme.h

@@ -14,6 +14,10 @@
 #define PERFC_DESC "Get and set perf characteristics feature"
 #define HCTM_DESC "Get and set host controlled thermal management feature"
 
+#define FEAT_ARGS(n, ...)                                                              \
+	NVME_ARGS(n, ##__VA_ARGS__, OPT_FLAG("save", 's', &cfg.save, save),            \
+		  OPT_BYTE("sel", 'S', &cfg.sel, sel))
+
 PLUGIN(NAME("feat", "NVMe feature extensions", FEAT_PLUGIN_VERSION),
 	COMMAND_LIST(
 		ENTRY("power-mgmt", POWER_MGMT_DESC, feat_power_mgmt)