mbox series

[v2,0/8] mhi: pci_generic: Misc improvements

Message ID 1606140666-4986-1-git-send-email-loic.poulain@linaro.org
Headers show
Series mhi: pci_generic: Misc improvements | expand

Message

Loic Poulain Nov. 23, 2020, 2:10 p.m. UTC
This series adjust some configuration values to ensure stability and
robustness of mhi pci devices (timeout, number of events, burst mode).

It also includes support for system sleep as well as a recovery procedure
that can be triggered when a PCI error is reported, either by PCI AER or by
the new health-check mechanism.

All these changes have been tested with Telit FN980m module.

Loic Poulain (8):
  mhi: pci-generic: Increase number of hardware events
  mhi: pci-generic: Perform hard reset on remove
  mhi: pci_generic: Enable burst mode for hardware channels
  mhi: pci_generic: Add support for reset
  mhi: pci_generic: Add suspend/resume/recovery procedure
  mhi: pci_generic: Add PCI error handlers
  mhi: pci_generic: Add health-check
  mhi: pci_generic: Increase controller timeout value

 drivers/bus/mhi/pci_generic.c | 352 ++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 335 insertions(+), 17 deletions(-)

-- 
2.7.4

Comments

Bhaumik Bhatt Nov. 24, 2020, 12:52 a.m. UTC | #1
On 2020-11-23 06:10 AM, Loic Poulain wrote:
> This series adjust some configuration values to ensure stability and

> robustness of mhi pci devices (timeout, number of events, burst mode).

> 

> It also includes support for system sleep as well as a recovery 

> procedure

> that can be triggered when a PCI error is reported, either by PCI AER 

> or by

> the new health-check mechanism.

> 

> All these changes have been tested with Telit FN980m module.

> 

> Loic Poulain (8):

>   mhi: pci-generic: Increase number of hardware events

>   mhi: pci-generic: Perform hard reset on remove

>   mhi: pci_generic: Enable burst mode for hardware channels

>   mhi: pci_generic: Add support for reset

>   mhi: pci_generic: Add suspend/resume/recovery procedure

>   mhi: pci_generic: Add PCI error handlers

>   mhi: pci_generic: Add health-check

>   mhi: pci_generic: Increase controller timeout value

> 

>  drivers/bus/mhi/pci_generic.c | 352 

> ++++++++++++++++++++++++++++++++++++++++--

>  1 file changed, 335 insertions(+), 17 deletions(-)

Just noticed that you may have missed my reviewed-by on patches 1 and 8 
from the
previous submission.

Thanks,
Bhaumik
---
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora 
Forum,
a Linux Foundation Collaborative Project
Loic Poulain Nov. 24, 2020, 8:14 a.m. UTC | #2
Hi Bhaumik,

On Tue, 24 Nov 2020 at 01:52, Bhaumik Bhatt <bbhatt@codeaurora.org> wrote:
>

> On 2020-11-23 06:10 AM, Loic Poulain wrote:

> > This series adjust some configuration values to ensure stability and

> > robustness of mhi pci devices (timeout, number of events, burst mode).

> >

> > It also includes support for system sleep as well as a recovery

> > procedure

> > that can be triggered when a PCI error is reported, either by PCI AER

> > or by

> > the new health-check mechanism.

> >

> > All these changes have been tested with Telit FN980m module.

> >

> > Loic Poulain (8):

> >   mhi: pci-generic: Increase number of hardware events

> >   mhi: pci-generic: Perform hard reset on remove

> >   mhi: pci_generic: Enable burst mode for hardware channels

> >   mhi: pci_generic: Add support for reset

> >   mhi: pci_generic: Add suspend/resume/recovery procedure

> >   mhi: pci_generic: Add PCI error handlers

> >   mhi: pci_generic: Add health-check

> >   mhi: pci_generic: Increase controller timeout value

> >

> >  drivers/bus/mhi/pci_generic.c | 352

> > ++++++++++++++++++++++++++++++++++++++++--

> >  1 file changed, 335 insertions(+), 17 deletions(-)

> Just noticed that you may have missed my reviewed-by on patches 1 and 8

> from the

> previous submission.


I don't find any reply to 1/8 or 8/8, could you please resubmit?

Regards,
Loic

>

> Thanks,

> Bhaumik

> ---

> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora

> Forum,

> a Linux Foundation Collaborative Project