Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752144Ab3FYNdZ (ORCPT ); Tue, 25 Jun 2013 09:33:25 -0400 Received: from mail-ie0-f169.google.com ([209.85.223.169]:46829 "EHLO mail-ie0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751953Ab3FYNdW (ORCPT ); Tue, 25 Jun 2013 09:33:22 -0400 MIME-Version: 1.0 In-Reply-To: <20130625132925.GC30827@ab42.lan> References: <20130530211944.05F4D3E0A90@localhost> <1370014348-21121-1-git-send-email-christian.ruppert@abilis.com> <20130531221814.534DF3E08FE@localhost> <51AC15FA.4060800@synopsys.com> <20130625132925.GC30827@ab42.lan> From: Grant Likely Date: Tue, 25 Jun 2013 14:33:02 +0100 X-Google-Sender-Auth: 8ckcBFkxuZFCICxNpotBy4emcrk Message-ID: Subject: Re: [PATCH V3] irqchip: Add TB10x interrupt controller driver To: Christian Ruppert Cc: Vineet Gupta , Thomas Gleixner , Rob Herring , Rob Landley , devicetree-discuss , "linux-doc@vger.kernel.org" , Linux Kernel Mailing List , Pierrick Hascoet Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1905 Lines: 39 On Tue, Jun 25, 2013 at 2:29 PM, Christian Ruppert wrote: > On Mon, Jun 03, 2013 at 10:51:06AM +0100, Grant Likely wrote: >> On Mon, Jun 3, 2013 at 5:05 AM, Vineet Gupta wrote: >> > On 06/01/2013 03:48 AM, Grant Likely wrote: >> >> If I were working on this system I'd drop the >> >> snps,arc700-intc node entirely and have a single abilis,tb10x-intc that >> >> encapsulated the properties of both (you would of course want to share >> >> handler functions for the 'normal' inputs without the custom features). >> >> That would eliminate the goofyness of listing 27 separate interrupts in >> >> the abilis,tb10x-ictl interrupts property. >> > >> > But how is this different from other systems with a primary in-core intc and a >> > cascaded external intc. How do they do it. I guess I need to read up more on this. >> >> Usually cascaded irq controllers have multiple irqs multiplexed onto a >> single irq on the parent controller. It's the 1:1 situation that makes >> this controller odd. > > You're right, this might be a bit confusing. The controller was mainly > designed as a compatibility layer between ARC770 built-in interrupts and > the rest of the system. > > Do you see a better way to drive this kind of hardware? Do you have any > other comments on the driver? > > Without this driver, arch/arc/plat-tb10x and related drivers will not > work and it would thus be good to have this in the kernel as quickly as > possible if there are no more issues with it. No, I don't have any other issues with it. It is unconventional, but the framework handles it fine the way you've set it up. g. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/