Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 09 Feb 2015 07:50:36 -0800
From:      Nathan Whitehorn <nwhitehorn@freebsd.org>
To:        "Bjoern A. Zeeb" <bz@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r278452 - head/sys/powerpc/pseries
Message-ID:  <54D8D74C.2000803@freebsd.org>
In-Reply-To: <201502091134.t19BYkaN090687@svn.freebsd.org>
References:  <201502091134.t19BYkaN090687@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Thank you  -- sorry for missing this!
-Nathan

On 02/09/15 03:34, Bjoern A. Zeeb wrote:
> Author: bz
> Date: Mon Feb  9 11:34:45 2015
> New Revision: 278452
> URL: https://svnweb.freebsd.org/changeset/base/278452
>
> Log:
>    Properly hide a variable under #ifdef as it is only used inside the
>    specific #ifdef block otherwise leaving an unused variable and breaking
>    other kernel builds.
>
> Modified:
>    head/sys/powerpc/pseries/platform_chrp.c
>
> Modified: head/sys/powerpc/pseries/platform_chrp.c
> ==============================================================================
> --- head/sys/powerpc/pseries/platform_chrp.c	Mon Feb  9 11:11:17 2015	(r278451)
> +++ head/sys/powerpc/pseries/platform_chrp.c	Mon Feb  9 11:34:45 2015	(r278452)
> @@ -124,9 +124,9 @@ chrp_probe(platform_t plat)
>   static int
>   chrp_attach(platform_t plat)
>   {
> +#ifdef __powerpc64__
>   	int i;
>   
> -#ifdef __powerpc64__
>   	/* XXX: check for /rtas/ibm,hypertas-functions? */
>   	if (!(mfmsr() & PSL_HV)) {
>   		struct mem_region *phys, *avail;
>




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?54D8D74C.2000803>