There doesn't seem to be a good reason for omitting the newlines here. If those are ever hit, it will look odd. Am I missing something?
Index: est.c =================================================================== RCS file: /cvs/src/sys/arch/amd64/amd64/est.c,v retrieving revision 1.40 diff -u -p -r1.40 est.c --- est.c 11 Aug 2021 18:15:50 -0000 1.40 +++ est.c 12 Aug 2021 04:40:14 -0000 @@ -185,7 +185,7 @@ p3_get_bus_clock(struct cpu_info *ci) bus_clock = BUS333; break; default: - printf("%s: unknown Core FSB_FREQ value %d", + printf("%s: unknown Core FSB_FREQ value %d\n", ci->ci_dev->dv_xname, bus); break; } @@ -209,7 +209,7 @@ p3_get_bus_clock(struct cpu_info *ci) bus_clock = BUS200; break; default: - printf("%s: unknown Atom FSB_FREQ value %d", + printf("%s: unknown Atom FSB_FREQ value %d\n", ci->ci_dev->dv_xname, bus); break; } @@ -291,14 +291,14 @@ est_acpi_pss_changed(struct acpicpu_pss if ((acpilist = malloc(sizeof(struct fqlist), M_DEVBUF, M_NOWAIT)) == NULL) { printf("est_acpi_pss_changed: cannot allocate memory for new " - "est state"); + "est state\n"); return; } if ((acpilist->table = mallocarray(npss, sizeof(struct est_op), M_DEVBUF, M_NOWAIT)) == NULL) { printf("est_acpi_pss_changed: cannot allocate memory for new " - "operating points"); + "operating points\n"); free(acpilist, M_DEVBUF, sizeof(struct fqlist)); return; }