[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACRpkdYiSimzz7UuXyBioj-L=tE=F8cWmH3rh2wzv889n6qOWg@mail.gmail.com>
Date: Mon, 21 Jan 2013 22:44:09 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: Samuel Ortiz <sameo@...ux.intel.com>
Cc: linux-kernel@...r.kernel.org, Anmar Oueja <anmar.oueja@...aro.org>,
stable@...nel.org, Fabio Baltieri <fabio.baltieri@...aro.org>
Subject: Re: [PATCH v2] mfd: tc3589x: use simple irqdomain
On Mon, Jan 14, 2013 at 11:28 PM, Linus Walleij
<linus.walleij@...aro.org> wrote:
> On Wed, Jan 2, 2013 at 2:40 PM, Linus Walleij
> <linus.walleij@...ricsson.com> wrote:
>
>> This fixes a regression in the TC3589x driver introduced in
>> commit 15e27b1088245a2de3b7d09d39cd209212eb16af
>> "mfd: Provide the tc3589x with its own IRQ domain"
>>
>> If a system with a TC3589x expander is booted and a base
>> IRQ is passed from platform data, a legacy domain will
>> be used. However, since the Ux500 is now switched to use
>> SPARSE_IRQ, no descriptors get allocated on-the-fly,
>> and we get a crash.
>>
>> Fix this by switching to using the simple irqdomain that
>> will handle this uniformly and also allocates descriptors
>> explicitly.
>>
>> Also fix two small whitespace errors in the vicinity while
>> we're at it.
>>
>> Cc: stable@...nel.org
>> Acked-by: Lee Jones <lee.jones@...aro.org>
>> Signed-off-by: Linus Walleij <linus.walleij@...ricsson.com>
>> ---
>> ChangeLog v1->v2:
>> - Update commit message after comments from Lee, add ACK, no
>> changes to the patche per se.
>
> Ping on this regression fix patch as well...
Ping on this!
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists