diff mbox

[v2,1/4] target-ppc: Fill in OpenFirmware names for some PowerPCCPU families

Message ID 1376606111-3518-2-git-send-email-afaerber@suse.de
State New
Headers show

Commit Message

Andreas Färber Aug. 15, 2013, 10:35 p.m. UTC 
Set the expected values for POWER7, POWER7+, POWER8 and POWER5+.
Note that POWER5+ and POWER7+ are intentionally lacking the '+', so the
lack of a POWER7P family constitutes no problem.

Signed-off-by: Andreas Färber <afaerber@suse.de>
---
 target-ppc/translate_init.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Alexey Kardashevskiy Sept. 10, 2013, 4:15 a.m. UTC | #1 
On 08/16/2013 08:35 AM, Andreas Färber wrote:
> Set the expected values for POWER7, POWER7+, POWER8 and POWER5+.
> Note that POWER5+ and POWER7+ are intentionally lacking the '+', so the
> lack of a POWER7P family constitutes no problem.
> 
> Signed-off-by: Andreas Färber <afaerber@suse.de>


Out of curiosity - is anything going to happen to this series? Is it
awaiting for someone's review? Just asking as it is quite old and nobody
seems to care :)
Alexey Kardashevskiy Sept. 16, 2013, 2:16 p.m. UTC | #2 
On 09/10/2013 02:15 PM, Alexey Kardashevskiy wrote:
> On 08/16/2013 08:35 AM, Andreas Färber wrote:
>> Set the expected values for POWER7, POWER7+, POWER8 and POWER5+.
>> Note that POWER5+ and POWER7+ are intentionally lacking the '+', so the
>> lack of a POWER7P family constitutes no problem.
>>
>> Signed-off-by: Andreas Färber <afaerber@suse.de>
> 
> 
> Out of curiosity - is anything going to happen to this series? Is it
> awaiting for someone's review? Just asking as it is quite old and nobody
> seems to care :)

Ping, anyone? Not sure if any of my mails even reaches maillists :-/
Alexey Kardashevskiy Sept. 25, 2013, 9:01 a.m. UTC | #3 
On 09/17/2013 12:16 AM, Alexey Kardashevskiy wrote:
> On 09/10/2013 02:15 PM, Alexey Kardashevskiy wrote:
>> On 08/16/2013 08:35 AM, Andreas Färber wrote:
>>> Set the expected values for POWER7, POWER7+, POWER8 and POWER5+.
>>> Note that POWER5+ and POWER7+ are intentionally lacking the '+', so the
>>> lack of a POWER7P family constitutes no problem.
>>>
>>> Signed-off-by: Andreas Färber <afaerber@suse.de>
>>
>>
>> Out of curiosity - is anything going to happen to this series? Is it
>> awaiting for someone's review? Just asking as it is quite old and nobody
>> seems to care :)
> 
> Ping, anyone? Not sure if any of my mails even reaches maillists :-/

Ping?

It conflicts with "[PATCH] pseries: Fix loading of little endian kernels"
posted today. It would be great to have either this series or the new patch
upstream...
Alexander Graf Sept. 30, 2013, 5:55 p.m. UTC | #4 
On 09/25/2013 11:01 AM, Alexey Kardashevskiy wrote:
> On 09/17/2013 12:16 AM, Alexey Kardashevskiy wrote:
>> On 09/10/2013 02:15 PM, Alexey Kardashevskiy wrote:
>>> On 08/16/2013 08:35 AM, Andreas Färber wrote:
>>>> Set the expected values for POWER7, POWER7+, POWER8 and POWER5+.
>>>> Note that POWER5+ and POWER7+ are intentionally lacking the '+', so the
>>>> lack of a POWER7P family constitutes no problem.
>>>>
>>>> Signed-off-by: Andreas Färber<afaerber@suse.de>
>>>
>>> Out of curiosity - is anything going to happen to this series? Is it
>>> awaiting for someone's review? Just asking as it is quite old and nobody
>>> seems to care :)
>> Ping, anyone? Not sure if any of my mails even reaches maillists :-/
> Ping?
>
> It conflicts with "[PATCH] pseries: Fix loading of little endian kernels"
> posted today. It would be great to have either this series or the new patch
> upstream...

Andreas is going to rework it, yes :).


Alex
Alexey Kardashevskiy Oct. 7, 2013, 1:59 p.m. UTC | #5 
On 10/01/2013 03:55 AM, Alexander Graf wrote:
> On 09/25/2013 11:01 AM, Alexey Kardashevskiy wrote:
>> On 09/17/2013 12:16 AM, Alexey Kardashevskiy wrote:
>>> On 09/10/2013 02:15 PM, Alexey Kardashevskiy wrote:
>>>> On 08/16/2013 08:35 AM, Andreas Färber wrote:
>>>>> Set the expected values for POWER7, POWER7+, POWER8 and POWER5+.
>>>>> Note that POWER5+ and POWER7+ are intentionally lacking the '+', so the
>>>>> lack of a POWER7P family constitutes no problem.
>>>>>
>>>>> Signed-off-by: Andreas Färber<afaerber@suse.de>
>>>>
>>>> Out of curiosity - is anything going to happen to this series? Is it
>>>> awaiting for someone's review? Just asking as it is quite old and nobody
>>>> seems to care :)
>>> Ping, anyone? Not sure if any of my mails even reaches maillists :-/
>> Ping?
>>
>> It conflicts with "[PATCH] pseries: Fix loading of little endian kernels"
>> posted today. It would be great to have either this series or the new patch
>> upstream...
> 
> Andreas is going to rework it, yes :).


Wheeeen? Not complaining or anything, just asking :)
diff mbox

Patch

diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c
index 13b290c..28ca447 100644
--- a/target-ppc/translate_init.c
+++ b/target-ppc/translate_init.c
@@ -7102,6 +7102,7 @@  POWERPC_FAMILY(POWER5P)(ObjectClass *oc, void *data)
     DeviceClass *dc = DEVICE_CLASS(oc);
     PowerPCCPUClass *pcc = POWERPC_CPU_CLASS(oc);
 
+    dc->fw_name = "PowerPC,POWER5";
     dc->desc = "POWER5+";
     pcc->init_proc = init_proc_power5plus;
     pcc->check_pow = check_pow_970FX;
@@ -7212,6 +7213,7 @@  POWERPC_FAMILY(POWER7)(ObjectClass *oc, void *data)
     DeviceClass *dc = DEVICE_CLASS(oc);
     PowerPCCPUClass *pcc = POWERPC_CPU_CLASS(oc);
 
+    dc->fw_name = "PowerPC,POWER7";
     dc->desc = "POWER7";
     pcc->init_proc = init_proc_POWER7;
     pcc->check_pow = check_pow_nocheck;
@@ -7246,6 +7248,7 @@  POWERPC_FAMILY(POWER8)(ObjectClass *oc, void *data)
     DeviceClass *dc = DEVICE_CLASS(oc);
     PowerPCCPUClass *pcc = POWERPC_CPU_CLASS(oc);
 
+    dc->fw_name = "PowerPC,POWER8";
     dc->desc = "POWER8";
     pcc->init_proc = init_proc_POWER7;
     pcc->check_pow = check_pow_nocheck;