This message generated a parse failure. Raw output follows here. Please use 'back' to navigate. From devnull@lkml.org Sat Jun 15 03:22:58 2024 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S272664AbTHRQpW (ORCPT ); Mon, 18 Aug 2003 12:45:22 -0400 Received: from [63.247.75.124] ([63.247.75.124]:26769 "EHLO havoc.gtf.org") by vger.kernel.org with ESMTP id S272568AbTHRQpN (ORCPT ); Mon, 18 Aug 2003 12:45:13 -0400 Received: by havoc.gtf.org (Postfix, from userid 500) id A77E36646; Mon, 18 Aug 2003 12:45:12 -0400 (EDT) Date: Mon, 18 Aug 2003 12:45:12 -0400 From: Jeff Garzik To: rth@twiddle.net Cc: linux-kernel@vger.kernel.org Subject: Re: [ALPHA] Update for "name" out of struct device. Message-Id: <20030818164512.GF24693@gtf.org> References: <200308181611.h7IGBEcW024487@hera.kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200308181611.h7IGBEcW024487@hera.kernel.org> User-Agent: Mutt/1.3.28i Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 18, 2003 at 05:24:13AM +0000, Linux Kernel Mailing List wrote: > --- a/arch/alpha/kernel/sys_marvel.c Mon Aug 18 09:11:18 2003 > +++ b/arch/alpha/kernel/sys_marvel.c Mon Aug 18 09:11:18 2003 > @@ -33,6 +33,13 @@ > +/* ??? Should probably be generic. */ > +#ifdef CONFIG_PCI_NAMES > +#define pci_pretty_name(x) ((x)->pretty_name) > +#else > +#define pci_pretty_name(x) "" > +#endif > + > @@ -378,7 +385,7 @@ > PCI_SLOT(dev->devfn), > PCI_FUNC(dev->devfn), > hose->index, > - dev->dev.name); > + pci_pretty_name (dev)); > printk(" %d message(s) from 0x%04x\n", hmmm, I think a better fix can be had... We store all that information. What do you think about the following patch? It follows the style of other PCI core messages, and prints out the same information as before. This assumes, of course, marvel_map_irq is called after pci_name() is assigned a value... Jeff ===== arch/alpha/kernel/sys_marvel.c 1.7 vs edited ===== --- 1.7/arch/alpha/kernel/sys_marvel.c Mon Aug 18 01:23:05 2003 +++ edited/arch/alpha/kernel/sys_marvel.c Mon Aug 18 12:43:51 2003 @@ -33,13 +33,6 @@ # error NR_IRQS < MARVEL_NR_IRQS !!! #endif -/* ??? Should probably be generic. */ -#ifdef CONFIG_PCI_NAMES -#define pci_pretty_name(x) ((x)->pretty_name) -#else -#define pci_pretty_name(x) "" -#endif - /* * Interrupt handling. @@ -380,12 +373,7 @@ irq += 0x80; /* offset for lsi */ #if 1 - printk("PCI:%d:%d:%d (hose %d) [%s] is using MSI\n", - dev->bus->number, - PCI_SLOT(dev->devfn), - PCI_FUNC(dev->devfn), - hose->index, - pci_pretty_name (dev)); + printk("PCI:%s is using MSI\n", pci_name (dev)); printk(" %d message(s) from 0x%04x\n", 1 << ((msg_ctl & PCI_MSI_FLAGS_QSIZE) >> 4), msg_dat); - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/