From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-bw0-f171.google.com (mail-bw0-f171.google.com [209.85.218.171]) by ozlabs.org (Postfix) with ESMTP id 73924DE14C for ; Fri, 17 Apr 2009 17:16:52 +1000 (EST) Received: by bwz19 with SMTP id 19so631489bwz.9 for ; Fri, 17 Apr 2009 00:16:50 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20090416220451.GA8636@oksana.dev.rtsoft.ru> References: <200904162130.01237.bzolnier@gmail.com> <20090416205735.GA26519@oksana.dev.rtsoft.ru> <20090416220451.GA8636@oksana.dev.rtsoft.ru> Date: Fri, 17 Apr 2009 09:16:49 +0200 Message-ID: Subject: Re: RFC Patch: Use x86 init_hwif in the alim15x3 for x86-like PowerPC systems From: Roderick Colenbrander To: avorontsov@ru.mvista.com Content-Type: text/plain; charset=ISO-8859-1 Cc: linux-ide@vger.kernel.org, Bartlomiej Zolnierkiewicz , linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Apr 17, 2009 at 12:04 AM, Anton Vorontsov wrote: > On Thu, Apr 16, 2009 at 11:27:34PM +0200, Roderick Colenbrander wrote: >> On Thu, Apr 16, 2009 at 10:57 PM, Anton Vorontsov >> wrote: >> > On Thu, Apr 16, 2009 at 09:30:00PM +0200, Bartlomiej Zolnierkiewicz wr= ote: >> >> >> >> Hi, >> >> >> >> On Wednesday 15 April 2009 16:34:22 Roderick Colenbrander wrote: >> >> > Hi, >> >> > >> >> > I'm using a Xilinx ML510 it features a PowerPC 440 cpu inside a >> >> > Virtex-5 FPGA. The board also contains a ALI M1533 south bridge >> >> > for IDE, USB and Audio. I did a lot of work to get the pci bus work= ing >> >> > on this board and it works correctly but the default init code >> >> > of the alim15x3 driver doesn't work for me. The driver explicitly >> >> > disabled some initialization code for powerpc after uncommenting th= is >> >> > code it works properly. Benjamin Herrenschmidt and I think this >> >> > !CONFIG_PPC check should be removed because the system behaves >> >> > like a real 'x86' system (also the i8259 interrupt controller is us= ed). >> >> >> >> Ben, I guess you are OK with the change and there are no longer other >> >> platforms requiring CONFIG_PPC check below? =A0[I don't see your ACK = here] >> > >> > I'd like to nak this patch. It'll break MPC8610HPCD boards. >> > >> > See commit 6d1cee44361b8d06ccd1812e80448d86ae60dfe3 >> > >> > I'm not sure how should we handle this for Xilinx PPC, MPC8610 >> > PPC and Sparc all at the same time, but it's obvious that #ifdef >> > isn't enough any longer... >> > >> > >> > >> >> If it is really a board specific issue (in case of powerpc it appears >> to be one) we could also use the default_irq override in ppc_md. What >> do you think? > > Um, I don't see "default_irq" in the ppc_md. Do you want to add one? > > I wonder if DECLARE_PCI_FIXUP_*() mechanism will work... see > arch/powerpc/platforms/fsl_uli1575.c as an example for board-specific > PCI fixups. It seems default_irq was the name on old kernels. These days it is called pci_get_legacy_ide_irq. Roderick