Re: [RFC PATCH V5 1/5] PCI: Embed pci_ecam_ops in pci_config_window structure

2016-09-04 Thread Bjorn Helgaas
On Fri, Sep 02, 2016 at 04:38:45PM +0100, Lorenzo Pieralisi wrote: > On Thu, Sep 01, 2016 at 01:23:45PM -0500, Bjorn Helgaas wrote: > > On Mon, Aug 08, 2016 at 03:05:37PM +0200, Tomasz Nowicki wrote: > > > pci_config_window keeps pointer to pci_ecam_ops and every time > > > we want to deallocate pc

Re: [RFC PATCH V5 1/5] PCI: Embed pci_ecam_ops in pci_config_window structure

2016-09-02 Thread Lorenzo Pieralisi
On Thu, Sep 01, 2016 at 01:23:45PM -0500, Bjorn Helgaas wrote: > On Mon, Aug 08, 2016 at 03:05:37PM +0200, Tomasz Nowicki wrote: > > pci_config_window keeps pointer to pci_ecam_ops and every time > > we want to deallocate pci_config_window (pci_ecam_free()) we need to make > > sure to free pci_ecam

Re: [RFC PATCH V5 1/5] PCI: Embed pci_ecam_ops in pci_config_window structure

2016-09-01 Thread Bjorn Helgaas
On Mon, Aug 08, 2016 at 03:05:37PM +0200, Tomasz Nowicki wrote: > pci_config_window keeps pointer to pci_ecam_ops and every time > we want to deallocate pci_config_window (pci_ecam_free()) we need to make > sure to free pci_ecam_ops in case it was dynamically allocated prior to > pci_ecam_create()

[RFC PATCH V5 1/5] PCI: Embed pci_ecam_ops in pci_config_window structure

2016-08-08 Thread Tomasz Nowicki
pci_config_window keeps pointer to pci_ecam_ops and every time we want to deallocate pci_config_window (pci_ecam_free()) we need to make sure to free pci_ecam_ops in case it was dynamically allocated prior to pci_ecam_create() call. To avoid that extra effort, embed pci_ecam_ops in pci_config_wind