Messages in this thread | | | Date | Fri, 30 Jul 2004 14:39:25 -0700 | From | Greg KH <> | Subject | Re: [PATCH] add PCI ROMs to sysfs |
| |
On Fri, Jul 30, 2004 at 02:34:50PM -0700, Jesse Barnes wrote: > > > +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?
Then the driver core will clean them up, like it does today :)
But it's "not nice", and we should clean them up as we now have a function in which we can do that, and based on changes we have planned for the driver core in the future, it will be something that is required to do later on. Might as well do it now, as you are modifying the same code...
thanks,
greg k-h - 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/
| |