Re: [Powerpc-utils-devel] [PATCH] PowerKVM is a distro, not a platform

[Michael E. <mp...@el...>]

On Fri, 2015-06-05 at 11:30 +0200, Dinar valeev wrote:
> From: Dinar Valeev <dv...@su...>
> 
> Name OPAL platform correctly.. Bare metal systems are not
> PowerKVM (which is IBM distro), but PowerNV.

Did you mean to use:

 PLATFORM_POWERNV
 PLATFORM_POWERKNV
 PLATFORM_POWERNVM

?

> diff --git a/scripts/update_flash b/scripts/update_flash
> index a1f2bbb..a38b776 100755
> --- a/scripts/update_flash
> +++ b/scripts/update_flash
> @@ -414,7 +414,7 @@ case "$platform" in
>      $PLATFORM_UNKNOWN | $PLATFORM_POWERKVM_GUEST)
>  	echo "update_flash: is not supported on the $platform_name platform"
>  	exit 1;;
> -    $PLATFORM_POWERKVM_HOST)
> +    $PLATFORM_POWERKNV)
>  	if [ ! -r "$UPDATE_FLASH_NV" ]; then
>  		error $E_PERM "Couldn't find $UPDATE_FLASH_NV file."
>  	fi

> diff --git a/src/set_poweron_time.c b/src/set_poweron_time.c
> index 32bab76..3caf1b5 100644
> --- a/src/set_poweron_time.c
> +++ b/src/set_poweron_time.c
> @@ -221,7 +221,7 @@ int main(int argc, char **argv) {
>  
>  	switch (get_platform()) {
>  	case PLATFORM_UNKNOWN:
> -	case PLATFORM_POWERKVM_HOST:
> +	case PLATFORM_POWERNVM:
>  		fprintf(stderr, "%s: is not supported on the %s platform\n",
>  							argv[0], platform_name);
>  		return 1;


cheers