From: mattsu <[email protected]> The status option already exercises WDIOC_GETTEMP, but the usage text does not mention that it also reports temperature. The file header also contains a stale TODO for adding WDIOC_GETTEMP coverage.
Mention temperature in the status option help and remove the stale TODO. Signed-off-by: mattsu <[email protected]> --- tools/testing/selftests/watchdog/watchdog-test.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tools/testing/selftests/watchdog/watchdog-test.c b/tools/testing/selftests/watchdog/watchdog-test.c index 4f09c5db0c7f..0626c96f2c9f 100644 --- a/tools/testing/selftests/watchdog/watchdog-test.c +++ b/tools/testing/selftests/watchdog/watchdog-test.c @@ -5,8 +5,6 @@ * - Tests Magic Close - CONFIG_WATCHDOG_NOWAYOUT * - Could be tested against softdog driver on systems that * don't have watchdog hardware. -* - TODO: -* - Enhance test to add coverage for WDIOC_GETTEMP. * * Reference: Documentation/watchdog/watchdog-api.rst */ @@ -83,7 +81,7 @@ static void usage(char *progname) printf(" -f, --file\t\tOpen watchdog device file\n"); printf("\t\t\tDefault is /dev/watchdog\n"); printf(" -i, --info\t\tShow watchdog_info\n"); - printf(" -s, --status\t\tGet status & supported features\n"); + printf(" -s, --status\t\tGet status, supported features, and temperature\n"); printf(" -b, --bootstatus\tGet last boot status (Watchdog/POR)\n"); printf(" -d, --disable\t\tTurn off the watchdog timer\n"); printf(" -e, --enable\t\tTurn on the watchdog timer\n"); -- 2.54.0
