From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ag-out-0708.google.com (ag-out-0708.google.com [72.14.246.246]) by ozlabs.org (Postfix) with ESMTP id 68898DDF1B for ; Sun, 29 Jun 2008 05:16:18 +1000 (EST) Received: by ag-out-0708.google.com with SMTP id 8so4148371agc.0 for ; Sat, 28 Jun 2008 12:16:17 -0700 (PDT) Message-ID: Date: Sat, 28 Jun 2008 13:16:16 -0600 From: "Grant Likely" Sender: glikely@secretlab.ca To: "Wolfram Sang" Subject: Re: [PATCH] Fix wrong 'no interrupt' handling in of_i2c In-Reply-To: <20080628183141.15298.2758.stgit@octopus.labnet.pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 References: <20080628183141.15298.2758.stgit@octopus.labnet.pengutronix.de> Cc: linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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 -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd.