mbox series

[v2,0/4] PCI: dwc: pci-dra7xx: miscellaneous improvements

Message ID 20210531085934.2662457-1-luca@lucaceresoli.net
Headers show
Series PCI: dwc: pci-dra7xx: miscellaneous improvements | expand

Message

Luca Ceresoli May 31, 2021, 8:59 a.m. UTC 
This is an series of mixed improvements to the DRA7 PCI controller driver:
allow building as a loadabel module, allow to get and enable a clock and a
small cleanup.

Luca

Luca Ceresoli (4):
  PCI: dwc: Export more symbols to allow modular drivers
  PCI: dra7xx: Make it a kernel module
  PCI: dra7xx: Remove unused include
  PCI: dra7xx: Get an optional clock

 drivers/pci/controller/dwc/Kconfig            |  6 ++---
 drivers/pci/controller/dwc/pci-dra7xx.c       | 22 +++++++++++++++++--
 .../pci/controller/dwc/pcie-designware-ep.c   |  1 +
 drivers/pci/controller/dwc/pcie-designware.c  |  1 +
 4 files changed, 25 insertions(+), 5 deletions(-)

Comments

Lorenzo Pieralisi June 21, 2021, 2:41 p.m. UTC | #1 
On Mon, May 31, 2021 at 10:59:30AM +0200, Luca Ceresoli wrote:
> This is an series of mixed improvements to the DRA7 PCI controller driver:

> allow building as a loadabel module, allow to get and enable a clock and a

> small cleanup.

> 

> Luca

> 

> Luca Ceresoli (4):

>   PCI: dwc: Export more symbols to allow modular drivers

>   PCI: dra7xx: Make it a kernel module

>   PCI: dra7xx: Remove unused include

>   PCI: dra7xx: Get an optional clock

> 

>  drivers/pci/controller/dwc/Kconfig            |  6 ++---

>  drivers/pci/controller/dwc/pci-dra7xx.c       | 22 +++++++++++++++++--

>  .../pci/controller/dwc/pcie-designware-ep.c   |  1 +

>  drivers/pci/controller/dwc/pcie-designware.c  |  1 +

>  4 files changed, 25 insertions(+), 5 deletions(-)


Hi Kishon,

I'd need your ACK to proceed with this series that looks like it
is ready to go, please let me know.

Thanks,
Lorenzo
Lorenzo Pieralisi Aug. 13, 2021, 3:53 p.m. UTC | #2 
On Mon, Jun 21, 2021 at 03:41:09PM +0100, Lorenzo Pieralisi wrote:
> On Mon, May 31, 2021 at 10:59:30AM +0200, Luca Ceresoli wrote:

> > This is an series of mixed improvements to the DRA7 PCI controller driver:

> > allow building as a loadabel module, allow to get and enable a clock and a

> > small cleanup.

> > 

> > Luca

> > 

> > Luca Ceresoli (4):

> >   PCI: dwc: Export more symbols to allow modular drivers

> >   PCI: dra7xx: Make it a kernel module

> >   PCI: dra7xx: Remove unused include

> >   PCI: dra7xx: Get an optional clock

> > 

> >  drivers/pci/controller/dwc/Kconfig            |  6 ++---

> >  drivers/pci/controller/dwc/pci-dra7xx.c       | 22 +++++++++++++++++--

> >  .../pci/controller/dwc/pcie-designware-ep.c   |  1 +

> >  drivers/pci/controller/dwc/pcie-designware.c  |  1 +

> >  4 files changed, 25 insertions(+), 5 deletions(-)

> 

> Hi Kishon,

> 

> I'd need your ACK to proceed with this series that looks like it

> is ready to go, please let me know.


Still need it - please let me know.

Lorenzo
Luca Ceresoli Sept. 19, 2021, 9:03 p.m. UTC | #3 
Hi Lorenzo, Kishon,

On 13/08/21 17:53, Lorenzo Pieralisi wrote:
> On Mon, Jun 21, 2021 at 03:41:09PM +0100, Lorenzo Pieralisi wrote:

>> On Mon, May 31, 2021 at 10:59:30AM +0200, Luca Ceresoli wrote:

>>> This is an series of mixed improvements to the DRA7 PCI controller driver:

>>> allow building as a loadabel module, allow to get and enable a clock and a

>>> small cleanup.

>>>

>>> Luca

