Adjusted frequency value displayed by do_freq is not an error, but a notication to the user. pr_err should not be used for providing notices with information about successful operations.
Fixes: bdb6a35883b0 ("linuxptp: add phc_ctl program to help debug PHC devices") Signed-off-by: Rahul Rameshbabu <rrameshb...@nvidia.com> --- phc_ctl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/phc_ctl.c b/phc_ctl.c index ddc0b9d..53b26e1 100644 --- a/phc_ctl.c +++ b/phc_ctl.c @@ -249,7 +249,7 @@ static int do_freq(clockid_t clkid, int cmdc, char *cmdv[]) if (cmdc < 1 || name_is_a_command(cmdv[0])) { ppb = clockadj_get_freq(clkid); - pr_err("clock frequency offset is %lfppb", ppb); + pr_notice("clock frequency offset is %lfppb", ppb); /* no argument was used */ return 0; @@ -272,7 +272,7 @@ static int do_freq(clockid_t clkid, int cmdc, char *cmdv[]) } clockadj_set_freq(clkid, ppb); - pr_err("adjusted clock frequency offset to %lfppb", ppb); + pr_notice("adjusted clock frequency offset to %lfppb", ppb); /* consumed one argument to determine the frequency adjustment value */ return 1; -- 2.40.1 _______________________________________________ Linuxptp-devel mailing list Linuxptp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxptp-devel