On Thu, 19 Jul 2018, Dou Liyang wrote: > At 07/18/2018 10:22 AM, Pavel Tatashin wrote: > > + (unsigned long)cpu_khz % KHZ); > > if (cpu_khz != tsc_khz) { > > pr_info("Detected %lu.%03lu MHz TSC", > > - (unsigned long)tsc_khz / 1000, > > - (unsigned long)tsc_khz % 1000); > > + (unsigned long)tsc_khz / KHZ, > > + (unsigned long)tsc_khz % KHZ); > > + } > > this curly brackets can be removed No. They want to stay, really. https://lkml.kernel.org/r/alpine.DEB.2.20.1701171956290.3645@nanos The pr_info() is a multiline statement due to the line breaks. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html