Re: [PATCH v2 4/5] ARC: MCIP: Set an initial affinity value in idu_irq_map

2016-10-26 Thread Vineet Gupta
On 10/26/2016 09:36 AM, Marc Zyngier wrote: > On Wed, Oct 26 2016 at 05:17:34 PM, Vineet Gupta > wrote: >> On 10/26/2016 07:05 AM, Marc Zyngier wrote: >>> It definitely feels weird to encode the interrupt affinity in the DT >>> (the kernel and possible userspace usually know much better than the

Re: [PATCH v2 4/5] ARC: MCIP: Set an initial affinity value in idu_irq_map

2016-10-26 Thread Marc Zyngier
On Wed, Oct 26 2016 at 05:17:34 PM, Vineet Gupta wrote: > On 10/26/2016 07:05 AM, Marc Zyngier wrote: >> It definitely feels weird to encode the interrupt affinity in the DT >> (the kernel and possible userspace usually know much better than the >> firmware). What is the actual reason for storing

Re: [PATCH v2 4/5] ARC: MCIP: Set an initial affinity value in idu_irq_map

2016-10-26 Thread Vineet Gupta
On 10/26/2016 07:05 AM, Marc Zyngier wrote: > It definitely feels weird to encode the interrupt affinity in the DT > (the kernel and possible userspace usually know much better than the > firmware). What is the actual reason for storing the affinity there? The IDU intc supports various interrupt d

Re: [PATCH v2 4/5] ARC: MCIP: Set an initial affinity value in idu_irq_map

2016-10-26 Thread Marc Zyngier
On 25/10/16 19:28, Vineet Gupta wrote: > On 10/24/2016 05:46 AM, Yuriy Kolerov wrote: >> Originally an initial distribution mode (its value resides in Device Tree) >> for each common interrupt is set in idu_irq_xlate. This leads to the >> following problems. idu_irq_xlate may be called several time

Re: [PATCH v2 4/5] ARC: MCIP: Set an initial affinity value in idu_irq_map

2016-10-25 Thread Vineet Gupta
On 10/24/2016 05:46 AM, Yuriy Kolerov wrote: > Originally an initial distribution mode (its value resides in Device Tree) > for each common interrupt is set in idu_irq_xlate. This leads to the > following problems. idu_irq_xlate may be called several times during parsing > of the Device Tree and it

[PATCH v2 4/5] ARC: MCIP: Set an initial affinity value in idu_irq_map

2016-10-24 Thread Yuriy Kolerov
Originally an initial distribution mode (its value resides in Device Tree) for each common interrupt is set in idu_irq_xlate. This leads to the following problems. idu_irq_xlate may be called several times during parsing of the Device Tree and it is semantically wrong to configure an initial distri