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

[Dinar V. <k0...@op...>]

On Fri, Jun 5, 2015 at 1:02 PM, Michael Ellerman <mp...@el...> wrote:
> 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
>
> ?
No, that was a typo :)

Thanks for review. Sent v2

>
>> 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
>
>