Re: [PATCH v2 1/8] x86/cpu/hygon: Adjust the die_id and logical_die_id for Hygon models 0x4-0x8
From: Borislav Petkov
Date: Tue Jun 02 2026 - 22:28:03 EST
On Tue, Apr 07, 2026 at 04:22:42PM +0800, Fu Hao wrote:
> The die ID should be got from the NodeId field of CPUID leaf 0x8000001e
> ecx for Hygon models 0x4-0x8 processors.
Correct the formulations in this text: "The die ID should be got from the
NodeId field of CPUID leaf 0x8000001e ecx for Hygon models 0x4-0x8
processors."
AI:
Here are some corrected versions of your sentence:
Option 1 (minimal changes): "The die ID should be obtained from the
NodeId field of CPUID leaf 0x8000001e ECX for Hygon processors of
models 0x4–0x8."
Option 2 (more natural phrasing): "The die ID should be read from the
NodeId field of CPUID leaf 0x8000001e (ECX) for Hygon processor models
0x4–0x8."
Option 3 (concise): "For Hygon processor models 0x4–0x8, the die ID
should be obtained from the NodeId field in ECX of CPUID leaf
0x8000001e."
Key issues corrected:
"should be got" → "should be obtained" / "should be read" (more formal and grammatically preferred)
"ecx" → "ECX" (CPU register names are conventionally capitalized)
"Hygon models 0x4-0x8 processors" → "Hygon processor models 0x4–0x8" (clearer word order)
Consider using an en dash (–) instead of a hyphen (-) for the range, though a hyphen is acceptable in code comments.
Now you do it for all your commit messages please.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette