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;
}
