Re: [PATCH] PCI Update for 2.6.3-rc1

[Karsten Keil]

Hi Greg,

On Tue, Feb 10, 2004 at 08:46:12AM -0800, Greg KH wrote:
> On Tue, Feb 10, 2004 at 05:03:17PM +0100, Geert Uytterhoeven wrote:
> > On Mon, 9 Feb 2004, Greg KH wrote:
> > > ChangeSet 1.1500.11.2, 2004/01/30 16:34:48-08:00, ambx1@xxxxxxxxxx
> > >
> > > [PATCH] PCI: Remove uneeded resource structures from pci_dev
> > >
> > > The following patch remove irq_resource and dma_resource from pci_dev.  It
> > > appears that the serial pci driver depends on irq_resource, however, it may be
> > > broken portions of an old quirk.  I attempted to maintain the existing behavior
> > > while removing irq_resource.  I changed FL_IRQRESOURCE to FL_NOIRQ.  Russell,
> > > could you provide any comments?  irq_resource and dma_resource are most likely
> > > remnants from when pci_dev was shared with pnp.
> > 
> > FYI, at least one ISDN driver seems to need it as well:
> > 
> > | drivers/isdn/hardware/avm/b1isa.c: In function `b1isa_init':
> > | drivers/isdn/hardware/avm/b1isa.c:183: structure has no member named `irq_resource'
> 
> Ick, I don't really think we want users trying to override the irq
> number of their pci cards...

No but this driver use the PCI struct to deliver IO/IRQ values
to the card init routines (used by ISA and PCI)
also for old (not PnP) ISA cards.

The bug is to use irq_resource[0].start here, it must be isa_dev[i].irq
instead, allready fixed in last night patch.

> 
> Here's the patch that fixes this, and one other isdn driver up.  ISDN
> people, feel free to add this to your huge patch :)
> 
> thanks,
> 
> greg k-h
> 
> 
> ===== b1isa.c 1.9 vs edited =====
> --- 1.9/drivers/isdn/hardware/avm/b1isa.c	Tue Jul 15 03:01:29 2003
> +++ edited/b1isa.c	Tue Feb 10 08:43:39 2004
> @@ -180,7 +180,6 @@
>  			break;
>  
>  		isa_dev[i].resource[0].start = io[i];
> -		isa_dev[i].irq_resource[0].start = irq[i];
>  
>  		if (b1isa_probe(&isa_dev[i]) == 0)
>  			found++;
> ===== t1isa.c 1.11 vs edited =====
> --- 1.11/drivers/isdn/hardware/avm/t1isa.c	Tue Jul 15 03:01:29 2003
> +++ edited/t1isa.c	Tue Feb 10 08:44:01 2004
> @@ -519,7 +519,6 @@
>  			break;
>  
>  		isa_dev[i].resource[0].start = io[i];
> -		isa_dev[i].irq_resource[0].start = irq[i];
>  
>  		if (t1isa_probe(&isa_dev[i], cardnr[i]) == 0)
>  			found++;
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 
Karsten Keil
SuSE Labs
ISDN development
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/