casaroli opened a new pull request, #21322: URL: https://github.com/apache/nuttx/pull/21322
## Summary `net/arp/arp_table.c` converts both ARP ages with `SEC2TICK(10 * value)`, so their unit is 10 seconds: ```c #define ARP_MAXAGE_TICK SEC2TICK(10 * CONFIG_NET_ARP_MAXAGE) #define ARP_MAXAGE_UNREACHABLE_TICK SEC2TICK(10 * CONFIG_NET_ARP_MAXAGE_UNREACHABLE) ``` The help texts of `CONFIG_NET_ARP_MAXAGE` and `CONFIG_NET_ARP_MAXAGE_UNREACHABLE` said deciseconds, and the comment in `include/nuttx/net/netconfig.h` said tenths of seconds: 100 times too small. The example in the same texts (120 is 20 minutes) already used 10 seconds. This PR corrects the three texts. The code does not change. ## Impact Text only: the Kconfig help and a header comment. A user who set a value from the old help text gets an age 100 times longer than intended; this PR does not change that value. ## Testing `lm3s6965-ek:qemu-flat` (with `CONFIG_NET_ARP`) configures with no Kconfig warnings, and the two options keep their defaults (120 and 1). `tools/checkpatch.sh -c -u -m -g` passes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
