On 29 December 2015 at 21:03, Maxim Uvarov <[email protected]> wrote:
> On 12/29/2015 13:31, [email protected] wrote:
>>
>> From: Hongbo Zhang <[email protected]>
>>
>> The values of cpu_arch_str were deleted due to codes reorganization, this
>> patch adds values to them again.
>>
>> Signed-off-by: Hongbo Zhang <[email protected]>
>> ---
>>   platform/linux-generic/arch/mips64/odp_sysinfo_parse.c  | 1 +
>>   platform/linux-generic/arch/powerpc/odp_sysinfo_parse.c | 1 +
>>   platform/linux-generic/arch/x86/odp_sysinfo_parse.c     | 1 +
>>   3 files changed, 3 insertions(+)
>>
>> diff --git a/platform/linux-generic/arch/mips64/odp_sysinfo_parse.c
>> b/platform/linux-generic/arch/mips64/odp_sysinfo_parse.c
>> index 8b430f5..3415516 100644
>> --- a/platform/linux-generic/arch/mips64/odp_sysinfo_parse.c
>> +++ b/platform/linux-generic/arch/mips64/odp_sysinfo_parse.c
>> @@ -16,6 +16,7 @@ int odp_cpuinfo_parser(FILE *file, odp_system_info_t
>> *sysinfo)
>>         int model = 0;
>>         int count = 2;
>>   +     strcpy(sysinfo->cpu_arch_str, "octeon");
>
>
> there are bunch of other mips based socs, not only octeon.
>

The string is the original value I deleted in previous patch.
It seemed in out previous codes we treated "octeon" as "mips", I also
though this may not good.

Then "mips" or "mips64" is good value?

> Maxim.
>
>>         while (fgets(str, sizeof(str), file) != NULL && count > 0) {
>>                 if (!mhz) {
>>                         pos = strstr(str, "BogoMIPS");
>> diff --git a/platform/linux-generic/arch/powerpc/odp_sysinfo_parse.c
>> b/platform/linux-generic/arch/powerpc/odp_sysinfo_parse.c
>> index 61714f8..5f4e69d 100644
>> --- a/platform/linux-generic/arch/powerpc/odp_sysinfo_parse.c
>> +++ b/platform/linux-generic/arch/powerpc/odp_sysinfo_parse.c
>> @@ -16,6 +16,7 @@ int odp_cpuinfo_parser(FILE *file, odp_system_info_t
>> *sysinfo)
>>         int model = 0;
>>         int count = 2;
>>   +     strcpy(sysinfo->cpu_arch_str, "powerpc");
>>         while (fgets(str, sizeof(str), file) != NULL && count > 0) {
>>                 if (!mhz) {
>>                         pos = strstr(str, "clock");
>> diff --git a/platform/linux-generic/arch/x86/odp_sysinfo_parse.c
>> b/platform/linux-generic/arch/x86/odp_sysinfo_parse.c
>> index 24187f0..02ec9ea 100644
>> --- a/platform/linux-generic/arch/x86/odp_sysinfo_parse.c
>> +++ b/platform/linux-generic/arch/x86/odp_sysinfo_parse.c
>> @@ -15,6 +15,7 @@ int odp_cpuinfo_parser(FILE *file, odp_system_info_t
>> *sysinfo)
>>         double ghz = 0.0;
>>         int id = 0;
>>   +     strcpy(sysinfo->cpu_arch_str, "x86");
>>         while (fgets(str, sizeof(str), file) != NULL && id <
>> MAX_CPU_NUMBER) {
>>                 pos = strstr(str, "model name");
>>                 if (pos) {
>
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to