Re: [PATCH 10/10] ARM: FIQ: Get rid of init_FIQ()

From: Anton Vorontsov
Date: Tue Nov 27 2012 - 04:09:02 EST


On Thu, Nov 22, 2012 at 11:51:38PM -0800, Anton Vorontsov wrote:
> On Fri, Nov 23, 2012 at 11:36:01AM +0400, Alexander Shiyan wrote:
> [...]
> > got_no_fiq_insn also not initialized.
>
> It is static, so by definition it is initialized to 0.
>
> > I think as a whole on this issue requires additional comments from Russell
> > as the author of implementation.
> >
> > PS: In any case, I can reproduce it and check it out if you send me a patchset
> > to my email as an attachment.
>
> Sent privately.

Andrew, Arnd, just FYI, it works for Alexander.

----- Forwarded message from Alexander Shiyan <shc_work@xxxxxxx> -----
Date: Tue, 27 Nov 2012 12:42:50 +0400
From: Alexander Shiyan <shc_work@xxxxxxx>
To: Anton Vorontsov <anton.vorontsov@xxxxxxxxxx>
Subject: Re[2]: FIQ

> Hello Alexander,
>
> On Tue, Nov 27, 2012 at 12:14:28PM +0400, Alexander Shiyan wrote:
> > Sorry, I lost the original message to the correct answer.
> > Today I had a test of removal init_FIQ.
> > The test took place on a custom board with a custom ARM CLPS711X OSS audio
> > driver, which uses the FIQ interrupts. Everything works fine.
>
> Nice! Thanks a lot for testing!
>
> Can you please reply to the original thread, with the
>
> Tested-by: Alexander Shiyan <shc_work@xxxxxxx>
> ?
>
> This will help pushing the series forward, plus will give you a credit for
> your efforts.

I am lost thread in my mailbox, you can just copy/paste my reply and ping for Arnd.

--
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/