mbox series

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

Message ID 20210531085934.2662457-1-luca@lucaceresoli.net (mailing list archive)
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.
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