>>>

>>> Luca Ceresoli (4):

>>>   PCI: dwc: Export more symbols to allow modular drivers

>>>   PCI: dra7xx: Make it a kernel module

>>>   PCI: dra7xx: Remove unused include

>>>   PCI: dra7xx: Get an optional clock

>>>

>>>  drivers/pci/controller/dwc/Kconfig            |  6 ++---

>>>  drivers/pci/controller/dwc/pci-dra7xx.c       | 22 +++++++++++++++++--

>>>  .../pci/controller/dwc/pcie-designware-ep.c   |  1 +

>>>  drivers/pci/controller/dwc/pcie-designware.c  |  1 +

>>>  4 files changed, 25 insertions(+), 5 deletions(-)

>>

>> Hi Kishon,

>>

>> I'd need your ACK to proceed with this series that looks like it

>> is ready to go, please let me know.

> 

> Still need it - please let me know.

> 

> Lorenzo


Should I resend the series?

BTW it still applies cleanly on both the pci next branch and on mainline
master.

-- 
Luca
Kishon Vijay Abraham I Sept. 20, 2021, 6:29 a.m. UTC | #4 
On 20/09/21 2:33 am, Luca Ceresoli wrote:
> Hi Lorenzo, Kishon,

> 

> On 13/08/21 17:53, Lorenzo Pieralisi wrote:

>> On Mon, Jun 21, 2021 at 03:41:09PM +0100, Lorenzo Pieralisi wrote:

>>> On Mon, May 31, 2021 at 10:59:30AM +0200, Luca Ceresoli wrote:

>>>> This is an series of mixed improvements to the DRA7 PCI controller driver:

>>>> allow building as a loadabel module, allow to get and enable a clock and a

>>>> small cleanup.

>>>>

>>>> Luca

>>>>

>>>> Luca Ceresoli (4):

>>>>   PCI: dwc: Export more symbols to allow modular drivers

>>>>   PCI: dra7xx: Make it a kernel module

>>>>   PCI: dra7xx: Remove unused include

>>>>   PCI: dra7xx: Get an optional clock

>>>>

>>>>  drivers/pci/controller/dwc/Kconfig            |  6 ++---

>>>>  drivers/pci/controller/dwc/pci-dra7xx.c       | 22 +++++++++++++++++--

>>>>  .../pci/controller/dwc/pcie-designware-ep.c   |  1 +

>>>>  drivers/pci/controller/dwc/pcie-designware.c  |  1 +

>>>>  4 files changed, 25 insertions(+), 5 deletions(-)

>>>

>>> Hi Kishon,

>>>

>>> I'd need your ACK to proceed with this series that looks like it

>>> is ready to go, please let me know.

>>

>> Still need it - please let me know.

>>

>> Lorenzo

> 

> Should I resend the series?

> 

> BTW it still applies cleanly on both the pci next branch and on mainline

> master.


For the series
Acked-by: Kishon Vijay Abraham I <kishon@ti.com>
Lorenzo Pieralisi Sept. 28, 2021, 9:30 a.m. UTC | #5 
On Sun, Sep 19, 2021 at 11:03:00PM +0200, Luca Ceresoli wrote:
> Hi Lorenzo, Kishon,

> 

> On 13/08/21 17:53, Lorenzo Pieralisi wrote:

> > On Mon, Jun 21, 2021 at 03:41:09PM +0100, Lorenzo Pieralisi wrote:

> >> On Mon, May 31, 2021 at 10:59:30AM +0200, Luca Ceresoli wrote:

> >>> This is an series of mixed improvements to the DRA7 PCI controller driver:

> >>> allow building as a loadabel module, allow to get and enable a clock and a

> >>> small cleanup.

> >>>

> >>> Luca

> >>>

> >>> Luca Ceresoli (4):

> >>>   PCI: dwc: Export more symbols to allow modular drivers

> >>>   PCI: dra7xx: Make it a kernel module

> >>>   PCI: dra7xx: Remove unused include

> >>>   PCI: dra7xx: Get an optional clock

> >>>

> >>>  drivers/pci/controller/dwc/Kconfig            |  6 ++---

> >>>  drivers/pci/controller/dwc/pci-dra7xx.c       | 22 +++++++++++++++++--

> >>>  .../pci/controller/dwc/pcie-designware-ep.c   |  1 +

