Re: [PATCH 2/8] irqdomain: Take interrupt-parent property into accountif specified

From: Rob Herring
Date: Fri Aug 31 2012 - 09:58:59 EST


On 08/31/2012 04:44 AM, Lee Jones wrote:
> On Tue, Aug 14, 2012 at 10:19:08AM +0200, Linus Walleij wrote:
>> On Thu, Aug 9, 2012 at 5:53 PM, Lee Jones <lee.jones@xxxxxxxxxx> wrote:
>>
>>> irq_find_host() currently ignores the 'interrupt-parent' property
>>> even if it's specified in the Device Tree. Meaning that a node can
>>> match to a domain in its hierarchy even if it doesn't belong to it.
>>> By searching for the parent first using of_irq_find_parent() we
>>> insist that the 'interrupt-parent' property is taken into account
>>> ensuring a greater chance of returning the correct domain.
>>>
>>> CC: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
>>> CC: Grant Likely <grant.likely@xxxxxxxxxxxx>
>>> Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx>
>>
>> This (with 1/8) is the right solution. Thanks, and I think
>> Mark may want to look at this too since I recognize he asked
>> you to work in this direction.
>>
>> Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
>
> Rob, did this get your Ack too?

Acked-by: Rob Herring <rob.herring@xxxxxxxxxxx>

Rob

>
> Ben, Grant, Mark, would you like to take a look also?
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/