From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from yow.seanm.ca (toronto-hs-216-138-233-67.s-ip.magma.ca [216.138.233.67]) by ozlabs.org (Postfix) with SMTP id CD7CBDDF6E for ; Thu, 17 Jul 2008 08:15:59 +1000 (EST) Date: Wed, 16 Jul 2008 18:15:56 -0400 From: Sean MacLennan To: "Grant Likely" Subject: Re: [PATCH] Fix wrong 'no interrupt' handling in of_i2c Message-ID: <20080716181556.2974a1a8@lappy.seanm.ca> In-Reply-To: References: <20080628183141.15298.2758.stgit@octopus.labnet.pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Cc: linuxppc-dev@ozlabs.org, Wolfram Sang List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sat, 28 Jun 2008 13:16:16 -0600 "Grant Likely" wrote: > On Sat, Jun 28, 2008 at 12:31 PM, Wolfram Sang > wrote: > > If an I2C device node does not specify an interrupt, the .irq > > member of the board_info struct was set to -1. This caused crashes > > on following irq_dispose_mappings. Leave it NO_IRQ as returned from > > irq_of_parse_and_map. (Suggesting -1 as 'i2c-no-irq' used to be a > > bug in linux/i2c.h.) > > > > Signed-off-by: Wolfram Sang > > Acked-by: Grant Likely > I just hit this while trying to get the at24 driver working. So: Acked-by: Sean MacLennan Cheers, Sean