From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH] irqchip: atmel-aic: fix irqdomain initialization Date: Tue, 11 Nov 2014 07:45:37 -0800 Message-ID: <7hioiloi5a.fsf@deeprootsystems.com> References: <20141110230301.GV4068@piout.net> <1415712816-9202-1-git-send-email-boris.brezillon@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <1415712816-9202-1-git-send-email-boris.brezillon@free-electrons.com> (Boris Brezillon's message of "Tue, 11 Nov 2014 14:33:36 +0100") Sender: linux-sh-owner@vger.kernel.org To: Boris Brezillon Cc: Kevin Cernekee , Alexandre Belloni , Thomas Gleixner , Jason Cooper , linux-sh@vger.kernel.org, Florian Fainelli , Ralf Baechle , Sergei Shtylyov , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Maxime Bizon , Jonas Gorski , linux-mips@linux-mips.org, Nicolas Ferre , Olof Johansson , Arnd Bergmann List-Id: devicetree@vger.kernel.org Boris Brezillon writes: > First of all IRQCHIP_SKIP_SET_WAKE is not a valid irq_gc_flags and thus > should not be passed as the last argument of > irq_alloc_domain_generic_chips. > > Then pass the correct handler (handle_fasteoi_irq) to > irq_alloc_domain_generic_chips instead of manually re-setting it in the > initialization loop. > > And eventually initialize default irq flags to the pseudo standard: > IRQ_REQUEST | IRQ_PROBE | IRQ_AUTOEN. > > Signed-off-by: Boris Brezillon > --- > Hello Kevin, > > This patch has not been tested yet but it should solve the issue you've > experienced with the IRQ_GC_BE_IO flag and the atmel-aic driver. > > I'll test it tomorrow and let you know if it actually works. Well, it at least boots on my xplained and my sama5d35-ek: Tested-by: Kevin Hilman I'll let Jason pick this up as a fix through his tree. Kevin