net: wifi: Use a macro for TWT wake interval

Remove the magic number 255, esp. now that we support 262.144ms.

Signed-off-by: Krishna T <[email protected]>

Author: Krishna T

include/zephyr/net/wifi_mgmt.h

@@ -230,6 +230,8 @@ struct wifi_twt_params {
 /* Flow ID is only 3 bits */
 #define WIFI_MAX_TWT_FLOWS 8
 #define WIFI_MAX_TWT_INTERVAL_US (ULONG_MAX - 1)
+/* 256 (u8) * 1TU */
+#define WIFI_MAX_TWT_WAKE_INTERVAL_US 262144
 struct wifi_twt_flow_info {
 	/* Interval = Wake up time + Sleeping time */
 	uint64_t  twt_interval;

subsys/net/l2/wifi/wifi_shell.c

@@ -599,7 +599,8 @@ static int cmd_wifi_twt_setup_quick(const struct shell *sh, size_t argc,
 	params.setup.trigger = 1;
 	params.setup.announce = 0;
 
-	if (!parse_number(sh, (long *)&params.setup.twt_wake_interval, argv[idx++], 1, 255) ||
+	if (!parse_number(sh, (long *)&params.setup.twt_wake_interval, argv[idx++],
+			  1, WIFI_MAX_TWT_WAKE_INTERVAL_US) ||
 	    !parse_number(sh, (long *)&params.setup.twt_interval, argv[idx++], 1,
 			  WIFI_MAX_TWT_INTERVAL_US))
 		return -EINVAL;
@@ -648,7 +649,8 @@ static int cmd_wifi_twt_setup(const struct shell *sh, size_t argc,
 	    !parse_number(sh, (long *)&params.setup.trigger, argv[idx++], 0, 1) ||
 	    !parse_number(sh, (long *)&params.setup.implicit, argv[idx++], 0, 1) ||
 	    !parse_number(sh, (long *)&params.setup.announce, argv[idx++], 0, 1) ||
-	    !parse_number(sh, (long *)&params.setup.twt_wake_interval, argv[idx++], 1, 255) ||
+	    !parse_number(sh, (long *)&params.setup.twt_wake_interval, argv[idx++], 1,
+			  WIFI_MAX_TWT_WAKE_INTERVAL_US) ||
 	    !parse_number(sh, (long *)&params.setup.twt_interval, argv[idx++], 1,
 			  WIFI_MAX_TWT_INTERVAL_US))
 		return -EINVAL;