Re: [PATCH] powerpc: correct CMO feature flag enablement

[Brian King <brking@linux.vnet.ibm.com>]

Acked by: Brian King <brking@linux.vnet.ibm.com>

Robert Jennings wrote:
> 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@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++;


-- 
Brian King
Linux on Power Virtualization
IBM Linux Technology Center