From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from fed1rmmtao02.cox.net (fed1rmmtao02.cox.net [68.230.241.37]) by ozlabs.org (Postfix) with ESMTP id B965D67A6C for ; Fri, 1 Apr 2005 00:28:26 +1000 (EST) Date: Thu, 31 Mar 2005 07:28:13 -0700 From: Tom Rini To: Sven Luther Message-ID: <20050331142813.GH25923@smtp.west.cox.net> References: <20050328232355.GA1982@pegasos> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20050328232355.GA1982@pegasos> Cc: linuxppc-dev@ozlabs.org Subject: Re: PREP sym53c8xx sym53c8xx brokeness due to 2.6.9-rc1-bk1 introduced residual data patch ... List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, Mar 29, 2005 at 01:23:55AM +0200, Sven Luther wrote: > Hello, > > I have been investigating the prep breakage of the sym53c8xx driver on my > powerstack II in recent kernels, and traced it to the residual data patch, > or more exactly to the whole prep_pci.c/prep_setup.c patches that where > introduced between 2.6.9-rc1 and 2.6.9-rc1-bk1. I didn't have time to find > more details as the patch is not so small, and seems to do many things, and > linux.bkbits.net seems done right now. > > Mmm, after a bit more of investigation, the changeset breaking it is : > > # This is a BitKeeper generated diff -Nru style patch. > # > # ChangeSet > # 2004/08/16 10:35:18-07:00 trini@kernel.crashing.org > # ppc32: On PReP, use residual data for PCI dev -> IRQ, and use it. > # > # Signed-off-by: Leigh Brown > # Signed-off-by: Tom Rini > # > # include/asm-ppc/residual.h > # 2004/08/16 10:35:09-07:00 trini@kernel.crashing.org +7 -0 > # This adds a function to use the residual data to determine the IRQ > # for a given PCI device, and changes prep_pcibios_fixup() to use it. > # > # arch/ppc/platforms/residual.c > # 2004/08/16 10:35:09-07:00 trini@kernel.crashing.org +60 -0 > # This adds a function to use the residual data to determine the IRQ > # for a given PCI device, and changes prep_pcibios_fixup() to use it. > # > # arch/ppc/platforms/prep_pci.c > # 2004/08/16 10:35:09-07:00 trini@kernel.crashing.org +37 -23 > # This adds a function to use the residual data to determine the IRQ > # for a given PCI device, and changes prep_pcibios_fixup() to use it. > > Mmm, i guess this makes sense, since it seems the sym53c8xx doesn'y find its > irq anymore or something. Tom Rini, or Leigh Brown, do you have any comments > on this one ? Well, we'd need to be certain it's this patch. If it's this patch, I think Leigh was suggesting at the time that the fix would be to fixup the residual data with the correct information. But what also went in were changes to specific irq maps, based on what fixed a given machine for someone else, so one of those changes could also be it. -- Tom Rini http://gate.crashing.org/~trini/