Re: 2.5.8 compile bugs

Jens Axboe (axboe@suse.de)
Mon, 15 Apr 2002 13:51:31 +0200


On Mon, Apr 15 2002, Roman Zippel wrote:
> > This should fix it.
> >
> > --- drivers/ide/ide.c~ 2002-04-15 09:05:58.000000000 +0200
> > +++ drivers/ide/ide.c 2002-04-15 09:06:52.000000000 +0200
> > @@ -2701,7 +2701,11 @@
> >
> > void ide_teardown_commandlist(ide_drive_t *drive)
> > {
> > +#ifdef CONFIG_BLK_DEV_IDEPCI
> > struct pci_dev *pdev= drive->channel->pci_dev;
> > +#else
> > + struct pci_dev *pdev = NULL;
> > +#endif
> > struct list_head *entry;
> >
> > list_for_each(entry, &drive->free_req) {
> > @@ -2716,7 +2720,11 @@
> >
> > int ide_build_commandlist(ide_drive_t *drive)
> > {
> > +#ifdef CONFIG_BLK_DEV_IDEPCI
> > struct pci_dev *pdev= drive->channel->pci_dev;
> > +#else
> > + struct pci_dev *pdev = NULL;
> > +#endif
> > struct ata_request *ar;
> > ide_tag_info_t *tcq;
> > int i, err;
>
> That's not enough, some archs don't define pci_alloc_consistent/
> pci_free_consistent, because they have neither PCI nor ISA.

Please, then those archs need to provide similar functionality. This is
the established api, unless you want to change the documentation and xxx
number of drivers?

-- 
Jens Axboe

- 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/