> >>>  drivers/pci/controller/dwc/pcie-designware.c  |  1 +

> >>>  4 files changed, 25 insertions(+), 5 deletions(-)

> >>

> >> Hi Kishon,

> >>

> >> I'd need your ACK to proceed with this series that looks like it

> >> is ready to go, please let me know.

> > 

> > Still need it - please let me know.

> > 

> > Lorenzo

> 

> Should I resend the series?

> 

> BTW it still applies cleanly on both the pci next branch and on mainline

> master.


Kishon, please consider ACKing or otherwise comment on this series
to make some progress.

Thanks,
Lorenzo
Kishon Vijay Abraham I Sept. 28, 2021, 9:41 a.m. UTC | #6 
Hi Lorenzo,

On 28/09/21 3:00 pm, Lorenzo Pieralisi wrote:
> On Sun, Sep 19, 2021 at 11:03:00PM +0200, Luca Ceresoli wrote:

>> Hi Lorenzo, Kishon,

>>

>> On 13/08/21 17:53, Lorenzo Pieralisi wrote:

>>> On Mon, Jun 21, 2021 at 03:41:09PM +0100, Lorenzo Pieralisi wrote:

>>>> On Mon, May 31, 2021 at 10:59:30AM +0200, Luca Ceresoli wrote:

>>>>> This is an series of mixed improvements to the DRA7 PCI controller driver:

>>>>> allow building as a loadabel module, allow to get and enable a clock and a

>>>>> small cleanup.

>>>>>

>>>>> Luca

>>>>>

>>>>> Luca Ceresoli (4):

>>>>>   PCI: dwc: Export more symbols to allow modular drivers

>>>>>   PCI: dra7xx: Make it a kernel module

>>>>>   PCI: dra7xx: Remove unused include

>>>>>   PCI: dra7xx: Get an optional clock

>>>>>

>>>>>  drivers/pci/controller/dwc/Kconfig            |  6 ++---

>>>>>  drivers/pci/controller/dwc/pci-dra7xx.c       | 22 +++++++++++++++++--

>>>>>  .../pci/controller/dwc/pcie-designware-ep.c   |  1 +

>>>>>  drivers/pci/controller/dwc/pcie-designware.c  |  1 +

>>>>>  4 files changed, 25 insertions(+), 5 deletions(-)

>>>>

>>>> Hi Kishon,

>>>>

>>>> I'd need your ACK to proceed with this series that looks like it

>>>> is ready to go, please let me know.

>>>

>>> Still need it - please let me know.

>>>

>>> Lorenzo

>>

>> Should I resend the series?

>>

>> BTW it still applies cleanly on both the pci next branch and on mainline

>> master.

> 

> Kishon, please consider ACKing or otherwise comment on this series

> to make some progress.


I had already ACKed this series
https://lore.kernel.org/all/598ee439-6882-5021-1351-f403e762430f@ti.com/

Anyways I'm adding it here again:
Acked-by: Kishon Vijay Abraham I <kishon@ti.com>


Thank You,
Kishon
> 

> Thanks,

> Lorenzo

>
Lorenzo Pieralisi Sept. 30, 2021, 8:53 a.m. UTC | #7 
On Mon, 31 May 2021 10:59:30 +0200, Luca Ceresoli wrote:
> This is an series of mixed improvements to the DRA7 PCI controller driver:

> allow building as a loadabel module, allow to get and enable a clock and a

> small cleanup.

> 

> Luca

> 

> Luca Ceresoli (4):

>   PCI: dwc: Export more symbols to allow modular drivers

>   PCI: dra7xx: Make it a kernel module

>   PCI: dra7xx: Remove unused include

>   PCI: dra7xx: Get an optional clock

> 

> [...]


Applied to pci/dwc, thanks!

[1/4] PCI: dwc: Export more symbols to allow modular drivers
      https://git.kernel.org/lpieralisi/pci/c/3a7fb86758
[2/4] PCI: dra7xx: Make it a kernel module
      https://git.kernel.org/lpieralisi/pci/c/3b868d150e
[3/4] PCI: dra7xx: Remove unused include
      https://git.kernel.org/lpieralisi/pci/c/b9a6943dc8
[4/4] PCI: dra7xx: Get an optional clock
      https://git.kernel.org/lpieralisi/pci/c/5af9405397

Thanks,
Lorenzo