Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759032Ab3EBWh1 (ORCPT ); Thu, 2 May 2013 18:37:27 -0400 Received: from mail-bk0-f54.google.com ([209.85.214.54]:34144 "EHLO mail-bk0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751202Ab3EBWh0 (ORCPT ); Thu, 2 May 2013 18:37:26 -0400 Message-ID: <5182EAA0.9070208@gmail.com> Date: Fri, 03 May 2013 00:37:20 +0200 From: Sebastian Hesselbarth User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.12) Gecko/20130116 Icedove/10.0.12 MIME-Version: 1.0 To: Arnd Bergmann CC: Thomas Gleixner , Grant Likely , Rob Herring , Rob Landley , Russell King , Jason Cooper , Andrew Lunn , Thomas Petazzoni , Gregory Clement , Ezequiel Garcia , Jean-Francois Moine , devicetree-discuss@lists.ozlabs.org, linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] irqchip: add support for Marvell Orion SoCs References: <1367519104-19677-1-git-send-email-sebastian.hesselbarth@gmail.com> <5182E11E.5010103@gmail.com> <201305030009.57041.arnd@arndb.de> In-Reply-To: <201305030009.57041.arnd@arndb.de> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2084 Lines: 49 On 05/03/2013 12:09 AM, Arnd Bergmann wrote: > On Thursday 02 May 2013, Sebastian Hesselbarth wrote: >>> Just look at the various implementations in drivers/irqchip/ and find >>> out how similar they are. Moving code to drivers/irqchip/ does not >>> make an excuse for reestablishing the mess which was addressed by the >>> generic irq chip implementation. >>> >>> Can you - and that means all of you ARM folks - please get your gear >>> together and add the missing features to the generic irq chip >>> implementation? I'm not going to accept more of that OF/DT frenzy. >> >> So you are suggesting to have a "linux,generic-intc" or you want me >> to have "marvell,orion-intc" make use of generic irq chip again? >> >> The second is easy, the first will take me a while to think about >> proper DT properties how to encode mask/unmask/ack/.. availability >> and offsets. > > I think it should not be "linux,..." since the description can > well be OS independent. I don't have a good idea for a generic > name, but it's not very important. > > The main missing piece in the generic irqchip code at the moment > is support for the linear irqdomain. Once we have that, a lot of > code can be simplified significantly. Arnd, Thomas, I still don't get it completely. Are you requesting a full blown DT-enabled generic irq chip driver that can be setup by DT properties? Or is it just to have current generic irq chip play with linear irqdomain? If it is just linear irqdomain, then I can have a look at it now. I would first send v2 to allow the others to integrate Kirkwood, Orion5x, and Discovery Innovation. Then v3 will touch generic irq and merge that with orion irqchip. (@Jason C: Are you sure that I should merge dove and orion irqchip patches? I doubt that anything touching generic irq will not go through irq tree.) Sebastian -- 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/