lkml.org 
[lkml]   [2016]   [Apr]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] mips: pistachio: Determine SoC revision during boot
From
Date
Hi Sergei

On 18/04/16 17:59, Sergei Shtylyov wrote:
> Hello.
>
> On 04/18/2016 05:24 PM, James Hartley wrote:
>
>> Now that there are different revisions of the Pistachio SoC
>> in circulation, add this information to the boot log to make
>> it easier for users to determine which hardware they have.
>>
>> Signed-off-by: James Hartley <james.hartley@imgtec.com>
>> Signed-off-by: Ionela Voinescu <ionela.voinescu@imgtec.com>
>>
>> diff --git a/arch/mips/pistachio/init.c b/arch/mips/pistachio/init.c
>> index 96ba2cc..48f8755 100644
>> --- a/arch/mips/pistachio/init.c
>> +++ b/arch/mips/pistachio/init.c
> [...]
>> @@ -24,9 +26,28 @@
>> #include <asm/smp-ops.h>
>> #include <asm/traps.h>
>>
>> +/*
>> + * Core revision register decoding
>> + * Bits 23 to 20: Major rev
>> + * Bits 15 to 8: Minor rev
>> + * Bits 7 to 0: Maintenance rev
>> + */
>> +#define PISTACHIO_CORE_REV_REG 0xB81483D0
>> +#define PISTACHIO_CORE_REV_A1 0x00100006
>> +#define PISTACHIO_CORE_REV_B0 0x00100106
>> +
>> const char *get_system_type(void)
>> {
>> - return "IMG Pistachio SoC";
>> + u32 core_rev;
>> +
>> + core_rev = __raw_readl((const void *)PISTACHIO_CORE_REV_REG);
>> +
>> + if (core_rev == PISTACHIO_CORE_REV_B0)
>> + return "IMG Pistachio SoC (B0)";
>> + else if (core_rev == PISTACHIO_CORE_REV_A1)
>> + return "IMG_Pistachio SoC (A1)";
>> + else
>> + return "IMG_Pistachio SoC";
>
> How about the *switch* instead?
Yes, that would be slightly more readable - I'll do that in V2.

Thanks for the review!

James.
>
> [...]
>
> MBR, Sergei
>

\
 
 \ /
  Last update: 2016-04-19 12:41    [W:0.045 / U:4.392 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site