linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Perrot <thomas.perrot@bootlin.com>
To: linux-arm-msm@vger.kernel.org
Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
	hemantk@codeaurora.org, loic.poulain@linaro.org,
	stable@vger.kernel.org
Subject: Re: [PATCH] bus: mhi: pci_generic: increase timeout value for operations to 24000ms
Date: Tue, 28 Sep 2021 22:11:12 +0200	[thread overview]
Message-ID: <5e1c5979b7fd35aa5715ed9b25ca3a59b7b259f4.camel@bootlin.com> (raw)
In-Reply-To: <20210816042206.GA3637@thinkpad>

[-- Attachment #1: Type: text/plain, Size: 1526 bytes --]

Hello Manivannan,

I just saw that this patch seems not yet been merged, is there a issue
with it?

Best regards,
Thomas

On Mon, 2021-08-16 at 09:52 +0530, Manivannan Sadhasivam wrote:
> On Thu, Aug 05, 2021 at 04:02:31PM +0200, Thomas Perrot wrote:
> > Otherwise, the waiting time was too short to use a Sierra Wireless
> > EM919X
> > connected to an i.MX6 through the PCIe bus.
> > 
> > Signed-off-by: Thomas Perrot <thomas.perrot@bootlin.com>
> 
> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> 
> Thanks,
> Mani
> 
> > ---
> >  drivers/bus/mhi/pci_generic.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/bus/mhi/pci_generic.c
> > b/drivers/bus/mhi/pci_generic.c
> > index 4dd1077354af..e08ed6e5031b 100644
> > --- a/drivers/bus/mhi/pci_generic.c
> > +++ b/drivers/bus/mhi/pci_generic.c
> > @@ -248,7 +248,7 @@ static struct mhi_event_config
> > modem_qcom_v1_mhi_events[] = {
> >  
> >  static const struct mhi_controller_config
> > modem_qcom_v1_mhiv_config = {
> >         .max_channels = 128,
> > -       .timeout_ms = 8000,
> > +       .timeout_ms = 24000,
> >         .num_channels = ARRAY_SIZE(modem_qcom_v1_mhi_channels),
> >         .ch_cfg = modem_qcom_v1_mhi_channels,
> >         .num_events = ARRAY_SIZE(modem_qcom_v1_mhi_events),
> > -- 
> > 2.31.1
> > 

-- 
Thomas Perrot, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  reply	other threads:[~2021-09-28 20:19 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-05 14:02 [PATCH] bus: mhi: pci_generic: increase timeout value for operations to 24000ms Thomas Perrot
2021-08-05 14:30 ` Loic Poulain
2021-08-16  4:22 ` Manivannan Sadhasivam
2021-09-28 20:11   ` Thomas Perrot [this message]
2021-09-29 11:58     ` Manivannan Sadhasivam
2021-09-29  9:37 ` Aleksander Morgado
2021-09-29 12:04   ` Manivannan Sadhasivam
2021-09-29 13:17     ` Aleksander Morgado
2021-09-30  8:07       ` Thomas Perrot
2021-09-30 10:19         ` Loic Poulain
2021-10-01  7:38           ` Manivannan Sadhasivam
2021-10-01  7:36         ` Manivannan Sadhasivam
2021-10-01  7:35       ` Manivannan Sadhasivam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5e1c5979b7fd35aa5715ed9b25ca3a59b7b259f4.camel@bootlin.com \
    --to=thomas.perrot@bootlin.com \
    --cc=hemantk@codeaurora.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=loic.poulain@linaro.org \
    --cc=manivannan.sadhasivam@linaro.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).