Re: [PATCH 5/10] drivers/char: pci_find_device remove (drivers/char/specialix.c)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Sep 10, 2005 at 11:00:34PM +0200, Jiri Slaby wrote:
> Jeff Garzik napsal(a):
> >Jiri Slaby wrote:
> >
> >>Signed-off-by: Jiri Slaby <[email protected]>
> >>
> >> specialix.c |    9 ++++++---
> >> 1 files changed, 6 insertions(+), 3 deletions(-)
> >>
> >>diff --git a/drivers/char/specialix.c b/drivers/char/specialix.c
> >>--- a/drivers/char/specialix.c
> >>+++ b/drivers/char/specialix.c
> >>@@ -2502,9 +2502,9 @@ static int __init specialix_init(void)
> >>                 i++;
> >>                 continue;
> >>             }
> >>-            pdev = pci_find_device (PCI_VENDOR_ID_SPECIALIX, 
> >>-                                    PCI_DEVICE_ID_SPECIALIX_IO8, 
> >>-                                    pdev);
> >>+            pdev = pci_get_device (PCI_VENDOR_ID_SPECIALIX,
> >>+                    PCI_DEVICE_ID_SPECIALIX_IO8,
> >>+                    pdev);
> >>             if (!pdev) break;
> >> 
> >>             if (pci_enable_device(pdev))
> >>@@ -2517,7 +2517,10 @@ static int __init specialix_init(void)
> >>             sx_board[i].flags |= SX_BOARD_IS_PCI;
> >>             if (!sx_probe(&sx_board[i]))
> >>                 found ++;
> >>+
> >>         }
> >>+        if (i >= SX_NBOARD)
> >>+            pci_dev_put(pdev);
> >
> >
> >should be converted to PCI probing, rather than this.
> I won't do that, i did that for 2 drivers and nobody was interested in 
> that (and its much time left for nothing). These (unrewritten) drivers 
> would be deleted in some time. Greg wants simply wipe this function out.

No, I want it done correctly.  If I simply wanted the function removed,
I would have done this kind of wholesale conversion a long time ago.

If the code needs to be converted to the proper pci probing logic,
that's the better way to do it, and that's what should be done.

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]
  Powered by Linux