Re: [PATCH] add PCI ROMs to sysfs
From: Jesse Barnes (jbarnes_at_engr.sgi.com)
Date: 07/30/04
- Previous message: Greg KH: "Re: [PATCH] add PCI ROMs to sysfs"
- In reply to: Greg KH: "Re: [PATCH] add PCI ROMs to sysfs"
- Next in thread: Jesse Barnes: "Re: [PATCH] add PCI ROMs to sysfs"
- Reply: Jesse Barnes: "Re: [PATCH] add PCI ROMs to sysfs"
- Reply: Jon Smirl: "Re: [PATCH] add PCI ROMs to sysfs"
- Reply: Greg KH: "Re: [PATCH] add PCI ROMs to sysfs"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
To: Greg KH <greg@kroah.com> Date: Fri, 30 Jul 2004 14:34:50 -0700
On Friday, July 30, 2004 2:29 pm, Greg KH wrote:
> On Fri, Jul 30, 2004 at 02:09:05PM -0700, Jesse Barnes wrote:
> > Thoughts? I've tried to add cleanup code, but I'm not sure how
> > acceptable it is and I don't have any way of testing it.
>
> You don't have access to a cardbus machine? Or how about using the
> fakephp driver to "remove" pci devices? You should be able to test this
> code path with either one of those methods...
I've never tried fakephp, I'll look into it. And I do have a laptop at home
that I could mess around on too, though I don't think I have any devices with
ROMs...
> > +void pci_remove_sysfs_dev_files(struct pci_dev *pdev)
> > +{
> > + /* Don't need to free config entries since they're static & global */
>
> What do you mean by this? You should still remove all files we added in
> the pci_create_sysfs_dev_files() function here, not just the rom file.
What will happen if we don't? They didn't used to be removed, and there's
only one global copy afaict. I'll add code to kill them at any rate.
Thanks,
Jesse
-
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/
- Previous message: Greg KH: "Re: [PATCH] add PCI ROMs to sysfs"
- In reply to: Greg KH: "Re: [PATCH] add PCI ROMs to sysfs"
- Next in thread: Jesse Barnes: "Re: [PATCH] add PCI ROMs to sysfs"
- Reply: Jesse Barnes: "Re: [PATCH] add PCI ROMs to sysfs"
- Reply: Jon Smirl: "Re: [PATCH] add PCI ROMs to sysfs"
- Reply: Greg KH: "Re: [PATCH] add PCI ROMs to sysfs"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
|
