powerpc: Coding style cleanups
While converting code to use for_each_node_by_type I noticed a number of coding style issues. Signed-off-by: Anton Blanchard <anton@samba.org> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
parent
94db7c5e14
commit
dfbe93a222
2 changed files with 17 additions and 10 deletions
|
@ -281,11 +281,11 @@ static void __init initialize_cache_info(void)
|
|||
for_each_node_by_type(np, "cpu") {
|
||||
num_cpus += 1;
|
||||
|
||||
/* We're assuming *all* of the CPUs have the same
|
||||
/*
|
||||
* We're assuming *all* of the CPUs have the same
|
||||
* d-cache and i-cache sizes... -Peter
|
||||
*/
|
||||
|
||||
if ( num_cpus == 1 ) {
|
||||
if (num_cpus == 1) {
|
||||
const u32 *sizep, *lsizep;
|
||||
u32 size, lsize;
|
||||
|
||||
|
@ -294,10 +294,13 @@ static void __init initialize_cache_info(void)
|
|||
sizep = of_get_property(np, "d-cache-size", NULL);
|
||||
if (sizep != NULL)
|
||||
size = *sizep;
|
||||
lsizep = of_get_property(np, "d-cache-block-size", NULL);
|
||||
lsizep = of_get_property(np, "d-cache-block-size",
|
||||
NULL);
|
||||
/* fallback if block size missing */
|
||||
if (lsizep == NULL)
|
||||
lsizep = of_get_property(np, "d-cache-line-size", NULL);
|
||||
lsizep = of_get_property(np,
|
||||
"d-cache-line-size",
|
||||
NULL);
|
||||
if (lsizep != NULL)
|
||||
lsize = *lsizep;
|
||||
if (sizep == 0 || lsizep == 0)
|
||||
|
@ -314,9 +317,12 @@ static void __init initialize_cache_info(void)
|
|||
sizep = of_get_property(np, "i-cache-size", NULL);
|
||||
if (sizep != NULL)
|
||||
size = *sizep;
|
||||
lsizep = of_get_property(np, "i-cache-block-size", NULL);
|
||||
lsizep = of_get_property(np, "i-cache-block-size",
|
||||
NULL);
|
||||
if (lsizep == NULL)
|
||||
lsizep = of_get_property(np, "i-cache-line-size", NULL);
|
||||
lsizep = of_get_property(np,
|
||||
"i-cache-line-size",
|
||||
NULL);
|
||||
if (lsizep != NULL)
|
||||
lsize = *lsizep;
|
||||
if (sizep == 0 || lsizep == 0)
|
||||
|
|
|
@ -709,7 +709,6 @@ static void __init parse_drconf_memory(struct device_node *memory)
|
|||
|
||||
static int __init parse_numa_properties(void)
|
||||
{
|
||||
struct device_node *cpu = NULL;
|
||||
struct device_node *memory;
|
||||
int default_nid = 0;
|
||||
unsigned long i;
|
||||
|
@ -732,6 +731,7 @@ static int __init parse_numa_properties(void)
|
|||
* each node to be onlined must have NODE_DATA etc backing it.
|
||||
*/
|
||||
for_each_present_cpu(i) {
|
||||
struct device_node *cpu;
|
||||
int nid;
|
||||
|
||||
cpu = of_get_cpu_node(i, NULL);
|
||||
|
@ -800,8 +800,9 @@ static int __init parse_numa_properties(void)
|
|||
}
|
||||
|
||||
/*
|
||||
* Now do the same thing for each MEMBLOCK listed in the ibm,dynamic-memory
|
||||
* property in the ibm,dynamic-reconfiguration-memory node.
|
||||
* Now do the same thing for each MEMBLOCK listed in the
|
||||
* ibm,dynamic-memory property in the
|
||||
* ibm,dynamic-reconfiguration-memory node.
|
||||
*/
|
||||
memory = of_find_node_by_path("/ibm,dynamic-reconfiguration-memory");
|
||||
if (memory)
|
||||
|
|
Loading…
Reference in a new issue