| Message ID | 20190426220309.55654-2-jeremy.linton@arm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: SPE ACPI enablement | expand |
On Sat, Apr 27, 2019 at 12:03 AM Jeremy Linton <jeremy.linton@arm.com> wrote: > > CPU is an acronym and customarily capitalized. Much of the > commenting in the PPTT code has been using "cpu" rather > than "CPU". Correct that, and other human readable strings. > > Signed-off-by: Jeremy Linton <jeremy.linton@arm.com> It would be better to say "Modify the spelling of CPU" in the subject IMO, but apart from that Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > --- > drivers/acpi/pptt.c | 48 ++++++++++++++++++++++----------------------- > 1 file changed, 24 insertions(+), 24 deletions(-) > > diff --git a/drivers/acpi/pptt.c b/drivers/acpi/pptt.c > index 065c4fc245d1..83a026765faa 100644 > --- a/drivers/acpi/pptt.c > +++ b/drivers/acpi/pptt.c > @@ -164,7 +164,7 @@ static struct acpi_pptt_cache *acpi_find_cache_level(struct acpi_table_header *t > } > > /** > - * acpi_count_levels() - Given a PPTT table, and a cpu node, count the caches > + * acpi_count_levels() - Given a PPTT table, and a CPU node, count the caches > * @table_hdr: Pointer to the head of the PPTT table > * @cpu_node: processor node we wish to count caches for > * > @@ -235,7 +235,7 @@ static int acpi_pptt_leaf_node(struct acpi_table_header *table_hdr, > /** > * acpi_find_processor_node() - Given a PPTT table find the requested processor > * @table_hdr: Pointer to the head of the PPTT table > - * @acpi_cpu_id: cpu we are searching for > + * @acpi_cpu_id: CPU we are searching for > * > * Find the subtable entry describing the provided processor. > * This is done by iterating the PPTT table looking for processor nodes > @@ -456,21 +456,21 @@ static struct acpi_pptt_processor *acpi_find_processor_package_id(struct acpi_ta > > static void acpi_pptt_warn_missing(void) > { > - pr_warn_once("No PPTT table found, cpu and cache topology may be inaccurate\n"); > + pr_warn_once("No PPTT table found, CPU and cache topology may be inaccurate\n"); > } > > /** > * topology_get_acpi_cpu_tag() - Find a unique topology value for a feature > * @table: Pointer to the head of the PPTT table > - * @cpu: Kernel logical cpu number > + * @cpu: Kernel logical CPU number > * @level: A level that terminates the search > * @flag: A flag which terminates the search > * > - * Get a unique value given a cpu, and a topology level, that can be > - * matched to determine which cpus share common topological features > + * Get a unique value given a CPU, and a topology level, that can be > + * matched to determine which CPUs share common topological features > * at that level. > * > - * Return: Unique value, or -ENOENT if unable to locate cpu > + * Return: Unique value, or -ENOENT if unable to locate CPU > */ > static int topology_get_acpi_cpu_tag(struct acpi_table_header *table, > unsigned int cpu, int level, int flag) > @@ -510,7 +510,7 @@ static int find_acpi_cpu_topology_tag(unsigned int cpu, int level, int flag) > return -ENOENT; > } > retval = topology_get_acpi_cpu_tag(table, cpu, level, flag); > - pr_debug("Topology Setup ACPI cpu %d, level %d ret = %d\n", > + pr_debug("Topology Setup ACPI CPU %d, level %d ret = %d\n", > cpu, level, retval); > acpi_put_table(table); > > @@ -519,9 +519,9 @@ static int find_acpi_cpu_topology_tag(unsigned int cpu, int level, int flag) > > /** > * acpi_find_last_cache_level() - Determines the number of cache levels for a PE > - * @cpu: Kernel logical cpu number > + * @cpu: Kernel logical CPU number > * > - * Given a logical cpu number, returns the number of levels of cache represented > + * Given a logical CPU number, returns the number of levels of cache represented > * in the PPTT. Errors caused by lack of a PPTT table, or otherwise, return 0 > * indicating we didn't find any cache levels. > * > @@ -534,7 +534,7 @@ int acpi_find_last_cache_level(unsigned int cpu) > int number_of_levels = 0; > acpi_status status; > > - pr_debug("Cache Setup find last level cpu=%d\n", cpu); > + pr_debug("Cache Setup find last level CPU=%d\n", cpu); > > acpi_cpu_id = get_acpi_id_for_cpu(cpu); > status = acpi_get_table(ACPI_SIG_PPTT, 0, &table); > @@ -551,14 +551,14 @@ int acpi_find_last_cache_level(unsigned int cpu) > > /** > * cache_setup_acpi() - Override CPU cache topology with data from the PPTT > - * @cpu: Kernel logical cpu number > + * @cpu: Kernel logical CPU number > * > * Updates the global cache info provided by cpu_get_cacheinfo() > * when there are valid properties in the acpi_pptt_cache nodes. A > * successful parse may not result in any updates if none of the > * cache levels have any valid flags set. Futher, a unique value is > * associated with each known CPU cache entry. This unique value > - * can be used to determine whether caches are shared between cpus. > + * can be used to determine whether caches are shared between CPUs. > * > * Return: -ENOENT on failure to find table, or 0 on success > */ > @@ -567,7 +567,7 @@ int cache_setup_acpi(unsigned int cpu) > struct acpi_table_header *table; > acpi_status status; > > - pr_debug("Cache Setup ACPI cpu %d\n", cpu); > + pr_debug("Cache Setup ACPI CPU %d\n", cpu); > > status = acpi_get_table(ACPI_SIG_PPTT, 0, &table); > if (ACPI_FAILURE(status)) { > @@ -582,8 +582,8 @@ int cache_setup_acpi(unsigned int cpu) > } > > /** > - * find_acpi_cpu_topology() - Determine a unique topology value for a given cpu > - * @cpu: Kernel logical cpu number > + * find_acpi_cpu_topology() - Determine a unique topology value for a given CPU > + * @cpu: Kernel logical CPU number > * @level: The topological level for which we would like a unique ID > * > * Determine a topology unique ID for each thread/core/cluster/mc_grouping > @@ -596,7 +596,7 @@ int cache_setup_acpi(unsigned int cpu) > * other levels beyond this use a generated value to uniquely identify > * a topological feature. > * > - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. > + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. > * Otherwise returns a value which represents a unique topological feature. > */ > int find_acpi_cpu_topology(unsigned int cpu, int level) > @@ -606,12 +606,12 @@ int find_acpi_cpu_topology(unsigned int cpu, int level) > > /** > * find_acpi_cpu_cache_topology() - Determine a unique cache topology value > - * @cpu: Kernel logical cpu number > + * @cpu: Kernel logical CPU number > * @level: The cache level for which we would like a unique ID > * > * Determine a unique ID for each unified cache in the system > * > - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. > + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. > * Otherwise returns a value which represents a unique topological feature. > */ > int find_acpi_cpu_cache_topology(unsigned int cpu, int level) > @@ -643,17 +643,17 @@ int find_acpi_cpu_cache_topology(unsigned int cpu, int level) > > > /** > - * find_acpi_cpu_topology_package() - Determine a unique cpu package value > - * @cpu: Kernel logical cpu number > + * find_acpi_cpu_topology_package() - Determine a unique CPU package value > + * @cpu: Kernel logical CPU number > * > - * Determine a topology unique package ID for the given cpu. > + * Determine a topology unique package ID for the given CPU. > * This ID can then be used to group peers, which will have matching ids. > * > * The search terminates when either a level is found with the PHYSICAL_PACKAGE > * flag set or we reach a root node. > * > - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. > - * Otherwise returns a value which represents the package for this cpu. > + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. > + * Otherwise returns a value which represents the package for this CPU. > */ > int find_acpi_cpu_topology_package(unsigned int cpu) > { > -- > 2.20.1 >
Hi, And sorry about the delay... On 4/29/19 3:50 AM, Rafael J. Wysocki wrote: > On Sat, Apr 27, 2019 at 12:03 AM Jeremy Linton <jeremy.linton@arm.com> wrote: >> >> CPU is an acronym and customarily capitalized. Much of the >> commenting in the PPTT code has been using "cpu" rather >> than "CPU". Correct that, and other human readable strings. >> >> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com> > > It would be better to say "Modify the spelling of CPU" in the subject > IMO, but apart from that Hmmm, spelling doesn't seem quite right either, how about "capitalization"? > > Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > >> --- >> drivers/acpi/pptt.c | 48 ++++++++++++++++++++++----------------------- >> 1 file changed, 24 insertions(+), 24 deletions(-) >> >> diff --git a/drivers/acpi/pptt.c b/drivers/acpi/pptt.c >> index 065c4fc245d1..83a026765faa 100644 >> --- a/drivers/acpi/pptt.c >> +++ b/drivers/acpi/pptt.c >> @@ -164,7 +164,7 @@ static struct acpi_pptt_cache *acpi_find_cache_level(struct acpi_table_header *t >> } >> >> /** >> - * acpi_count_levels() - Given a PPTT table, and a cpu node, count the caches >> + * acpi_count_levels() - Given a PPTT table, and a CPU node, count the caches >> * @table_hdr: Pointer to the head of the PPTT table >> * @cpu_node: processor node we wish to count caches for >> * >> @@ -235,7 +235,7 @@ static int acpi_pptt_leaf_node(struct acpi_table_header *table_hdr, >> /** >> * acpi_find_processor_node() - Given a PPTT table find the requested cessor >> * @table_hdr: Pointer to the head of the PPTT table >> - * @acpi_cpu_id: cpu we are searching for >> + * @acpi_cpu_id: CPU we are searching for >> * >> * Find the subtable entry describing the provided processor. >> * This is done by iterating the PPTT table looking for processor nodes >> @@ -456,21 +456,21 @@ static struct acpi_pptt_processor *acpi_find_processor_package_id(struct acpi_ta >> >> static void acpi_pptt_warn_missing(void) >> { >> - pr_warn_once("No PPTT table found, cpu and cache topology may be inaccurate\n"); >> + pr_warn_once("No PPTT table found, CPU and cache topology may be inaccurate\n"); >> } >> >> /** >> * topology_get_acpi_cpu_tag() - Find a unique topology value for a feature >> * @table: Pointer to the head of the PPTT table >> - * @cpu: Kernel logical cpu number >> + * @cpu: Kernel logical CPU number >> * @level: A level that terminates the search >> * @flag: A flag which terminates the search >> * >> - * Get a unique value given a cpu, and a topology level, that can be >> - * matched to determine which cpus share common topological features >> + * Get a unique value given a CPU, and a topology level, that can be >> + * matched to determine which CPUs share common topological features >> * at that level. >> * >> - * Return: Unique value, or -ENOENT if unable to locate cpu >> + * Return: Unique value, or -ENOENT if unable to locate CPU >> */ >> static int topology_get_acpi_cpu_tag(struct acpi_table_header *table, >> unsigned int cpu, int level, int flag) >> @@ -510,7 +510,7 @@ static int find_acpi_cpu_topology_tag(unsigned int cpu, int level, int flag) >> return -ENOENT; >> } >> retval = topology_get_acpi_cpu_tag(table, cpu, level, flag); >> - pr_debug("Topology Setup ACPI cpu %d, level %d ret = %d\n", >> + pr_debug("Topology Setup ACPI CPU %d, level %d ret = %d\n", >> cpu, level, retval); >> acpi_put_table(table); >> >> @@ -519,9 +519,9 @@ static int find_acpi_cpu_topology_tag(unsigned int cpu, int level, int flag) >> >> /** >> * acpi_find_last_cache_level() - Determines the number of cache levels for a PE >> - * @cpu: Kernel logical cpu number >> + * @cpu: Kernel logical CPU number >> * >> - * Given a logical cpu number, returns the number of levels of cache represented >> + * Given a logical CPU number, returns the number of levels of cache represented >> * in the PPTT. Errors caused by lack of a PPTT table, or otherwise, return 0 >> * indicating we didn't find any cache levels. >> * >> @@ -534,7 +534,7 @@ int acpi_find_last_cache_level(unsigned int cpu) >> int number_of_levels = 0; >> acpi_status status; >> >> - pr_debug("Cache Setup find last level cpu=%d\n", cpu); >> + pr_debug("Cache Setup find last level CPU=%d\n", cpu); >> >> acpi_cpu_id = get_acpi_id_for_cpu(cpu); >> status = acpi_get_table(ACPI_SIG_PPTT, 0, &table); >> @@ -551,14 +551,14 @@ int acpi_find_last_cache_level(unsigned int cpu) >> >> /** >> * cache_setup_acpi() - Override CPU cache topology with data from the PPTT >> - * @cpu: Kernel logical cpu number >> + * @cpu: Kernel logical CPU number >> * >> * Updates the global cache info provided by cpu_get_cacheinfo() >> * when there are valid properties in the acpi_pptt_cache nodes. A >> * successful parse may not result in any updates if none of the >> * cache levels have any valid flags set. Futher, a unique value is >> * associated with each known CPU cache entry. This unique value >> - * can be used to determine whether caches are shared between cpus. >> + * can be used to determine whether caches are shared between CPUs. >> * >> * Return: -ENOENT on failure to find table, or 0 on success >> */ >> @@ -567,7 +567,7 @@ int cache_setup_acpi(unsigned int cpu) >> struct acpi_table_header *table; >> acpi_status status; >> >> - pr_debug("Cache Setup ACPI cpu %d\n", cpu); >> + pr_debug("Cache Setup ACPI CPU %d\n", cpu); >> >> status = acpi_get_table(ACPI_SIG_PPTT, 0, &table); >> if (ACPI_FAILURE(status)) { >> @@ -582,8 +582,8 @@ int cache_setup_acpi(unsigned int cpu) >> } >> >> /** >> - * find_acpi_cpu_topology() - Determine a unique topology value for a given cpu >> - * @cpu: Kernel logical cpu number >> + * find_acpi_cpu_topology() - Determine a unique topology value for a given CPU >> + * @cpu: Kernel logical CPU number >> * @level: The topological level for which we would like a unique ID >> * >> * Determine a topology unique ID for each thread/core/cluster/mc_grouping >> @@ -596,7 +596,7 @@ int cache_setup_acpi(unsigned int cpu) >> * other levels beyond this use a generated value to uniquely identify >> * a topological feature. >> * >> - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. >> + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. >> * Otherwise returns a value which represents a unique topological feature. >> */ >> int find_acpi_cpu_topology(unsigned int cpu, int level) >> @@ -606,12 +606,12 @@ int find_acpi_cpu_topology(unsigned int cpu, int level) >> >> /** >> * find_acpi_cpu_cache_topology() - Determine a unique cache topology value >> - * @cpu: Kernel logical cpu number >> + * @cpu: Kernel logical CPU number >> * @level: The cache level for which we would like a unique ID >> * >> * Determine a unique ID for each unified cache in the system >> * >> - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. >> + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. >> * Otherwise returns a value which represents a unique topological feature. >> */ >> int find_acpi_cpu_cache_topology(unsigned int cpu, int level) >> @@ -643,17 +643,17 @@ int find_acpi_cpu_cache_topology(unsigned int cpu, int level) >> >> >> /** >> - * find_acpi_cpu_topology_package() - Determine a unique cpu package value >> - * @cpu: Kernel logical cpu number >> + * find_acpi_cpu_topology_package() - Determine a unique CPU package value >> + * @cpu: Kernel logical CPU number >> * >> - * Determine a topology unique package ID for the given cpu. >> + * Determine a topology unique package ID for the given CPU. >> * This ID can then be used to group peers, which will have matching ids. >> * >> * The search terminates when either a level is found with the PHYSICAL_PACKAGE >> * flag set or we reach a root node. >> * >> - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. >> - * Otherwise returns a value which represents the package for this cpu. >> + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. >> + * Otherwise returns a value which represents the package for this CPU. >> */ >> int find_acpi_cpu_topology_package(unsigned int cpu) >> { >> -- >> 2.20.1 >>
On Wednesday, May 1, 2019 5:46:49 PM CEST Jeremy Linton wrote: > Hi, > > And sorry about the delay... > > On 4/29/19 3:50 AM, Rafael J. Wysocki wrote: > > On Sat, Apr 27, 2019 at 12:03 AM Jeremy Linton <jeremy.linton@arm.com> wrote: > >> > >> CPU is an acronym and customarily capitalized. Much of the > >> commenting in the PPTT code has been using "cpu" rather > >> than "CPU". Correct that, and other human readable strings. > >> > >> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com> > > > > It would be better to say "Modify the spelling of CPU" in the subject > > IMO, but apart from that > > Hmmm, spelling doesn't seem quite right either, how about "capitalization"? That would work for me too.
diff --git a/drivers/acpi/pptt.c b/drivers/acpi/pptt.c index 065c4fc245d1..83a026765faa 100644 --- a/drivers/acpi/pptt.c +++ b/drivers/acpi/pptt.c @@ -164,7 +164,7 @@ static struct acpi_pptt_cache *acpi_find_cache_level(struct acpi_table_header *t } /** - * acpi_count_levels() - Given a PPTT table, and a cpu node, count the caches + * acpi_count_levels() - Given a PPTT table, and a CPU node, count the caches * @table_hdr: Pointer to the head of the PPTT table * @cpu_node: processor node we wish to count caches for * @@ -235,7 +235,7 @@ static int acpi_pptt_leaf_node(struct acpi_table_header *table_hdr, /** * acpi_find_processor_node() - Given a PPTT table find the requested processor * @table_hdr: Pointer to the head of the PPTT table - * @acpi_cpu_id: cpu we are searching for + * @acpi_cpu_id: CPU we are searching for * * Find the subtable entry describing the provided processor. * This is done by iterating the PPTT table looking for processor nodes @@ -456,21 +456,21 @@ static struct acpi_pptt_processor *acpi_find_processor_package_id(struct acpi_ta static void acpi_pptt_warn_missing(void) { - pr_warn_once("No PPTT table found, cpu and cache topology may be inaccurate\n"); + pr_warn_once("No PPTT table found, CPU and cache topology may be inaccurate\n"); } /** * topology_get_acpi_cpu_tag() - Find a unique topology value for a feature * @table: Pointer to the head of the PPTT table - * @cpu: Kernel logical cpu number + * @cpu: Kernel logical CPU number * @level: A level that terminates the search * @flag: A flag which terminates the search * - * Get a unique value given a cpu, and a topology level, that can be - * matched to determine which cpus share common topological features + * Get a unique value given a CPU, and a topology level, that can be + * matched to determine which CPUs share common topological features * at that level. * - * Return: Unique value, or -ENOENT if unable to locate cpu + * Return: Unique value, or -ENOENT if unable to locate CPU */ static int topology_get_acpi_cpu_tag(struct acpi_table_header *table, unsigned int cpu, int level, int flag) @@ -510,7 +510,7 @@ static int find_acpi_cpu_topology_tag(unsigned int cpu, int level, int flag) return -ENOENT; } retval = topology_get_acpi_cpu_tag(table, cpu, level, flag); - pr_debug("Topology Setup ACPI cpu %d, level %d ret = %d\n", + pr_debug("Topology Setup ACPI CPU %d, level %d ret = %d\n", cpu, level, retval); acpi_put_table(table); @@ -519,9 +519,9 @@ static int find_acpi_cpu_topology_tag(unsigned int cpu, int level, int flag) /** * acpi_find_last_cache_level() - Determines the number of cache levels for a PE - * @cpu: Kernel logical cpu number + * @cpu: Kernel logical CPU number * - * Given a logical cpu number, returns the number of levels of cache represented + * Given a logical CPU number, returns the number of levels of cache represented * in the PPTT. Errors caused by lack of a PPTT table, or otherwise, return 0 * indicating we didn't find any cache levels. * @@ -534,7 +534,7 @@ int acpi_find_last_cache_level(unsigned int cpu) int number_of_levels = 0; acpi_status status; - pr_debug("Cache Setup find last level cpu=%d\n", cpu); + pr_debug("Cache Setup find last level CPU=%d\n", cpu); acpi_cpu_id = get_acpi_id_for_cpu(cpu); status = acpi_get_table(ACPI_SIG_PPTT, 0, &table); @@ -551,14 +551,14 @@ int acpi_find_last_cache_level(unsigned int cpu) /** * cache_setup_acpi() - Override CPU cache topology with data from the PPTT - * @cpu: Kernel logical cpu number + * @cpu: Kernel logical CPU number * * Updates the global cache info provided by cpu_get_cacheinfo() * when there are valid properties in the acpi_pptt_cache nodes. A * successful parse may not result in any updates if none of the * cache levels have any valid flags set. Futher, a unique value is * associated with each known CPU cache entry. This unique value - * can be used to determine whether caches are shared between cpus. + * can be used to determine whether caches are shared between CPUs. * * Return: -ENOENT on failure to find table, or 0 on success */ @@ -567,7 +567,7 @@ int cache_setup_acpi(unsigned int cpu) struct acpi_table_header *table; acpi_status status; - pr_debug("Cache Setup ACPI cpu %d\n", cpu); + pr_debug("Cache Setup ACPI CPU %d\n", cpu); status = acpi_get_table(ACPI_SIG_PPTT, 0, &table); if (ACPI_FAILURE(status)) { @@ -582,8 +582,8 @@ int cache_setup_acpi(unsigned int cpu) } /** - * find_acpi_cpu_topology() - Determine a unique topology value for a given cpu - * @cpu: Kernel logical cpu number + * find_acpi_cpu_topology() - Determine a unique topology value for a given CPU + * @cpu: Kernel logical CPU number * @level: The topological level for which we would like a unique ID * * Determine a topology unique ID for each thread/core/cluster/mc_grouping @@ -596,7 +596,7 @@ int cache_setup_acpi(unsigned int cpu) * other levels beyond this use a generated value to uniquely identify * a topological feature. * - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. * Otherwise returns a value which represents a unique topological feature. */ int find_acpi_cpu_topology(unsigned int cpu, int level) @@ -606,12 +606,12 @@ int find_acpi_cpu_topology(unsigned int cpu, int level) /** * find_acpi_cpu_cache_topology() - Determine a unique cache topology value - * @cpu: Kernel logical cpu number + * @cpu: Kernel logical CPU number * @level: The cache level for which we would like a unique ID * * Determine a unique ID for each unified cache in the system * - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. * Otherwise returns a value which represents a unique topological feature. */ int find_acpi_cpu_cache_topology(unsigned int cpu, int level) @@ -643,17 +643,17 @@ int find_acpi_cpu_cache_topology(unsigned int cpu, int level) /** - * find_acpi_cpu_topology_package() - Determine a unique cpu package value - * @cpu: Kernel logical cpu number + * find_acpi_cpu_topology_package() - Determine a unique CPU package value + * @cpu: Kernel logical CPU number * - * Determine a topology unique package ID for the given cpu. + * Determine a topology unique package ID for the given CPU. * This ID can then be used to group peers, which will have matching ids. * * The search terminates when either a level is found with the PHYSICAL_PACKAGE * flag set or we reach a root node. * - * Return: -ENOENT if the PPTT doesn't exist, or the cpu cannot be found. - * Otherwise returns a value which represents the package for this cpu. + * Return: -ENOENT if the PPTT doesn't exist, or the CPU cannot be found. + * Otherwise returns a value which represents the package for this CPU. */ int find_acpi_cpu_topology_package(unsigned int cpu) {
CPU is an acronym and customarily capitalized. Much of the commenting in the PPTT code has been using "cpu" rather than "CPU". Correct that, and other human readable strings. Signed-off-by: Jeremy Linton <jeremy.linton@arm.com> --- drivers/acpi/pptt.c | 48 ++++++++++++++++++++++----------------------- 1 file changed, 24 insertions(+), 24 deletions(-)