lkml.org 
[lkml]   [2001]   [Sep]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] parport_pc.c PnP BIOS handling code
Attached is a patch which, although independent of the
PnP BIOS patch I posted earlier, is the companion of
that patch. This patch modifies init_pnp040x() so
that it checks for the IORESOURCE_UNSET bit which
the PnP BIOS driver uses to indicate that a given
"struct resource" is vacuous.

The parport driver's mysterious rejection of dma==0,
which I asked about before, I now suspect to have been a
workaround for bugs in pnpbios which resulted in pnpbios
reporting dma=0 when in fact the PnP BIOS had reported
no dma resource at all or else dma disabled. Now that
those bugs are getting fixed, it seems okay to allow
the use of DMA0, and the patch does so.

--
Thomas Hood--- linux-2.4.9-ac13-mwave/drivers/parport/parport_pc.c_ORIG Fri Sep 21 15:23:19 2001
+++ linux-2.4.9-ac13-mwave/drivers/parport/parport_pc.c Tue Sep 25 23:10:17 2001
@@ -2797,26 +2797,66 @@
return count;
}

#if defined (CONFIG_PNPBIOS) || defined (CONFIG_PNPBIOS_MODULE)

+#define UNSET(res) ((res).flags & IORESOURCE_UNSET)
+
int init_pnp040x(struct pci_dev *dev)
-{ int io,iohi,irq,dma;
+{
+ int io,iohi,irq,dma;
+
+ printk(KERN_INFO
+ "parport: PnP BIOS reports device %s %s (node number 0x%x) is ",
+ dev->name, dev->slot_name, dev->devfn
+ );
+
+ if ( UNSET(dev->resource[0]) ) {
+ printk("not configured.\n");
+ return 0;
+ }
+ io = dev->resource[0].start;
+ printk("configured to use io 0x%04x",io);
+ if ( UNSET(dev->resource[1]) ) {
+ iohi = 0;
+ } else {
+ iohi = dev->resource[1].start;
+ printk(", io 0x%04x",iohi);
+ }

- io=dev->resource[0].start;
- iohi=dev->resource[1].start;
- irq=dev->irq_resource[0].start;
- dma=dev->dma_resource[0].start;
+ if ( UNSET(dev->irq_resource[0]) ) {
+ irq = PARPORT_IRQ_NONE;
+ } else {
+ if ( dev->irq_resource[0].start == -1 ) {
+ irq = PARPORT_IRQ_NONE;
+ printk(", irq disabled");
+ } else {
+ irq = dev->irq_resource[0].start;
+ printk(", irq %d",irq);
+ }
+ }

- if(dma==0) dma=-1;
+ if ( UNSET(dev->dma_resource[0]) ) {
+ dma = PARPORT_DMA_NONE;
+ } else {
+ if ( dev->dma_resource[0].start == -1 ) {
+ dma = PARPORT_DMA_NONE;
+ printk(", dma disabled");
+ } else {
+ dma = dev->dma_resource[0].start;
+ printk(", dma %d",irq);
+ }
+ }
+
+ printk("\n");

- printk(KERN_INFO "PnPBIOS: Parport found %s %s at io=%04x,%04x irq=%d dma=%d\n",
- dev->name,dev->slot_name,io,iohi,irq,dma);
if (parport_pc_probe_port(io,iohi,irq,dma,NULL))
return 1;
+
return 0;
}
+#undef UNSET

#endif

/* This function is called by parport_pc_init if the user didn't
* specify any ports to probe. Its job is to find some ports. Order
\
 
 \ /
  Last update: 2005-03-22 13:03    [W:1.763 / U:0.548 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site