diff mbox series

[v2,1/5] ACPI/PPTT: Trivial, Modify the case of CPU

Message ID 20190426220309.55654-2-jeremy.linton@arm.com
State Superseded
Headers show
Series arm64: SPE ACPI enablement | expand

Commit Message

Jeremy Linton April 26, 2019, 10:03 p.m. UTC
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(-)

-- 
2.20.1

Comments

Rafael J. Wysocki April 29, 2019, 8:50 a.m. UTC | #1
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

>
Jeremy Linton May 1, 2019, 3:46 p.m. UTC | #2
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

>>
Rafael J. Wysocki May 6, 2019, 12:07 p.m. UTC | #3
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 mbox series

Patch

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)
 {