On Tue, Nov 12, 2024 at 2:12 PM Len Brown lenb@kernel.org wrote:

On Tue, Nov 12, 2024 at 6:44 AM Rafael J. Wysocki rafael@kernel.org wrote:

...

...

•   if (boot_cpu_has(X86_FEATURE_MWAIT) && c->x86_vfm == INTEL_ATOM_GOLDMONT)
  

•   if (boot_cpu_has(X86_FEATURE_MWAIT) &&
  

•       (c->x86_vfm == INTEL_ATOM_GOLDMONT
  

•        || c->x86_vfm == INTEL_LUNARLAKE_M))
  

I would put the || at the end of the previous line, that is

It isn't my personal preference for human readability either, but this is what scripts/Lindent does...

Well, it doesn't match the coding style of the first line ...