From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762837AbZDBThl (ORCPT ); Thu, 2 Apr 2009 15:37:41 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1759666AbZDBTgi (ORCPT ); Thu, 2 Apr 2009 15:36:38 -0400 Received: from mail-bw0-f169.google.com ([209.85.218.169]:44079 "EHLO mail-bw0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759794AbZDBTgf (ORCPT ); Thu, 2 Apr 2009 15:36:35 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to :mime-version:content-disposition:message-id:content-type :content-transfer-encoding; b=IXIidVm0d1bYWcyCFdGnHNuXwQ4tCHCNOwhTJGbduEuY+Lf4iLDqs0NCWd7ujapf4R X5BoGB7vF2y09D2ZXDUm30rLVj6ibcsYjGX6H6+fSwO4AAJLzWWkBXX34rxXXzUKVplJ F5tcv+23czS1ZfZFnO5mtlF3257cjH6c8necA= From: Bartlomiej Zolnierkiewicz To: Geert Uytterhoeven Subject: Re: [PATCH] ide: falconide/q40ide - Use __ide_mm_{in,out}sw() for data transfer Date: Thu, 2 Apr 2009 20:50:51 +0200 User-Agent: KMail/1.11.1 (Linux/2.6.29-next-20090401; KDE/4.2.1; i686; ; ) Cc: linux-ide@vger.kernel.org, Linux Kernel Development , "Linux/m68k" References: In-Reply-To: MIME-Version: 1.0 Content-Disposition: inline Message-Id: <200904022050.52002.bzolnier@gmail.com> Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday 01 April 2009, Geert Uytterhoeven wrote: > Both of commits f94116aeec7a299640dd692128e1d22178affa8d ("ide: cleanup > ") and 15a453a955f89f6545118770c669b52e925368bd ("ide: include > only when needed") break falconide: > > | Uniform Multi-Platform E-IDE driver > | ide: Falcon IDE controller > | Probing IDE interface ide0... > | hda: Sarge m68k, ATA DISK drive > | ide0 at 0xfff00000 on irq 15 (serialized) > | ide-gd driver 1.18 > | hda: max request size: 128KiB > | hda: 2118816 sectors (1084 MB) w/256KiB Cache, CHS=2102/16/63 > | hda:<4>hda: lost interrupt > > This happens because falconide relies on {in,out}sw() being redefined in > , as included by , which is no longer the case. > Use __ide_mm_{in,out}sw() from instead, just like > ide_{in,out}put_data() do. > > The same problem seems to exist in q40ide. > > Signed-off-by: Geert Uytterhoeven applied Many thanks for fixing this issue and the IRQF_SHARED one!