Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754927Ab2HET5n (ORCPT ); Sun, 5 Aug 2012 15:57:43 -0400 Received: from mailrelay1.diasemi.com ([82.210.246.133]:38332 "EHLO mailrelay1.diasemi.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754883Ab2HET5W convert rfc822-to-8bit (ORCPT ); Sun, 5 Aug 2012 15:57:22 -0400 From: "Opensource [Anthony Olech]" To: Mark Brown , "Opensource [Anthony Olech]" CC: LKML , David Dajun Chen Subject: RE: [PATCH] regmap-irq: allow auto-allocated IRQs to be mapped Thread-Topic: [PATCH] regmap-irq: allow auto-allocated IRQs to be mapped Thread-Index: AQHNckG72kyxY08IiEqTgpBfEDN7qZdLobTw Date: Sun, 5 Aug 2012 19:57:19 +0000 Message-ID: <24DF37198A1E704D9811D8F72B87EB51032C30AC@NB-EX-MBX02.diasemi.com> References: <201208020805.q72858Yj003147@latitude.olech.com> <20120804105424.GB10523@opensource.wolfsonmicro.com> In-Reply-To: <20120804105424.GB10523@opensource.wolfsonmicro.com> Accept-Language: en-GB, de-DE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.1.50.207] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2073 Lines: 42 Hi Mark, I am not testing the da9052 driver, but the da9058 that I recently submitted. The problem is that the auto allocated "irq_base" comes back as zero. I repeat - in v3.4 the daa9058 driver work, but fails (without my patch) in v3.5 If my suggested fix is not acceptable, then is it possible to fix the bug in some other way? thanks, Tony Olech -----Original Message----- From: Mark Brown [mailto:broonie@opensource.wolfsonmicro.com] Sent: 04 August 2012 11:54 To: Opensource [Anthony Olech] Cc: LKML; Anthony Olech; David Dajun Chen Subject: Re: [PATCH] regmap-irq: allow auto-allocated IRQs to be mapped On Wed, Aug 01, 2012 at 07:05:15PM +0100, Anthony Olech wrote: > if the irq_base is set to -1 when calling regmap_add_irq_chip() then > allow the IRQ to be mapped even if the allocated irq_base is actually > zero. > This restores the behaviour seen in v3.4, and I assume that the > tidy-ups just made in v3.5 INADVERTENTLY introduce this change in > behaviour. Please pay MORE attention to the changelog - obviously there's no problem mapping automatically allocated IRQs, there's only any effect if they happen to GET allocated at zero. The only real issue I see with the current code is that if the user explicitly wants to statically allocate an IRQ range at zero they can't. The current intended behaviour is that we use a linear domain unless a positive IRQ base is specified, though we're not quite doing that right now as a transitional measure until drivers are updated. The current da9052 driver usage seems to have quite a few problems, I do recall having to fix some problems that make me doubt if it ever worked well. Looking at the code now I see it's using hard coded references to absolute IRQ numbers which is an issue... It should be being converted to use regmap_irq_get_virq(). -- 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/