lkml.org 
[lkml]   [2013]   [Oct]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ARM: OMAP2+: wakeupgen: AM43x adaptation
Hi Tony,

On Wednesday 09 October 2013 02:54 AM, Tony Lindgren wrote:
> * Afzal Mohammed <afzal@ti.com> [130905 04:03]:

>> -#define MAX_NR_REG_BANKS 5
>> -#define MAX_IRQS 160
>> +/* maximum value correspond to that of AM43x */
>> +#define MAX_NR_REG_BANKS 7
>> +#define MAX_IRQS 224
>> +#define DEFAULT_NR_REG_BANKS 5
>> +#define DEFAULT_IRQS 160
>> #define WKG_MASK_ALL 0x00000000
>> #define WKG_UNMASK_ALL 0xffffffff
>> #define CPU_ENA_OFFSET 0x400
>
> How about define it like this to avoid updating things
> in multiple places for new SoCs:
>
> #define AM43X_NR_REG_BANKS 7
> #define MAX_NR_REG_BANKS AM43X_NR_REG_BANKS

Yes that is better

>> for (i = 0; i < irq_banks; i++) {
>> wakeupgen_writel(0, i, CPU0_ID);
>> - wakeupgen_writel(0, i, CPU1_ID);
>> + if (!am43x)
>> + wakeupgen_writel(0, i, CPU1_ID);
>> }
>
> Why not use soc_is_am43xx() directly here?

Ok, it has been changed in the new version.

Regards
Afzal


\
 
 \ /
  Last update: 2013-10-09 10:21    [W:0.047 / U:0.812 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site