wireless/lpwan/rn2xx3: Fix invalid sync word bug.

linguini1 · acassis · commit 0cb7bd7f7625 · 2025-04-20T16:55:45.000-03:00
This commit resolves issue #16009, where a sync word of less than 64 bytes was printed as 0-padded hex. 0-padding is invalid for the RN2xx3 modules, so now it has been removed. Signed-off-by: Matteo Golin <matteo.golin@gmail.com>
diff --git a/drivers/wireless/lpwan/rn2xx3/rn2xx3.c b/drivers/wireless/lpwan/rn2xx3/rn2xx3.c
@@ -1165,7 +1165,7 @@ static int rn2xx3_setsync(FAR struct rn2xx3_dev_s *priv, uint64_t sync)
 
   /* Write actual parameter and end command */
 
-  length = snprintf(syncstr, sizeof(syncstr), "%016llX\r\n", sync);
+  length = snprintf(syncstr, sizeof(syncstr), "%llX\r\n", sync);
   length = file_write(&priv->uart, syncstr, length);
   if (length < 0)
     {

Original file line number	Diff line number	Diff line change
`@@ -1165,7 +1165,7 @@ static int rn2xx3_setsync(FAR struct rn2xx3_dev_s *priv, uint64_t sync)`
`1165`	`1165`
`1166`	`1166`	`/* Write actual parameter and end command */`
`1167`	`1167`
`1168`		`- length = snprintf(syncstr, sizeof(syncstr), "%016llX\r\n", sync);`
	`1168`	`+ length = snprintf(syncstr, sizeof(syncstr), "%llX\r\n", sync);`
`1169`	`1169`	`length = file_write(&priv->uart, syncstr, length);`
`1170`	`1170`	`if (length < 0)`
`1171`	`1171`	`{`