[PATCH] powerpc: correct CMO feature flag enablement

Robert Jennings rcj at linux.vnet.ibm.com
Wed Jul 16 07:20:08 EST 2008


Correct string conversion for rtas value being read for CMO configuration.
A value of -1 in the string indicates that CMO is not enabled and we
had used simple_strtoul rather than simple_strtol which caused problems.

Signed-off-by: Robert Jennings <rcj at linux.vnet.ibm.com>

---

This patch applies on top of the CMO patchset sent to the list.

---
 arch/powerpc/platforms/pseries/setup.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Index: b/arch/powerpc/platforms/pseries/setup.c
===================================================================
--- a/arch/powerpc/platforms/pseries/setup.c
+++ b/arch/powerpc/platforms/pseries/setup.c
@@ -366,9 +366,9 @@ void pSeries_cmo_feature_init(void)
 			}
 
 			if (0 == strcmp(key, "PrPSP"))
-				PrPSP = simple_strtoul(value, NULL, 10);
+				PrPSP = simple_strtol(value, NULL, 10);
 			else if (0 == strcmp(key, "SecPSP"))
-				SecPSP = simple_strtoul(value, NULL, 10);
+				SecPSP = simple_strtol(value, NULL, 10);
 			value = key = ptr + 1;
 		}
 		ptr++;



More information about the Linuxppc-dev mailing list