Re: [PATCH 4/6] firewire: OHCI-1394 lowlevel driver

2007-05-09 Thread Kristian Høgsberg
Christoph Hellwig wrote: + if (pci_enable_device(dev)) { + fw_error("Failed to enable OHCI hardware.\n"); + return cleanup(ohci, CLEANUP_PUT_CARD, -ENODEV); Please use normal goto unwinding so the driver follows the same model as almost all other pci drivers an

Re: [PATCH 4/6] firewire: OHCI-1394 lowlevel driver

2007-05-04 Thread Christoph Hellwig
> +/* -- pci subsystem interface -- */ > + > +enum { > + CLEANUP_SELF_ID, > + CLEANUP_REGISTERS, > + CLEANUP_IOMEM, > + CLEANUP_DISABLE, > + CLEANUP_PUT_CARD, > +}; > + > +static int cleanup(struct fw_ohci *ohci, int stage, int code) > +{ > + struct pci_dev *