diff options
author | David S. Miller <davem@davemloft.net> | 2008-12-05 17:06:47 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-12-05 17:06:47 -0800 |
commit | 59966e3b4e9e6573b352301ad0ceed15196d4e2d (patch) | |
tree | 39f8da655c71208c1c03bc292b2980d7e1c70fd9 | |
parent | b9e5567cda6387e0f4061bfceebbc1bd63ee42a6 (diff) | |
download | op-kernel-dev-59966e3b4e9e6573b352301ad0ceed15196d4e2d.zip op-kernel-dev-59966e3b4e9e6573b352301ad0ceed15196d4e2d.tar.gz |
sparc: Commonize get_one_property() implementations.
Add final len assignment in sparc64's get_one_property() (it's necessary
to avoid unchecked return value warnings on the sparc32 side),
and mark name argument const on sparc32's copy.
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | arch/sparc/kernel/prom_32.c | 2 | ||||
-rw-r--r-- | arch/sparc/kernel/prom_64.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/arch/sparc/kernel/prom_32.c b/arch/sparc/kernel/prom_32.c index a9859d2..a6809bc 100644 --- a/arch/sparc/kernel/prom_32.c +++ b/arch/sparc/kernel/prom_32.c @@ -196,7 +196,7 @@ static char * __init build_full_name(struct device_node *dp) return n; } -static char * __init get_one_property(phandle node, char *name) +static char * __init get_one_property(phandle node, const char *name) { char *buf = "<NULL>"; int len; diff --git a/arch/sparc/kernel/prom_64.c b/arch/sparc/kernel/prom_64.c index e3969db..ce96c1c 100644 --- a/arch/sparc/kernel/prom_64.c +++ b/arch/sparc/kernel/prom_64.c @@ -407,7 +407,7 @@ static char * __init get_one_property(phandle node, const char *name) len = prom_getproplen(node, name); if (len > 0) { buf = prom_early_alloc(len); - prom_getproperty(node, name, buf, len); + len = prom_getproperty(node, name, buf, len); } return buf; |